From baf1183d77f89d8068074dd017823bb611e7f6a8 Mon Sep 17 00:00:00 2001 From: Nate Myren Date: Wed, 23 Mar 2022 13:57:56 -0700 Subject: [PATCH] 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 --- .../server/pm/permission/PermissionManagerServiceImpl.java | 5 +---- 1 file changed, 1 insertion(+), 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 009d155e96ef1..0eea01d3b3bff 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);