Revert "Respect past user choices in permisson review UI" (5ae831d8) · Commits · e / os / android_packages_apps_PackageInstaller

src/com/android/packageinstaller/permission/model/AppPermissionGroup.java

+0 −14

Original line number	Diff line number	Diff line
		@@ -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);
		}

src/com/android/packageinstaller/permission/ui/ReviewPermissionsActivity.java

+5 −18

Original line number	Diff line number	Diff line
		@@ -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) {