1 files changed, 5 insertions, 14 deletions
diff --git a/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java b/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java
index fdb77934..050e8b1e 100644
--- a/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java
+++ b/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java
@@ -215,6 +215,9 @@ public final class AppPermissionGroup implements Comparable<AppPermissionGroup>
     }
 
     public boolean isReviewRequired() {
+        if (mAppSupportsRuntimePermissions) {
+            return false;
+        }
         final int permissionCount = mPermissions.size();
         for (int i = 0; i < permissionCount; i++) {
             Permission permission = mPermissions.valueAt(i);
@@ -250,17 +253,6 @@ public final class AppPermissionGroup implements Comparable<AppPermissionGroup>
         return false;
     }
 
-    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() {
         return mPackageInfo;
     }
@@ -316,9 +308,8 @@ public final class AppPermissionGroup implements Comparable<AppPermissionGroup>
                 if (permission.isGranted()) {
                     return true;
                 }
-            } else if (permission.isGranted() && ((permission.getAppOp()
-                    == AppOpsManager.OP_NONE || permission.isAppOpAllowed()))
-                    && !permission.isReviewRequired()) {
+            } else if (permission.isGranted() && (permission.getAppOp()
+                    == AppOpsManager.OP_NONE || permission.isAppOpAllowed())) {
                 return true;
             }
         }