summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Nate Myren <ntmyren@google.com> 2022-03-23 13:57:56 -0700
committer Nate Myren <ntmyren@google.com> 2022-03-23 13:57:56 -0700
commitbaf1183d77f89d8068074dd017823bb611e7f6a8 (patch)
treeb732b09597f7402215a85eaea67b2038fedab91b
parentfe6bf4186a58cee524d17dca9c1585957f363843 (diff)
Remove temp handling of POST_NOTIFICATIONS
I don't think this did anything anymore. It should have been removed a while back. Test: atest NotificationPermissionTest Fixes: 205888750 Change-Id: Ie5f0f7b34f5a036fc8f5e54bc63ddd38360c45e7
-rw-r--r--services/core/java/com/android/server/pm/permission/PermissionManagerServiceImpl.java5
1 files changed, 1 insertions, 4 deletions
diff --git a/services/core/java/com/android/server/pm/permission/PermissionManagerServiceImpl.java b/services/core/java/com/android/server/pm/permission/PermissionManagerServiceImpl.java
index 009d155e96ef..0eea01d3b3bf 100644
--- a/services/core/java/com/android/server/pm/permission/PermissionManagerServiceImpl.java
+++ b/services/core/java/com/android/server/pm/permission/PermissionManagerServiceImpl.java
@@ -2636,12 +2636,9 @@ public class PermissionManagerServiceImpl implements PermissionManagerServiceInt
// Cache newImplicitPermissions before modifing permissionsState as for the
// shared uids the original and new state are the same object
- // TODO(205888750): remove the line for LEGACY_REVIEW once propagated through
- // droidfood
if (!origState.hasPermissionState(permName)
&& (pkg.getImplicitPermissions().contains(permName)
- || (permName.equals(Manifest.permission.ACTIVITY_RECOGNITION)))
- || NOTIFICATION_PERMISSIONS.contains(permName)) {
+ || (permName.equals(Manifest.permission.ACTIVITY_RECOGNITION)))) {
if (pkg.getImplicitPermissions().contains(permName)) {
// If permName is an implicit permission, try to auto-grant
newImplicitPermissions.add(permName);