Merge "Delete NPVC.mIsLaunchTransitionFinished and associated dead code" into tm-qpr-dev

This commit is contained in:
Justin Weir
2022-11-04 00:17:43 +00:00
committed by Android (Google) Code Review
4 changed files with 2 additions and 43 deletions

View File

@@ -455,7 +455,6 @@ public final class NotificationPanelViewController {
* need to take this into account in our panel height calculation. * need to take this into account in our panel height calculation.
*/ */
private boolean mQsAnimatorExpand; private boolean mQsAnimatorExpand;
private boolean mIsLaunchTransitionFinished;
private ValueAnimator mQsSizeChangeAnimator; private ValueAnimator mQsSizeChangeAnimator;
private boolean mQsScrimEnabled = true; private boolean mQsScrimEnabled = true;
private boolean mQsTouchAboveFalsingThreshold; private boolean mQsTouchAboveFalsingThreshold;
@@ -1753,7 +1752,6 @@ public final class NotificationPanelViewController {
} }
public void resetViews(boolean animate) { public void resetViews(boolean animate) {
mIsLaunchTransitionFinished = false;
mCentralSurfaces.getGutsManager().closeAndSaveGuts(true /* leavebehind */, true /* force */, mCentralSurfaces.getGutsManager().closeAndSaveGuts(true /* leavebehind */, true /* force */,
true /* controls */, -1 /* x */, -1 /* y */, true /* resetMenu */); true /* controls */, -1 /* x */, -1 /* y */, true /* resetMenu */);
if (animate && !isFullyCollapsed()) { if (animate && !isFullyCollapsed()) {
@@ -3779,10 +3777,6 @@ public final class NotificationPanelViewController {
mQs.closeCustomizer(); mQs.closeCustomizer();
} }
public boolean isLaunchTransitionFinished() {
return mIsLaunchTransitionFinished;
}
public void setIsLaunchAnimationRunning(boolean running) { public void setIsLaunchAnimationRunning(boolean running) {
boolean wasRunning = mIsLaunchAnimationRunning; boolean wasRunning = mIsLaunchAnimationRunning;
mIsLaunchAnimationRunning = running; mIsLaunchAnimationRunning = running;

View File

@@ -2569,12 +2569,6 @@ public class CentralSurfacesImpl implements CoreStartable, CentralSurfaces {
// ordering. // ordering.
mMainExecutor.execute(mShadeController::runPostCollapseRunnables); mMainExecutor.execute(mShadeController::runPostCollapseRunnables);
} }
} else if (mNotificationPanelViewController.isLaunchTransitionFinished()) {
// We are not dismissing the shade, but the launch transition is already
// finished,
// so nobody will call readyForKeyguardDone anymore. Post it such that
// keyguardDonePending gets called first.
mMainExecutor.execute(mStatusBarKeyguardViewManager::readyForKeyguardDone);
} }
return deferred; return deferred;
} }

View File

