Commit ab4bb65e authored by Svetoslav's avatar Svetoslav Committed by Android Git Automerger
Browse files

am 3c44cc54: Display properly runtime permissions without a corresponding app op.

* commit '3c44cc54':
  Display properly runtime permissions without a corresponding app op.
parents 5d2d9baa 3c44cc54
......@@ -275,8 +275,9 @@ public final class AppPermissionGroup implements Comparable<AppPermissionGroup>
if (permission.isGranted()) {
return true;
}
} else if (permission.isGranted() && permission.getAppOp()
!= AppOpsManager.OP_NONE && permission.isAppOpAllowed()) {
} else if (permission.isGranted() && ((permission.getAppOp()
!= AppOpsManager.OP_NONE && permission.isAppOpAllowed())
|| permission.getAppOp() == AppOpsManager.OP_NONE)) {
return true;
}
}
......
......@@ -84,13 +84,6 @@ public class Utils {
return false;
}
// Yes this is possible. We have leftover permissions that
// are not in the final groups and we want to get rid of,
// therefore we do not have app ops for legacy support.
if (!group.hasRuntimePermission() && !group.hasAppOpPermission()) {
return false;
}
final boolean isPlatformPermission = group.getDeclaringPackage().equals(OS_PKG);
// Show legacy permissions only if the user chose that.
if (isPlatformPermission
......@@ -108,12 +101,6 @@ public class Utils {
return false;
}
// Yes this is possible. We have leftover permissions that
// are not in the final groups and we want to get rid of,
// therefore we do not have app ops for legacy support.
if (!app.hasRuntimePermissions() && !app.hasAppOpPermissions()) {
return false;
}
return true;
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment