From 15fa03702aae07fb325d6be98d8c0cdeb344d20d Mon Sep 17 00:00:00 2001
From: Svetoslav Ganov <svetoslavganov@google.com>
Date: Mon, 21 Mar 2016 17:06:54 -0700
Subject: Don't show app defined permissions in the review UI

Only platform defined runtime permissions have corresponding
app ops, hence there is no way to disable functionality guarded
by app defined permissions. Therefore, app defined permissions
should not show in the review UI.

bug:27102458

Change-Id: Iedc7c4de9216007176b87cfecaeed69dbadc2068
---
 .../permission/ui/ReviewPermissionsActivity.java   | 23 ++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

(limited to 'src/com/android/packageinstaller/permission/ui')

diff --git a/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java b/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java
index 38080e6b..f31d618e 100644
--- a/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java
+++ b/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java
@@ -120,13 +120,18 @@ public final class ReviewPermissionsActivity extends Activity
         public void onCreate(@Nullable Bundle savedInstanceState) {
             super.onCreate(savedInstanceState);
 
+            Activity activity = getActivity();
+            if (activity == null) {
+                return;
+            }
+
             PackageInfo packageInfo = getArguments().getParcelable(EXTRA_PACKAGE_INFO);
             if (packageInfo == null) {
-                getActivity().finish();
+                activity.finish();
                 return;
             }
 
-            mAppPermissions = new AppPermissions(getActivity(), packageInfo, null, false,
+            mAppPermissions = new AppPermissions(activity, packageInfo, null, false,
                     new Runnable() {
                         @Override
                         public void run() {
@@ -135,7 +140,7 @@ public final class ReviewPermissionsActivity extends Activity
                     });
 
             if (mAppPermissions.getPermissionGroups().isEmpty()) {
-                getActivity().finish();
+                activity.finish();
                 return;
             }
 
@@ -148,7 +153,7 @@ public final class ReviewPermissionsActivity extends Activity
             }
 
             if (!reviewRequired) {
-                getActivity().finish();
+                activity.finish();
             }
         }
 
@@ -166,14 +171,18 @@ public final class ReviewPermissionsActivity extends Activity
 
         @Override
         public void onClick(View view) {
+            Activity activity = getActivity();
+            if (activity == null) {
+                return;
+            }
             if (view == mContinueButton) {
                 confirmPermissionsReview();
                 executeCallback(true);
             } else if (view == mCancelButton) {
                 executeCallback(false);
-                getActivity().setResult(Activity.RESULT_CANCELED);
+                activity.setResult(Activity.RESULT_CANCELED);
             }
-            getActivity().finish();
+            activity.finish();
         }
 
         @Override
@@ -185,6 +194,8 @@ public final class ReviewPermissionsActivity extends Activity
                 SwitchPreference switchPreference = (SwitchPreference) preference;
                 if (switchPreference.isChecked()) {
                     showWarnRevokeDialog(switchPreference.getKey());
+                } else {
+                    return true;
                 }
             }
             return false;
-- 
cgit v1.2.3