Merge "Delete NPVC.mIsLaunchTransitionFinished and associated dead code" into tm-qpr-dev
This commit is contained in:
@@ -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;
|
||||||
|
|||||||
@@ -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;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -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
|
||||||
|
|||||||
@@ -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 */);
|
||||||
|
|||||||
Reference in New Issue
Block a user