diff options
author | Svet Ganov <svetoslavganov@google.com> | 2015-05-14 19:34:07 -0700 |
---|---|---|
committer | Svet Ganov <svetoslavganov@google.com> | 2015-05-14 19:42:08 -0700 |
commit | 4ef4881c8625437083c4fee36a68b235c5c36b2b (patch) | |
tree | a916f3709bbc72ef64a8f94feb8d6319b510a280 /src/com/android/packageinstaller/permission/model/PermissionGroup.java | |
parent | e80b9b7f4d4220e0b3b32a7f676fe805f87006c4 (diff) | |
download | android_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.java | 17 |
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); } } |