Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2df9accd authored by Treehugger Robot's avatar Treehugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Clean up aconfig flag animation_library_delay_leash_cleanup." into main

parents 6820dea8 faf73ec4
Loading
Loading
Loading
Loading
+0 −10
Original line number Diff line number Diff line
@@ -1932,16 +1932,6 @@ flag {
    bug: "430582532"
}

flag {
    name: "animation_library_delay_leash_cleanup"
    namespace: "systemui"
    description: "Clean up the transition leash with a delay to ensure that the finish callback happens first, avoiding flickers"
    bug: "356065603"
    metadata {
        purpose: PURPOSE_BUGFIX
    }
}

flag {
   name: "new_dozing_keyguard_states"
   namespace: "systemui"
+9 −13
Original line number Diff line number Diff line
@@ -61,7 +61,6 @@ import androidx.annotation.UiThread
import com.android.app.animation.Interpolators
import com.android.internal.annotations.VisibleForTesting
import com.android.internal.policy.ScreenDecorationsUtils
import com.android.systemui.Flags.animationLibraryDelayLeashCleanup
import com.android.systemui.Flags.animationLibraryShellMigration
import com.android.systemui.Flags.moveTransitionAnimationLayer
import com.android.systemui.animation.TransitionAnimator.Companion.toTransitionState
@@ -2366,18 +2365,15 @@ constructor(
                                    )
                                }
                            }
                            if (animationLibraryDelayLeashCleanup()) {

                            // This cleanup is not time-sensitive as it is just to avoid leaking
                                // leashes. By delaying it, we make (reasonably) sure that the
                                // finish callback above is executed before the reparent
                                // transaction, which avoids flaky flickers. Unfortunately both are
                                // async, so we need to resort to this hacky solution. Fortunately
                                // none of this will be necessary as soon as b/397180418 is done.
                            // leashes. By delaying it, we make (reasonably) sure that the finish
                            // callback above is executed before the reparent transaction, which
                            // avoids flaky flickers. Unfortunately both are async, so we need to
                            // resort to this hacky solution. Fortunately none of this will be
                            // necessary as soon as b/397180418 is done.
                            Handler(Looper.getMainLooper())
                                .postDelayed(cleanUpTransitionLeash, 500L)
                            } else {
                                cleanUpTransitionLeash()
                            }
                        }

                        if (DEBUG_TRANSITION_ANIMATION) {