diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java index be7670d7c4cd4..9246f4d99cd62 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java @@ -872,10 +872,6 @@ public class PhoneStatusBar extends BaseStatusBar implements DemoMode, mKeyguardUserSwitcher = new KeyguardUserSwitcher(mContext, (ViewStub) mStatusBarWindow.findViewById(R.id.keyguard_user_switcher), mKeyguardStatusBar, mNotificationPanel, mUserSwitcherController); - if (mUserSwitcherController != null) { - mUserSwitcherController.setKeyguardUserSwitcherAvailable( - mKeyguardUserSwitcher.isEnabled()); - } // Set up the quick settings tile panel @@ -2142,9 +2138,6 @@ public class PhoneStatusBar extends BaseStatusBar implements DemoMode, public void setQsExpanded(boolean expanded) { mStatusBarWindowManager.setQsExpanded(expanded); - if (mUserSwitcherController != null) { - mUserSwitcherController.setQsExpanded(expanded); - } } public boolean isGoingToNotificationShade() { diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/policy/KeyguardUserSwitcher.java b/packages/SystemUI/src/com/android/systemui/statusbar/policy/KeyguardUserSwitcher.java index 0392e0daca339..7ee1fc57e7acb 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/policy/KeyguardUserSwitcher.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/policy/KeyguardUserSwitcher.java @@ -227,10 +227,6 @@ public class KeyguardUserSwitcher { } }; - public boolean isEnabled() { - return mUserSwitcherContainer != null; - } - public static class Adapter extends UserSwitcherController.BaseUserAdapter implements View.OnClickListener { diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java b/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java index 5f6c39968abd9..e5b357a8bc3e7 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/policy/UserSwitcherController.java @@ -78,9 +78,6 @@ public class UserSwitcherController { private boolean mSimpleUserSwitcher; private boolean mAddUsersWhenLocked; - private boolean mKeyguardUserSwitcherAvailable; - private boolean mQsExpanded; - public UserSwitcherController(Context context, KeyguardMonitor keyguardMonitor) { mContext = context; mGuestResumeSessionReceiver.register(context); @@ -119,18 +116,16 @@ public class UserSwitcherController { */ @SuppressWarnings("unchecked") private void refreshUsers(int forcePictureLoadForId) { - SparseArray bitmaps = null; - if (allowCachingOfBitmaps()) { - bitmaps = new SparseArray<>(mUsers.size()); - final int N = mUsers.size(); - for (int i = 0; i < N; i++) { - UserRecord r = mUsers.get(i); - if (r == null || r.info == null - || r.info.id == forcePictureLoadForId || r.picture == null) { - continue; - } - bitmaps.put(r.info.id, r.picture); + + SparseArray bitmaps = new SparseArray<>(mUsers.size()); + final int N = mUsers.size(); + for (int i = 0; i < N; i++) { + UserRecord r = mUsers.get(i); + if (r == null || r.info == null + || r.info.id == forcePictureLoadForId || r.picture == null) { + continue; } + bitmaps.put(r.info.id, r.picture); } final boolean addUsersWhenLocked = mAddUsersWhenLocked; @@ -156,14 +151,13 @@ public class UserSwitcherController { true /* isGuest */, isCurrent, false /* isAddUser */, false /* isRestricted */); } else if (info.supportsSwitchTo()) { - Bitmap picture = bitmaps != null ? bitmaps.get(info.id) : null; - if (picture == null && allowCachingOfBitmaps()) { - Bitmap loadedPicture = mUserManager.getUserIcon(info.id); + Bitmap picture = bitmaps.get(info.id); + if (picture == null) { + picture = mUserManager.getUserIcon(info.id); - if (loadedPicture != null) { + if (picture != null) { picture = BitmapHelper.createCircularClip( - loadedPicture, avatarSize, avatarSize); - loadedPicture.recycle(); + picture, avatarSize, avatarSize); } } int index = isCurrent ? 0 : records.size(); @@ -559,32 +553,6 @@ public class UserSwitcherController { } } - /** - * Notify if the keyguard user switcher is available. - */ - public void setKeyguardUserSwitcherAvailable(boolean available) { - boolean oldShouldCacheBitmaps = allowCachingOfBitmaps(); - mKeyguardUserSwitcherAvailable = available; - if (allowCachingOfBitmaps() != oldShouldCacheBitmaps) { - refreshUsers(UserHandle.USER_NULL); - } - } - - /** - * Notify if the quick settings are expanded. - */ - public void setQsExpanded(boolean qsExpanded) { - boolean oldShouldCacheBitmaps = allowCachingOfBitmaps(); - mQsExpanded = qsExpanded; - if (allowCachingOfBitmaps() != oldShouldCacheBitmaps) { - refreshUsers(UserHandle.USER_NULL); - } - } - - private boolean allowCachingOfBitmaps() { - return mQsExpanded || mKeyguardUserSwitcherAvailable; - } - private final class AddUserDialog extends SystemUIDialog implements DialogInterface.OnClickListener {