From 5ae831d84f03d360f2e087762e4ceb8bdc5a0142 Mon Sep 17 00:00:00 2001
From: Svetoslav Ganov <svetoslavganov@google.com>
Date: Wed, 23 Mar 2016 19:18:55 +0000
Subject: Revert "Respect past user choices in permisson review UI"

This reverts commit 437a6bfedb33daf66592bbed8595025f3e707850.

Change-Id: I5a46f94aadb0ab6dcbe85761f3b1390749b8b1cf
---
 .../permission/model/AppPermissionGroup.java       | 14 -------------
 .../permission/ui/ReviewPermissionsActivity.java   | 23 +++++-----------------
 2 files changed, 5 insertions(+), 32 deletions(-)

(limited to 'src')

diff --git a/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java b/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java
index 11d89341..0771a0f4 100644
--- a/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java
+++ b/src/com/android/packageinstaller/permission/model/AppPermissionGroup.java
@@ -321,20 +321,6 @@ public final class AppPermissionGroup implements Comparable<AppPermissionGroup>
         return false;
     }
 
-    public boolean shouldRevokeOnUpgrade() {
-        if (mAppSupportsRuntimePermissions) {
-            return false;
-        }
-        final int permissionCount = mPermissions.size();
-        for (int i = 0; i < permissionCount; i++) {
-            Permission permission = mPermissions.valueAt(i);
-            if ((permission.getFlags() & PackageManager.FLAG_PERMISSION_REVOKE_ON_UPGRADE) != 0) {
-                return true;
-            }
-        }
-        return false;
-    }
-
     public boolean grantRuntimePermissions(boolean fixedByTheUser) {
         return grantRuntimePermissions(fixedByTheUser, null);
     }
diff --git a/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java b/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java
index ae656ebf..38080e6b 100644
--- a/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java
+++ b/src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java
@@ -229,9 +229,6 @@ public final class ReviewPermissionsActivity extends Activity
 
         private void bindUi() {
             Activity activity = getActivity();
-            if (activity == null) {
-                return;
-            }
 
             // Set icon
             Drawable icon = mAppPermissions.getPackageInfo().applicationInfo.loadIcon(
@@ -271,11 +268,6 @@ public final class ReviewPermissionsActivity extends Activity
         }
 
         private void loadPreferences() {
-            Activity activity = getActivity();
-            if (activity == null) {
-                return;
-            }
-
             PreferenceScreen screen = getPreferenceScreen();
             if (screen == null) {
                 screen = getPreferenceManager().createPreferenceScreen(getActivity());
@@ -308,17 +300,15 @@ public final class ReviewPermissionsActivity extends Activity
 
                     // We update permission grants based on the final preference states
                     if (group.isReviewRequired()) {
-                        // If review is required use granted as default unless
-                        // the user went to settings first and expressed that
-                        // the permission should be revoked for this app.
-                        preference.setChecked(!group.shouldRevokeOnUpgrade());
+                        // If review is required use granted as default
+                        preference.setChecked(true);
                     } else {
                         // If review not required use the current grant state as default
                         preference.setChecked(group.areRuntimePermissionsGranted());
                     }
 
                     preference.setKey(group.getName());
-                    Drawable icon = Utils.loadDrawable(activity.getPackageManager(),
+                    Drawable icon = Utils.loadDrawable(getActivity().getPackageManager(),
                             group.getIconPkg(), group.getIconResId());
                     preference.setIcon(Utils.applyTint(getContext(), icon,
                             android.R.attr.colorControlNormal));
@@ -343,7 +333,7 @@ public final class ReviewPermissionsActivity extends Activity
                         screen.addPreference(preference);
                     } else {
                         if (mNewPermissionsCategory == null) {
-                            mNewPermissionsCategory = new PreferenceCategory(activity);
+                            mNewPermissionsCategory = new PreferenceCategory(getActivity());
                             mNewPermissionsCategory.setTitle(R.string.new_permissions_category);
                             mNewPermissionsCategory.setOrder(1);
                             screen.addPreference(mNewPermissionsCategory);
@@ -352,7 +342,7 @@ public final class ReviewPermissionsActivity extends Activity
                     }
                 } else {
                     if (currentPermissionsCategory == null) {
-                        currentPermissionsCategory = new PreferenceCategory(activity);
+                        currentPermissionsCategory = new PreferenceCategory(getActivity());
                         currentPermissionsCategory.setTitle(R.string.current_permissions_category);
                         currentPermissionsCategory.setOrder(2);
                         screen.addPreference(currentPermissionsCategory);
@@ -376,9 +366,6 @@ public final class ReviewPermissionsActivity extends Activity
 
         private void executeCallback(boolean success) {
             Activity activity = getActivity();
-            if (activity == null) {
-                return;
-            }
             if (success) {
                 IntentSender intent = activity.getIntent().getParcelableExtra(Intent.EXTRA_INTENT);
                 if (intent != null) {
-- 
cgit v1.2.3