summaryrefslogtreecommitdiffstats
path: root/src/com/android/packageinstaller/permission/model/PermissionGroup.java
diff options
context:
space:
mode:
authorSvet Ganov <svetoslavganov@google.com>2015-05-14 19:34:07 -0700
committerSvet Ganov <svetoslavganov@google.com>2015-05-14 19:42:08 -0700
commit4ef4881c8625437083c4fee36a68b235c5c36b2b (patch)
treea916f3709bbc72ef64a8f94feb8d6319b510a280 /src/com/android/packageinstaller/permission/model/PermissionGroup.java
parente80b9b7f4d4220e0b3b32a7f676fe805f87006c4 (diff)
downloadandroid_packages_apps_PackageInstaller-4ef4881c8625437083c4fee36a68b235c5c36b2b.tar.gz
android_packages_apps_PackageInstaller-4ef4881c8625437083c4fee36a68b235c5c36b2b.tar.bz2
android_packages_apps_PackageInstaller-4ef4881c8625437083c4fee36a68b235c5c36b2b.zip
Don't show permission with no app ops.
We have some permissions that we will be getting rid of and for that reason do not have app ops. Such permission were showing in the UI for legacy apps but changing them has no effect because there is no app ops support. We are not not showing them in the UI to avoid user frustration. bug:21168530 Change-Id: I2b6988f003e27b266fb2b48ddbb692ae5cf131fb
Diffstat (limited to 'src/com/android/packageinstaller/permission/model/PermissionGroup.java')
-rw-r--r--src/com/android/packageinstaller/permission/model/PermissionGroup.java17
1 files changed, 14 insertions, 3 deletions
diff --git a/src/com/android/packageinstaller/permission/model/PermissionGroup.java b/src/com/android/packageinstaller/permission/model/PermissionGroup.java
index 9e90d4ff..901f1a4e 100644
--- a/src/com/android/packageinstaller/permission/model/PermissionGroup.java
+++ b/src/com/android/packageinstaller/permission/model/PermissionGroup.java
@@ -173,8 +173,19 @@ public final class PermissionGroup implements Comparable<PermissionGroup> {
: com.android.internal.R.drawable.ic_perm_device_info;
}
- public boolean isAppOpPermission() {
- return !mAppSupportsRuntimePermissions;
+ public boolean hasRuntimePermission() {
+ return mAppSupportsRuntimePermissions;
+ }
+
+ public boolean hasAppOpPermission() {
+ final int permissionCount = mPermissions.size();
+ for (int i = 0; i < permissionCount; i++) {
+ Permission permission = mPermissions.valueAt(i);
+ if (permission.getAppOp() != AppOpsManager.OP_NONE) {
+ return true;
+ }
+ }
+ return false;
}
public PackageInfo getApp() {
@@ -512,7 +523,7 @@ public final class PermissionGroup implements Comparable<PermissionGroup> {
return builder.toString();
}
- void addPermission(Permission permission) {
+ private void addPermission(Permission permission) {
mPermissions.put(permission.getName(), permission);
}
}