Merge "Revert "Fixed the performance of unlocking when dozing"" into sc-v2-dev

This commit is contained in:
Selim Cinek
2021-11-08 19:51:53 +00:00
committed by Android (Google) Code Review
3 changed files with 5 additions and 54 deletions

View File

@@ -207,11 +207,6 @@ class LightRevealScrim(context: Context?, attrs: AttributeSet?) : View(context,
*/
lateinit var isScrimOpaqueChangedListener: Consumer<Boolean>
/**
* A runnable to call when the scrim has been fully revealed. This is only invoked once
*/
var fullyRevealedRunnable: Runnable? = null
/**
* How much of the underlying views are revealed, in percent. 0 means they will be completely
* obscured and 1 means they'll be fully visible.
@@ -223,20 +218,10 @@ class LightRevealScrim(context: Context?, attrs: AttributeSet?) : View(context,
revealEffect.setRevealAmountOnScrim(value, this)
updateScrimOpaque()
maybeTriggerFullyRevealedRunnable()
invalidate()
}
}
private fun maybeTriggerFullyRevealedRunnable() {
if (revealAmount == 1.0f) {
fullyRevealedRunnable?.let {
it.run()
fullyRevealedRunnable = null
}
}
}
/**
* The [LightRevealEffect] used to manipulate the radial gradient whenever [revealAmount]
* changes.

View File

@@ -37,6 +37,7 @@ import com.android.systemui.dagger.SysUISingleton
import com.android.systemui.dump.DumpManager
import com.android.systemui.plugins.statusbar.StatusBarStateController
import com.android.systemui.statusbar.phone.BiometricUnlockController
import com.android.systemui.statusbar.phone.BiometricUnlockController.MODE_WAKE_AND_UNLOCK
import com.android.systemui.statusbar.phone.DozeParameters
import com.android.systemui.statusbar.phone.ScrimController
import com.android.systemui.statusbar.phone.panelstate.PanelExpansionListener
@@ -73,10 +74,6 @@ class NotificationShadeDepthController @Inject constructor(
private const val TAG = "DepthController"
}
/**
* Did we already unblur while dozing?
*/
private var alreadyUnblurredWhileDozing = false
lateinit var root: View
private var blurRoot: View? = null
private var keyguardAnimator: Animator? = null
@@ -233,11 +230,9 @@ class NotificationShadeDepthController @Inject constructor(
private val keyguardStateCallback = object : KeyguardStateController.Callback {
override fun onKeyguardFadingAwayChanged() {
if (!keyguardStateController.isKeyguardFadingAway ||
!biometricUnlockController.isWakeAndUnlock) {
biometricUnlockController.mode != MODE_WAKE_AND_UNLOCK) {
return
}
// When wakeAndUnlocking the screen remains dozing, so we have to manually trigger
// the unblur earlier
keyguardAnimator?.cancel()
keyguardAnimator = ValueAnimator.ofFloat(1f, 0f).apply {
@@ -259,7 +254,6 @@ class NotificationShadeDepthController @Inject constructor(
})
start()
}
alreadyUnblurredWhileDozing = statusBarStateController.dozeAmount != 0.0f
}
override fun onKeyguardShowingChanged() {
@@ -281,24 +275,10 @@ class NotificationShadeDepthController @Inject constructor(
if (isDozing) {
shadeAnimation.finishIfRunning()
brightnessMirrorSpring.finishIfRunning()
// unset this for safety, to be ready for the next wakeup
alreadyUnblurredWhileDozing = false
}
}
override fun onDozeAmountChanged(linear: Float, eased: Float) {
if (alreadyUnblurredWhileDozing) {
if (linear == 0.0f) {
// We finished waking up, let's reset
alreadyUnblurredWhileDozing = false
} else {
// We've already handled the unbluring from the keyguardAnimator above.
// if we would continue, we'd play another unzoom / blur animation from the
// dozing changing.
return
}
}
wakeAndUnlockBlurRadius = blurUtils.blurRadiusOfRatio(eased)
scheduleUpdate()
}
@@ -458,7 +438,6 @@ class NotificationShadeDepthController @Inject constructor(
it.println("blursDisabledForAppLaunch: $blursDisabledForAppLaunch")
it.println("qsPanelExpansion: $qsPanelExpansion")
it.println("transitionToFullShadeProgress: $transitionToFullShadeProgress")
it.println("alreadyUnblurredWhileDozing: $alreadyUnblurredWhileDozing")
it.println("lastAppliedBlur: $lastAppliedBlur")
}
}

View File

@@ -335,7 +335,6 @@ public class StatusBar extends SystemUI implements
}
private final LockscreenShadeTransitionController mLockscreenShadeTransitionController;
private boolean mCallingFadingAwayAfterReveal;
private StatusBarCommandQueueCallbacks mCommandQueueCallbacks;
void setWindowState(int state) {
@@ -3097,20 +3096,8 @@ public class StatusBar extends SystemUI implements
public void fadeKeyguardWhilePulsing() {
mNotificationPanelViewController.fadeOut(0, FADE_KEYGUARD_DURATION_PULSING,
()-> {
Runnable finishFading = () -> {
mCallingFadingAwayAfterReveal = false;
hideKeyguard();
mStatusBarKeyguardViewManager.onKeyguardFadedAway();
};
if (mLightRevealScrim.getRevealAmount() != 1.0f) {
mCallingFadingAwayAfterReveal = true;
// We're still revealing the Light reveal, let's only go to keyguard once
// that has finished and nothing moves anymore.
// Going there introduces lots of jank
mLightRevealScrim.setFullyRevealedRunnable(finishFading);
} else {
finishFading.run();
}
hideKeyguard();
mStatusBarKeyguardViewManager.onKeyguardFadedAway();
}).start();
}
@@ -4257,7 +4244,7 @@ public class StatusBar extends SystemUI implements
+ "mStatusBarKeyguardViewManager was null");
return;
}
if (mKeyguardStateController.isKeyguardFadingAway() && !mCallingFadingAwayAfterReveal) {
if (mKeyguardStateController.isKeyguardFadingAway()) {
mStatusBarKeyguardViewManager.onKeyguardFadedAway();
}
}