Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ffb783b1 authored by Yuri Lin's avatar Yuri Lin Committed by Automerger Merge Worker
Browse files

Merge "Fix system zen rules by using owner package name if caller is system"... 

Merge "Fix system zen rules by using owner package name if caller is system" into tm-dev am: cfc0cfa5 am: 4cfca240

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/19868476



Change-Id: I0764c3c0f340a7e3755020592fa49df08b082ee2
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents a4735bf9 4cfca240

services/core/java/com/android/server/notification/NotificationManagerService.java

+10 −1
Original line number Diff line number Diff line
@@ -4927,7 +4927,16 @@ public class NotificationManagerService extends SystemService { 
            }

            enforcePolicyAccess(Binder.getCallingUid(), "addAutomaticZenRule");













            return mZenModeHelper.addAutomaticZenRule(pkg, automaticZenRule,













            // If the caller is system, take the package name from the rule's owner rather than













            // from the caller's package.













            String rulePkg = pkg;













            if (isCallingUidSystem()) {













                if (automaticZenRule.getOwner() != null) {













                    rulePkg = automaticZenRule.getOwner().getPackageName();













                }













            }

























            return mZenModeHelper.addAutomaticZenRule(rulePkg, automaticZenRule,















                    "addAutomaticZenRule");















        }















































services/tests/uiservicestests/src/com/android/server/notification/NotificationManagerServiceTest.java



+37
−0




























Original line number
Diff line number
Diff line








@@ -7387,6 +7387,43 @@ public class NotificationManagerServiceTest extends UiServiceTestCase {













        mBinderService.addAutomaticZenRule(rule, mContext.getPackageName());















    }





























    @Test













    public void testAddAutomaticZenRule_systemCallTakesPackageFromOwner() throws Exception {













        mService.isSystemUid = true;













        ZenModeHelper mockZenModeHelper = mock(ZenModeHelper.class);













        when(mConditionProviders.isPackageOrComponentAllowed(anyString(), anyInt()))













                .thenReturn(true);













        mService.setZenHelper(mockZenModeHelper);













        ComponentName owner = new ComponentName("android", "ProviderName");













        ZenPolicy zenPolicy = new ZenPolicy.Builder().allowAlarms(true).build();













        boolean isEnabled = true;













        AutomaticZenRule rule = new AutomaticZenRule("test", owner, owner, mock(Uri.class),













                zenPolicy, NotificationManager.INTERRUPTION_FILTER_PRIORITY, isEnabled);













        mBinderService.addAutomaticZenRule(rule, "com.android.settings");



























        // verify that zen mode helper gets passed in a package name of "android"













        verify(mockZenModeHelper).addAutomaticZenRule(eq("android"), eq(rule), anyString());













    }



























    @Test













    public void testAddAutomaticZenRule_nonSystemCallTakesPackageFromArg() throws Exception {













        mService.isSystemUid = false;













        ZenModeHelper mockZenModeHelper = mock(ZenModeHelper.class);













        when(mConditionProviders.isPackageOrComponentAllowed(anyString(), anyInt()))













                .thenReturn(true);













        mService.setZenHelper(mockZenModeHelper);













        ComponentName owner = new ComponentName("android", "ProviderName");













        ZenPolicy zenPolicy = new ZenPolicy.Builder().allowAlarms(true).build();













        boolean isEnabled = true;













        AutomaticZenRule rule = new AutomaticZenRule("test", owner, owner, mock(Uri.class),













                zenPolicy, NotificationManager.INTERRUPTION_FILTER_PRIORITY, isEnabled);













        mBinderService.addAutomaticZenRule(rule, "another.package");



























        // verify that zen mode helper gets passed in the package name from the arg, not the owner













        verify(mockZenModeHelper).addAutomaticZenRule(













                eq("another.package"), eq(rule), anyString());













    }





























    @Test















    public void testAreNotificationsEnabledForPackage() throws Exception {















        mBinderService.areNotificationsEnabledForPackage(mContext.getPackageName(),