@@ -476,7 +476,6 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
} else if (mKeyguardStateController.isShowing() && !hideBouncerOverDream) { } else if (mKeyguardStateController.isShowing() && !hideBouncerOverDream) {
if (!isWakeAndUnlocking() if (!isWakeAndUnlocking()
&& !(mBiometricUnlockController.getMode() == MODE_DISMISS_BOUNCER) && !(mBiometricUnlockController.getMode() == MODE_DISMISS_BOUNCER)
&& !mNotificationPanelViewController.isLaunchTransitionFinished()
&& !isUnlockCollapsing()) { && !isUnlockCollapsing()) {
if (mBouncer != null) { if (mBouncer != null) {
mBouncer.setExpansion(fraction); mBouncer.setExpansion(fraction);
@@ -845,21 +844,6 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
if (isShowing && isOccluding) { if (isShowing && isOccluding) {
SysUiStatsLog.write(SysUiStatsLog.KEYGUARD_STATE_CHANGED, SysUiStatsLog.write(SysUiStatsLog.KEYGUARD_STATE_CHANGED,
SysUiStatsLog.KEYGUARD_STATE_CHANGED__STATE__OCCLUDED); SysUiStatsLog.KEYGUARD_STATE_CHANGED__STATE__OCCLUDED);
if (mNotificationPanelViewController.isLaunchTransitionFinished()) {
final Runnable endRunnable = new Runnable() {
@Override
public void run() {
mNotificationShadeWindowController.setKeyguardOccluded(isOccluded);
reset(true /* hideBouncerWhenShowing */);
}
};
mCentralSurfaces.fadeKeyguardAfterLaunchTransition(
null /* beforeFading */,
endRunnable,
endRunnable);
return;
}
if (mCentralSurfaces.isLaunchingActivityOverLockscreen()) { if (mCentralSurfaces.isLaunchingActivityOverLockscreen()) {
// When isLaunchingActivityOverLockscreen() is true, we know for sure that the post // When isLaunchingActivityOverLockscreen() is true, we know for sure that the post
// collapse runnables will be run. // collapse runnables will be run.
@@ -931,8 +915,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
long uptimeMillis = SystemClock.uptimeMillis(); long uptimeMillis = SystemClock.uptimeMillis();
long delay = Math.max(0, startTime + HIDE_TIMING_CORRECTION_MS - uptimeMillis); long delay = Math.max(0, startTime + HIDE_TIMING_CORRECTION_MS - uptimeMillis);
if (mNotificationPanelViewController.isLaunchTransitionFinished() if (mKeyguardStateController.isFlingingToDismissKeyguard()) {
|| mKeyguardStateController.isFlingingToDismissKeyguard()) {
final boolean wasFlingingToDismissKeyguard = final boolean wasFlingingToDismissKeyguard =
mKeyguardStateController.isFlingingToDismissKeyguard(); mKeyguardStateController.isFlingingToDismissKeyguard();
mCentralSurfaces.fadeKeyguardAfterLaunchTransition(new Runnable() { mCentralSurfaces.fadeKeyguardAfterLaunchTransition(new Runnable() {
@@ -1309,7 +1292,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
@Override @Override
public boolean shouldDisableWindowAnimationsForUnlock() { public boolean shouldDisableWindowAnimationsForUnlock() {
return mNotificationPanelViewController.isLaunchTransitionFinished(); return false;
} }
@Override @Override

View File

@@ -305,17 +305,6 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase {
verify(mBouncer, never()).setExpansion(anyFloat()); verify(mBouncer, never()).setExpansion(anyFloat());
} }
@Test
public void onPanelExpansionChanged_neverTranslatesBouncerWhenLaunchingApp() {
when(mNotificationPanelView.isLaunchTransitionFinished()).thenReturn(true);
mStatusBarKeyguardViewManager.onPanelExpansionChanged(
expansionEvent(
/* fraction= */ KeyguardBouncer.EXPANSION_VISIBLE,
/* expanded= */ true,
/* tracking= */ false));
verify(mBouncer, never()).setExpansion(anyFloat());
}
@Test @Test
public void onPanelExpansionChanged_neverTranslatesBouncerWhenShadeLocked() { public void onPanelExpansionChanged_neverTranslatesBouncerWhenShadeLocked() {
when(mStatusBarStateController.getState()).thenReturn(StatusBarState.SHADE_LOCKED); when(mStatusBarStateController.getState()).thenReturn(StatusBarState.SHADE_LOCKED);
@@ -361,7 +350,6 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase {
@Test @Test
public void setOccluded_isInLaunchTransition_onKeyguardOccludedChangedCalled() { public void setOccluded_isInLaunchTransition_onKeyguardOccludedChangedCalled() {
when(mNotificationPanelView.isLaunchTransitionFinished()).thenReturn(true);
mStatusBarKeyguardViewManager.show(null); mStatusBarKeyguardViewManager.show(null);
mStatusBarKeyguardViewManager.setOccluded(true /* occluded */, false /* animated */); mStatusBarKeyguardViewManager.setOccluded(true /* occluded */, false /* animated */);