Revert "[Unfold animation] Do not show dark vignette when folding"
Revert submission 21009614-fold-no-vignette Reason for revert: Duplicate flag ids. Please change "2600" to something else. Reverted changes: /q/submissionid:21009614-fold-no-vignette Change-Id: I16e71542ed3179bc37feefbb5fc4d7ac42cbfdd7
This commit is contained in:
committed by
Android (Google) Code Review
parent
22105c51ac
commit
aa7ac3c9b2
@@ -530,13 +530,6 @@ object Flags {
|
||||
val OUTPUT_SWITCHER_SHOW_API_ENABLED =
|
||||
unreleasedFlag(2503, "output_switcher_show_api_enabled", teamfood = true)
|
||||
|
||||
// 2600 - unfold transitions
|
||||
// TODO(b/265764985): Tracking Bug
|
||||
@Keep
|
||||
@JvmField
|
||||
val ENABLE_DARK_VIGNETTE_WHEN_FOLDING =
|
||||
unreleasedFlag(2600, "enable_dark_vignette_when_folding")
|
||||
|
||||
// TODO(b259590361): Tracking bug
|
||||
val EXPERIMENTAL_FLAG = unreleasedFlag(2, "exp_flag_release")
|
||||
}
|
||||
|
||||
@@ -36,8 +36,6 @@ import android.view.SurfaceSession
|
||||
import android.view.WindowManager
|
||||
import android.view.WindowlessWindowManager
|
||||
import com.android.systemui.dagger.qualifiers.Main
|
||||
import com.android.systemui.flags.FeatureFlags
|
||||
import com.android.systemui.flags.Flags
|
||||
import com.android.systemui.statusbar.LightRevealEffect
|
||||
import com.android.systemui.statusbar.LightRevealScrim
|
||||
import com.android.systemui.statusbar.LinearLightRevealEffect
|
||||
@@ -59,7 +57,6 @@ class UnfoldLightRevealOverlayAnimation
|
||||
@Inject
|
||||
constructor(
|
||||
private val context: Context,
|
||||
private val featureFlags: FeatureFlags,
|
||||
private val deviceStateManager: DeviceStateManager,
|
||||
private val contentResolver: ContentResolver,
|
||||
private val displayManager: DisplayManager,
|
||||
@@ -84,7 +81,6 @@ constructor(
|
||||
private var scrimView: LightRevealScrim? = null
|
||||
private var isFolded: Boolean = false
|
||||
private var isUnfoldHandled: Boolean = true
|
||||
private var overlayAddReason: AddOverlayReason? = null
|
||||
|
||||
private var currentRotation: Int = context.display!!.rotation
|
||||
|
||||
@@ -162,8 +158,6 @@ constructor(
|
||||
ensureInBackground()
|
||||
ensureOverlayRemoved()
|
||||
|
||||
overlayAddReason = reason
|
||||
|
||||
val newRoot = SurfaceControlViewHost(context, context.display!!, wwm)
|
||||
val params = getLayoutParams()
|
||||
val newView =
|
||||
@@ -176,7 +170,11 @@ constructor(
|
||||
.apply {
|
||||
revealEffect = createLightRevealEffect()
|
||||
isScrimOpaqueChangedListener = Consumer {}
|
||||
revealAmount = calculateRevealAmount()
|
||||
revealAmount =
|
||||
when (reason) {
|
||||
FOLD -> TRANSPARENT
|
||||
UNFOLD -> BLACK
|
||||
}
|
||||
}
|
||||
|
||||
newRoot.setView(newView, params)
|
||||
@@ -209,31 +207,6 @@ constructor(
|
||||
root = newRoot
|
||||
}
|
||||
|
||||
private fun calculateRevealAmount(animationProgress: Float? = null): Float {
|
||||
val overlayAddReason = overlayAddReason ?: UNFOLD
|
||||
|
||||
if (animationProgress == null) {
|
||||
// Animation progress is unknown, calculate the initial value based on the overlay
|
||||
// add reason
|
||||
return when (overlayAddReason) {
|
||||
FOLD -> TRANSPARENT
|
||||
UNFOLD -> BLACK
|
||||
}
|
||||
}
|
||||
|
||||
val showVignetteWhenFolding =
|
||||
featureFlags.isEnabled(Flags.ENABLE_DARK_VIGNETTE_WHEN_FOLDING)
|
||||
|
||||
return if (!showVignetteWhenFolding && overlayAddReason == FOLD) {
|
||||
// Do not darken the content when SHOW_VIGNETTE_WHEN_FOLDING flag is off
|
||||
// and we are folding the device. We still add the overlay to block touches
|
||||
// while the animation is running but the overlay is transparent.
|
||||
TRANSPARENT
|
||||
} else {
|
||||
animationProgress
|
||||
}
|
||||
}
|
||||
|
||||
private fun getLayoutParams(): WindowManager.LayoutParams {
|
||||
val params: WindowManager.LayoutParams = WindowManager.LayoutParams()
|
||||
|
||||
@@ -286,7 +259,7 @@ constructor(
|
||||
private inner class TransitionListener : TransitionProgressListener {
|
||||
|
||||
override fun onTransitionProgress(progress: Float) {
|
||||
executeInBackground { scrimView?.revealAmount = calculateRevealAmount(progress) }
|
||||
executeInBackground { scrimView?.revealAmount = progress }
|
||||
}
|
||||
|
||||
override fun onTransitionFinished() {
|
||||
|
||||
Reference in New Issue
Block a user