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

Commit 490d7825 authored by Valentin Iftime's avatar Valentin Iftime
Browse files

Only use alive users when handling ACTION_DEVICE_POLICY_MANAGER_STATE_CHANGED 

 The default UserManager.getUsers returns users set for deletion by default.

Flag: EXEMPT bug fix

Test: atest NotificationLockscreenUserManagerTest

Bug: 364940857
Change-Id: I5bcb9201d1973b16bd004c37ba0239ba8dfbc4d9
parent ca506fc7

packages/SystemUI/multivalentTests/src/com/android/systemui/statusbar/NotificationLockscreenUserManagerTest.java

+2 −0
Original line number Diff line number Diff line
@@ -227,6 +227,8 @@ public class NotificationLockscreenUserManagerTest extends SysuiTestCase { 
                Lists.newArrayList(mCurrentUser, mWorkUser, mCommunalUser));

        when(mUserManager.getUsers()).thenReturn(Lists.newArrayList(

                mCurrentUser, mWorkUser, mSecondaryUser, mCommunalUser));

        when(mUserManager.getAliveUsers()).thenReturn(Lists.newArrayList(

                mCurrentUser, mWorkUser, mSecondaryUser, mCommunalUser));

        when(mUserManager.getProfiles(mSecondaryUser.id)).thenReturn(Lists.newArrayList(

                mSecondaryUser));

        when(mUserManager.getProfilesIncludingCommunal(mSecondaryUser.id)).thenReturn(

packages/SystemUI/src/com/android/systemui/statusbar/NotificationLockscreenUserManagerImpl.java

+1 −1
Original line number Diff line number Diff line
@@ -201,7 +201,7 @@ public class NotificationLockscreenUserManagerImpl implements 
                    // the user before calling into DPM

                    sendingUserId = mCurrentUserId;

                    @SuppressLint("MissingPermission")

                    List<UserInfo> users = mUserManager.getUsers();

                    List<UserInfo> users = mUserManager.getAliveUsers();

                    for (int i = users.size() - 1; i >= 0; i--) {

                        changed |= updateDpcSettings(users.get(i).id);

                    }