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

Commit 0513d5fa authored by Fabian Kozynski's avatar Fabian Kozynski Committed by Automerger Merge Worker
Browse files

Merge "Clear listeners after animation" into tm-qpr-dev am: 7c08dcb5 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/21084243



Change-Id: If2fc26034b049a2a0c0aea986a488802fd78ca6a
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 68bcac99 7c08dcb5

packages/SystemUI/src/com/android/systemui/shade/LargeScreenShadeHeaderController.kt

+25 −1
Original line number Original line Diff line number Diff line
@@ -16,6 +16,7 @@ 




package com.android.systemui.shade

package com.android.systemui.shade





import android.animation.Animator

import android.annotation.IdRes

import android.annotation.IdRes

import android.app.StatusBarManager

import android.app.StatusBarManager

import android.content.res.Configuration

import android.content.res.Configuration
@@ -175,11 +176,15 @@ class LargeScreenShadeHeaderController @Inject constructor( 
    var shadeExpandedFraction = -1f

    var shadeExpandedFraction = -1f

        set(value) {

        set(value) {

            if (field != value) {

            if (field != value) {

                val oldAlpha = header.alpha

                header.alpha = ShadeInterpolation.getContentAlpha(value)

                header.alpha = ShadeInterpolation.getContentAlpha(value)

                field = value

                field = value

                if ((oldAlpha == 0f && header.alpha > 0f) ||

                        (oldAlpha > 0f && header.alpha == 0f)) {

                    updateVisibility()

                    updateVisibility()

                }

                }

            }

            }

        }





    /**

    /**

     * Expansion fraction of the QQS <-> QS animation.

     * Expansion fraction of the QQS <-> QS animation.
@@ -336,9 +341,28 @@ class LargeScreenShadeHeaderController @Inject constructor( 
                .setUpdateListener {

                .setUpdateListener {

                    updateVisibility()

                    updateVisibility()

                }

                }

                .setListener(endAnimationListener)

                .start()

                .start()

    }

    }





    private val endAnimationListener = object : Animator.AnimatorListener {

        override fun onAnimationCancel(animation: Animator?) {

            clearListeners()

        }



        override fun onAnimationEnd(animation: Animator?) {

            clearListeners()

        }



        override fun onAnimationRepeat(animation: Animator?) {}



        override fun onAnimationStart(animation: Animator?) {}



        private fun clearListeners() {

            header.animate().setListener(null).setUpdateListener(null)

        }

    }



    private fun loadConstraints() {

    private fun loadConstraints() {

        if (header is MotionLayout) {

        if (header is MotionLayout) {

            // Use resources.getXml instead of passing the resource id due to bug b/205018300

            // Use resources.getXml instead of passing the resource id due to bug b/205018300

packages/SystemUI/tests/src/com/android/systemui/shade/LargeScreenShadeHeaderControllerTest.kt

+30 −1
Original line number Original line Diff line number Diff line 
package com.android.systemui.shade

package com.android.systemui.shade





import android.animation.Animator

import android.animation.ValueAnimator

import android.animation.ValueAnimator

import android.app.StatusBarManager

import android.app.StatusBarManager

import android.content.Context

import android.content.Context
@@ -45,8 +46,8 @@ import org.mockito.Mock 
import org.mockito.Mockito.mock

import org.mockito.Mockito.mock

import org.mockito.Mockito.verify

import org.mockito.Mockito.verify

import org.mockito.Mockito.verifyZeroInteractions

import org.mockito.Mockito.verifyZeroInteractions

import org.mockito.Mockito.`when` as whenever

import org.mockito.junit.MockitoJUnit

import org.mockito.junit.MockitoJUnit

import org.mockito.Mockito.`when` as whenever





@SmallTest

@SmallTest

@RunWith(AndroidTestingRunner::class)

@RunWith(AndroidTestingRunner::class)
@@ -292,6 +293,34 @@ class LargeScreenShadeHeaderControllerTest : SysuiTestCase() { 
        assertThat(viewVisibility).isEqualTo(View.INVISIBLE)

        assertThat(viewVisibility).isEqualTo(View.INVISIBLE)

    }

    }





    @Test

    fun animatorListenersClearedAtEnd() {

        val animator = mock(ViewPropertyAnimator::class.java, Answers.RETURNS_SELF)

        whenever(view.animate()).thenReturn(animator)



        mLargeScreenShadeHeaderController.startCustomizingAnimation(show = true, 0L)

        val listenerCaptor = argumentCaptor<Animator.AnimatorListener>()

        verify(animator).setListener(capture(listenerCaptor))



        listenerCaptor.value.onAnimationEnd(mock())

        verify(animator).setListener(null)

        verify(animator).setUpdateListener(null)

    }



    @Test

    fun animatorListenersClearedOnCancel() {

        val animator = mock(ViewPropertyAnimator::class.java, Answers.RETURNS_SELF)

        whenever(view.animate()).thenReturn(animator)



        mLargeScreenShadeHeaderController.startCustomizingAnimation(show = true, 0L)

        val listenerCaptor = argumentCaptor<Animator.AnimatorListener>()

        verify(animator).setListener(capture(listenerCaptor))



        listenerCaptor.value.onAnimationCancel(mock())

        verify(animator).setListener(null)

        verify(animator).setUpdateListener(null)

    }



    @Test

    @Test

    fun demoMode_attachDemoMode() {

    fun demoMode_attachDemoMode() {

        val cb = argumentCaptor<DemoMode>()

        val cb = argumentCaptor<DemoMode>()