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

Commit 8727dfe6 authored by Automerger Merge Worker's avatar Automerger Merge Worker
Browse files

Merge "Merge "Avoid callback leak in ComplicationHostViewController" into... 

Merge "Merge "Avoid callback leak in ComplicationHostViewController" into tm-qpr-dev am: 0348c776 am: db5c344b" into udc-dev am: b347fede

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



Change-Id: Ib96ec9a3556677800c632c8498c51d423270ba82
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents c3e46191 b347fede

packages/SystemUI/src/com/android/systemui/dreams/complication/ComplicationHostViewController.java

+2 −12
Original line number Diff line number Diff line
@@ -61,9 +61,6 @@ public class ComplicationHostViewController extends ViewController<ConstraintLay 
    @VisibleForTesting

    boolean mIsAnimationEnabled;



    // Whether dream entry animations are finished.

    private boolean mEntryAnimationsFinished = false;



    @Inject

    protected ComplicationHostViewController(

            @Named(SCOPED_COMPLICATIONS_LAYOUT) ConstraintLayout view,
@@ -78,14 +75,6 @@ public class ComplicationHostViewController extends ViewController<ConstraintLay 
        mComplicationCollectionViewModel = viewModel;

        mDreamOverlayStateController = dreamOverlayStateController;



        mDreamOverlayStateController.addCallback(new DreamOverlayStateController.Callback() {

            @Override

            public void onStateChanged() {

                mEntryAnimationsFinished =

                        mDreamOverlayStateController.areEntryAnimationsFinished();

            }

        });



        // Whether animations are enabled.

        mIsAnimationEnabled = secureSettings.getFloatForUser(

                Settings.Global.ANIMATOR_DURATION_SCALE, 1.0f, UserHandle.USER_CURRENT) != 0.0f;
@@ -159,7 +148,8 @@ public class ComplicationHostViewController extends ViewController<ConstraintLay 


                    // Complications to be added before dream entry animations are finished are set

                    // to invisible and are animated in.

                    if (!mEntryAnimationsFinished && mIsAnimationEnabled) {

                    if (!mDreamOverlayStateController.areEntryAnimationsFinished()

                            && mIsAnimationEnabled) {

                        view.setVisibility(View.INVISIBLE);

                    }

                    mComplications.put(id, viewHolder);

packages/SystemUI/tests/src/com/android/systemui/dreams/complication/ComplicationHostViewControllerTest.java

+0 −10
Original line number Diff line number Diff line
@@ -92,9 +92,6 @@ public class ComplicationHostViewControllerTest extends SysuiTestCase { 
    @Captor

    private ArgumentCaptor<Observer<Collection<ComplicationViewModel>>> mObserverCaptor;



    @Captor

    private ArgumentCaptor<DreamOverlayStateController.Callback> mCallbackCaptor;



    @Complication.Category

    static final int COMPLICATION_CATEGORY = Complication.CATEGORY_SYSTEM;
@@ -189,8 +186,6 @@ public class ComplicationHostViewControllerTest extends SysuiTestCase { 


        // Dream entry animations finished.

        when(mDreamOverlayStateController.areEntryAnimationsFinished()).thenReturn(true);

        final DreamOverlayStateController.Callback stateCallback = captureOverlayStateCallback();

        stateCallback.onStateChanged();



        // Add a complication after entry animations are finished.

        final HashSet<ComplicationViewModel> complications = new HashSet<>(
@@ -223,9 +218,4 @@ public class ComplicationHostViewControllerTest extends SysuiTestCase { 
                mObserverCaptor.capture());

        return mObserverCaptor.getValue();

    }



    private DreamOverlayStateController.Callback captureOverlayStateCallback() {

        verify(mDreamOverlayStateController).addCallback(mCallbackCaptor.capture());

        return mCallbackCaptor.getValue();

    }

}