Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Initialize Clock Size from NotificationPanelViewController" into tm-qpr-dev am: 5db8d1ea4d

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

Change-Id: I0d5a1405af58808fa0e6e1d53454efdf27414e8d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Hawkwood Glazier
2022-12-05 15:50:32 +00:00
committed by Automerger Merge Worker
parent 5d042793fa 5db8d1ea4d
commit d2ec635950
3 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/SystemUI/src/com/android/systemui/shade/NotificationPanelViewController.java
View File

@@ -915,6 +915,7 @@ public final class NotificationPanelViewController implements Dumpable {
mQsFrameTranslateController = qsFrameTranslateController; mQsFrameTranslateController = qsFrameTranslateController;
updateUserSwitcherFlags(); updateUserSwitcherFlags();
mKeyguardBottomAreaViewModel = keyguardBottomAreaViewModel; mKeyguardBottomAreaViewModel = keyguardBottomAreaViewModel;
mKeyguardBottomAreaInteractor = keyguardBottomAreaInteractor;
onFinishInflate(); onFinishInflate();
keyguardUnlockAnimationController.addKeyguardUnlockAnimationListener( keyguardUnlockAnimationController.addKeyguardUnlockAnimationListener(
new KeyguardUnlockAnimationController.KeyguardUnlockAnimationListener() { new KeyguardUnlockAnimationController.KeyguardUnlockAnimationListener() {
@@ -932,7 +933,6 @@ public final class NotificationPanelViewController implements Dumpable {
unlockAnimationStarted(playingCannedAnimation, isWakeAndUnlock, startDelay); unlockAnimationStarted(playingCannedAnimation, isWakeAndUnlock, startDelay);
} }
}); });
mKeyguardBottomAreaInteractor = keyguardBottomAreaInteractor;
dumpManager.registerDumpable(this); dumpManager.registerDumpable(this);
} }
@@ -1108,6 +1108,7 @@ public final class NotificationPanelViewController implements Dumpable {
mKeyguardStatusViewComponentFactory.build(keyguardStatusView); mKeyguardStatusViewComponentFactory.build(keyguardStatusView);
mKeyguardStatusViewController = statusViewComponent.getKeyguardStatusViewController(); mKeyguardStatusViewController = statusViewComponent.getKeyguardStatusViewController();
mKeyguardStatusViewController.init(); mKeyguardStatusViewController.init();
updateClockAppearance();
if (mKeyguardUserSwitcherController != null) { if (mKeyguardUserSwitcherController != null) {
// Try to close the switcher so that callbacks are triggered if necessary. // Try to close the switcher so that callbacks are triggered if necessary.

6
packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackScrollLayoutController.java
View File

@@ -983,7 +983,7 @@ public class NotificationStackScrollLayoutController {
} }
public boolean isAddOrRemoveAnimationPending() { public boolean isAddOrRemoveAnimationPending() {
return mView.isAddOrRemoveAnimationPending(); return mView != null && mView.isAddOrRemoveAnimationPending();
} }
public int getVisibleNotificationCount() { public int getVisibleNotificationCount() {
@@ -1140,7 +1140,9 @@ public class NotificationStackScrollLayoutController {
} }
public void setAlpha(float alpha) { public void setAlpha(float alpha) {
mView.setAlpha(alpha); if (mView != null) {
mView.setAlpha(alpha);
}
} }
public float calculateAppearFraction(float height) { public float calculateAppearFraction(float height) {

12
packages/SystemUI/tests/src/com/android/systemui/shade/NotificationPanelViewControllerTest.java
View File

@@ -43,6 +43,7 @@ import static org.mockito.Mockito.doAnswer;
import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.inOrder;
import static org.mockito.Mockito.mock; import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.never; import static org.mockito.Mockito.never;
import static org.mockito.Mockito.reset;
import static org.mockito.Mockito.times; import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify; import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when; import static org.mockito.Mockito.when;
@@ -530,6 +531,8 @@ public class NotificationPanelViewControllerTest extends SysuiTestCase {
.setHeadsUpAppearanceController(mock(HeadsUpAppearanceController.class)); .setHeadsUpAppearanceController(mock(HeadsUpAppearanceController.class));
verify(mNotificationStackScrollLayoutController) verify(mNotificationStackScrollLayoutController)
.setOnEmptySpaceClickListener(mEmptySpaceClickListenerCaptor.capture()); .setOnEmptySpaceClickListener(mEmptySpaceClickListenerCaptor.capture());
verify(mKeyguardStatusViewController).displayClock(LARGE, /* animate */ true);
reset(mKeyguardStatusViewController);
} }
@After @After
@@ -609,7 +612,7 @@ public class NotificationPanelViewControllerTest extends SysuiTestCase {
@Test @Test
public void getVerticalSpaceForLockscreenNotifications_useLockIconBottomPadding_returnsSpaceAvailable() { public void getVerticalSpaceForLockscreenNotifications_useLockIconBottomPadding_returnsSpaceAvailable() {
setBottomPadding(/* stackScrollLayoutBottom= */ 100, setBottomPadding(/* stackScrollLayoutBottom= */ 180,
/* lockIconPadding= */ 20, /* lockIconPadding= */ 20,
/* indicationPadding= */ 0, /* indicationPadding= */ 0,
/* ambientPadding= */ 0); /* ambientPadding= */ 0);
@@ -620,7 +623,7 @@ public class NotificationPanelViewControllerTest extends SysuiTestCase {
@Test @Test
public void getVerticalSpaceForLockscreenNotifications_useIndicationBottomPadding_returnsSpaceAvailable() { public void getVerticalSpaceForLockscreenNotifications_useIndicationBottomPadding_returnsSpaceAvailable() {
setBottomPadding(/* stackScrollLayoutBottom= */ 100, setBottomPadding(/* stackScrollLayoutBottom= */ 180,
/* lockIconPadding= */ 0, /* lockIconPadding= */ 0,
/* indicationPadding= */ 30, /* indicationPadding= */ 30,
/* ambientPadding= */ 0); /* ambientPadding= */ 0);
@@ -631,7 +634,7 @@ public class NotificationPanelViewControllerTest extends SysuiTestCase {
@Test @Test
public void getVerticalSpaceForLockscreenNotifications_useAmbientBottomPadding_returnsSpaceAvailable() { public void getVerticalSpaceForLockscreenNotifications_useAmbientBottomPadding_returnsSpaceAvailable() {
setBottomPadding(/* stackScrollLayoutBottom= */ 100, setBottomPadding(/* stackScrollLayoutBottom= */ 180,
/* lockIconPadding= */ 0, /* lockIconPadding= */ 0,
/* indicationPadding= */ 0, /* indicationPadding= */ 0,
/* ambientPadding= */ 40); /* ambientPadding= */ 40);
@@ -954,7 +957,7 @@ public class NotificationPanelViewControllerTest extends SysuiTestCase {
} }
@Test @Test
public void testFinishInflate_userSwitcherDisabled_doNotInflateUserSwitchView() { public void testFinishInflate_userSwitcherDisabled_doNotInflateUserSwitchView_initClock() {
givenViewAttached(); givenViewAttached();
when(mResources.getBoolean( when(mResources.getBoolean(
com.android.internal.R.bool.config_keyguardUserSwitcher)).thenReturn(true); com.android.internal.R.bool.config_keyguardUserSwitcher)).thenReturn(true);
@@ -965,6 +968,7 @@ public class NotificationPanelViewControllerTest extends SysuiTestCase {
mNotificationPanelViewController.onFinishInflate(); mNotificationPanelViewController.onFinishInflate();
verify(mUserSwitcherStubView, never()).inflate(); verify(mUserSwitcherStubView, never()).inflate();
verify(mKeyguardStatusViewController, times(3)).displayClock(LARGE, /* animate */ true);
} }
@Test @Test