Merge "Initialize Clock Size from NotificationPanelViewController" into tm-qpr-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
5db8d1ea4d
@@ -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.
|
||||||
|
|||||||
@@ -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) {
|
||||||
|
|||||||
@@ -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
|
||||||
|
|||||||
Reference in New Issue
Block a user