Merge "Weakly hold callback reference from OnPredrawListener" into main

import com.android.systemui.res.R

import com.android.systemui.shared.R as sharedR

import com.google.android.material.math.MathUtils

import java.lang.ref.WeakReference

import kotlin.math.abs

internal fun View.getRect(): Rect = Rect(this.left, this.top, this.right, this.bottom)

                // We enforce the animation parameters on the target view every frame using a

                // predraw listener. This is suboptimal but prevents issues with layout passes

                // overwriting the animation for individual frames.

                val predrawCallback = OnPreDrawListener {

                    assignAnimValues("predraw", anim.animatedFraction, log = false)

                    return@OnPreDrawListener true

                }

                val predrawCallback = PredrawAnimationCallback(anim, ::assignAnimValues)

                this@VisibilityBoundsTransition.addListener(

                    object : TransitionListenerAdapter() {

        }

    }

    private class PredrawAnimationCallback(

        anim: ValueAnimator,

        cb: (String, Float, Int?, Boolean) -> Unit,

    ) : OnPreDrawListener {

        // Holding only WeakReferences prevents leaking the entire transition and all related views

        // in the event that a predraw listener survives the cleanup steps and is still attached to

        // the ViewTreeObserver past the terminiation of the transition.

        private val callback = WeakReference(cb)

        private val animation = WeakReference(anim)

        override fun onPreDraw(): Boolean {

            val cb = callback.get() ?: return true

            val fraction = animation.get()?.animatedFraction ?: 1f

            cb("predraw", fraction, null, false)

            return true

        }

    }

    abstract class ClockFaceTransition(

        config: IntraBlueprintTransition.Config,

        val viewModel: KeyguardClockViewModel,