[DO NOT MERGE] Revert "Fixed the performance of unlocking when dozing"
This reverts commit 87e83cd766.
We are forward fixing in sc-v2-dev
Reason for revert: b/205193749
Change-Id: I9c26643bcfbd37fb36d610291e883112e680e308
This commit is contained in:
@@ -154,11 +154,6 @@ class LightRevealScrim(context: Context?, attrs: AttributeSet?) : View(context,
|
|||||||
*/
|
*/
|
||||||
lateinit var isScrimOpaqueChangedListener: Consumer<Boolean>
|
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
|
* 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.
|
* obscured and 1 means they'll be fully visible.
|
||||||
@@ -170,20 +165,10 @@ class LightRevealScrim(context: Context?, attrs: AttributeSet?) : View(context,
|
|||||||
|
|
||||||
revealEffect.setRevealAmountOnScrim(value, this)
|
revealEffect.setRevealAmountOnScrim(value, this)
|
||||||
updateScrimOpaque()
|
updateScrimOpaque()
|
||||||
maybeTriggerFullyRevealedRunnable()
|
|
||||||
invalidate()
|
invalidate()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun maybeTriggerFullyRevealedRunnable() {
|
|
||||||
if (revealAmount == 1.0f) {
|
|
||||||
fullyRevealedRunnable?.let {
|
|
||||||
it.run()
|
|
||||||
fullyRevealedRunnable = null
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* The [LightRevealEffect] used to manipulate the radial gradient whenever [revealAmount]
|
* The [LightRevealEffect] used to manipulate the radial gradient whenever [revealAmount]
|
||||||
* changes.
|
* changes.
|
||||||
|
|||||||
@@ -37,6 +37,7 @@ import com.android.systemui.dagger.SysUISingleton
|
|||||||
import com.android.systemui.dump.DumpManager
|
import com.android.systemui.dump.DumpManager
|
||||||
import com.android.systemui.plugins.statusbar.StatusBarStateController
|
import com.android.systemui.plugins.statusbar.StatusBarStateController
|
||||||
import com.android.systemui.statusbar.phone.BiometricUnlockController
|
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.DozeParameters
|
||||||
import com.android.systemui.statusbar.phone.PanelExpansionListener
|
import com.android.systemui.statusbar.phone.PanelExpansionListener
|
||||||
import com.android.systemui.statusbar.phone.ScrimController
|
import com.android.systemui.statusbar.phone.ScrimController
|
||||||
@@ -72,10 +73,6 @@ class NotificationShadeDepthController @Inject constructor(
|
|||||||
private const val TAG = "DepthController"
|
private const val TAG = "DepthController"
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Is did we already unblur while dozing?
|
|
||||||
*/
|
|
||||||
private var alreadyUnblurredWhileDozing = false
|
|
||||||
lateinit var root: View
|
lateinit var root: View
|
||||||
private var blurRoot: View? = null
|
private var blurRoot: View? = null
|
||||||
private var keyguardAnimator: Animator? = null
|
private var keyguardAnimator: Animator? = null
|
||||||
@@ -240,11 +237,9 @@ class NotificationShadeDepthController @Inject constructor(
|
|||||||
private val keyguardStateCallback = object : KeyguardStateController.Callback {
|
private val keyguardStateCallback = object : KeyguardStateController.Callback {
|
||||||
override fun onKeyguardFadingAwayChanged() {
|
override fun onKeyguardFadingAwayChanged() {
|
||||||
if (!keyguardStateController.isKeyguardFadingAway ||
|
if (!keyguardStateController.isKeyguardFadingAway ||
|
||||||
!biometricUnlockController.isWakeAndUnlock) {
|
biometricUnlockController.mode != MODE_WAKE_AND_UNLOCK) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
// When wakeAndUnlocking the screen remains dozing, so we have to manually trigger
|
|
||||||
// the unblur earlier
|
|
||||||
|
|
||||||
keyguardAnimator?.cancel()
|
keyguardAnimator?.cancel()
|
||||||
keyguardAnimator = ValueAnimator.ofFloat(1f, 0f).apply {
|
keyguardAnimator = ValueAnimator.ofFloat(1f, 0f).apply {
|
||||||
@@ -266,7 +261,6 @@ class NotificationShadeDepthController @Inject constructor(
|
|||||||
})
|
})
|
||||||
start()
|
start()
|
||||||
}
|
}
|
||||||
alreadyUnblurredWhileDozing = statusBarStateController.dozeAmount != 0.0f
|
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun onKeyguardShowingChanged() {
|
override fun onKeyguardShowingChanged() {
|
||||||
@@ -288,24 +282,10 @@ class NotificationShadeDepthController @Inject constructor(
|
|||||||
if (isDozing) {
|
if (isDozing) {
|
||||||
shadeAnimation.finishIfRunning()
|
shadeAnimation.finishIfRunning()
|
||||||
brightnessMirrorSpring.finishIfRunning()
|
brightnessMirrorSpring.finishIfRunning()
|
||||||
|
|
||||||
// unset this for safety, to be ready for the next wakeup
|
|
||||||
alreadyUnblurredWhileDozing = false
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun onDozeAmountChanged(linear: Float, eased: Float) {
|
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)
|
wakeAndUnlockBlurRadius = blurUtils.blurRadiusOfRatio(eased)
|
||||||
scheduleUpdate()
|
scheduleUpdate()
|
||||||
}
|
}
|
||||||
@@ -463,7 +443,6 @@ class NotificationShadeDepthController @Inject constructor(
|
|||||||
it.println("blursDisabledForAppLaunch: $blursDisabledForAppLaunch")
|
it.println("blursDisabledForAppLaunch: $blursDisabledForAppLaunch")
|
||||||
it.println("qsPanelExpansion: $qsPanelExpansion")
|
it.println("qsPanelExpansion: $qsPanelExpansion")
|
||||||
it.println("transitionToFullShadeProgress: $transitionToFullShadeProgress")
|
it.println("transitionToFullShadeProgress: $transitionToFullShadeProgress")
|
||||||
it.println("alreadyUnblurredWhileDozing: $alreadyUnblurredWhileDozing")
|
|
||||||
it.println("lastAppliedBlur: $lastAppliedBlur")
|
it.println("lastAppliedBlur: $lastAppliedBlur")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -360,7 +360,6 @@ public class StatusBar extends SystemUI implements DemoMode,
|
|||||||
}
|
}
|
||||||
|
|
||||||
private LockscreenShadeTransitionController mLockscreenShadeTransitionController;
|
private LockscreenShadeTransitionController mLockscreenShadeTransitionController;
|
||||||
private boolean mCallingFadingAwayAfterReveal;
|
|
||||||
|
|
||||||
public interface ExpansionChangedListener {
|
public interface ExpansionChangedListener {
|
||||||
void onExpansionChanged(float expansion, boolean expanded);
|
void onExpansionChanged(float expansion, boolean expanded);
|
||||||
@@ -640,7 +639,7 @@ public class StatusBar extends SystemUI implements DemoMode,
|
|||||||
+ "mStatusBarKeyguardViewManager was null");
|
+ "mStatusBarKeyguardViewManager was null");
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
if (mKeyguardStateController.isKeyguardFadingAway() && !mCallingFadingAwayAfterReveal) {
|
if (mKeyguardStateController.isKeyguardFadingAway()) {
|
||||||
mStatusBarKeyguardViewManager.onKeyguardFadedAway();
|
mStatusBarKeyguardViewManager.onKeyguardFadedAway();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -3584,19 +3583,8 @@ public class StatusBar extends SystemUI implements DemoMode,
|
|||||||
public void fadeKeyguardWhilePulsing() {
|
public void fadeKeyguardWhilePulsing() {
|
||||||
mNotificationPanelViewController.fadeOut(0, FADE_KEYGUARD_DURATION_PULSING,
|
mNotificationPanelViewController.fadeOut(0, FADE_KEYGUARD_DURATION_PULSING,
|
||||||
()-> {
|
()-> {
|
||||||
Runnable finishFading = () -> {
|
hideKeyguard();
|
||||||
mCallingFadingAwayAfterReveal = false;
|
mStatusBarKeyguardViewManager.onKeyguardFadedAway();
|
||||||
hideKeyguard();
|
|
||||||
mStatusBarKeyguardViewManager.onKeyguardFadedAway();
|
|
||||||
};
|
|
||||||
if (mLightRevealScrim.getRevealAmount() != 1.0f) {
|
|
||||||
mCallingFadingAwayAfterReveal = true;
|
|
||||||
// we're still revealing the Light reveal, let's only go to keyguard once
|
|
||||||
// Going there introduces lots of jank
|
|
||||||
mLightRevealScrim.setFullyRevealedRunnable(finishFading);
|
|
||||||
} else {
|
|
||||||
finishFading.run();
|
|
||||||
}
|
|
||||||
}).start();
|
}).start();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user