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

Commit f7a7621c authored by Luca Zuccarini's avatar Luca Zuccarini
Browse files

Revert "Set a relative layer instead of reparenting launch animations." 

This reverts commit f50caa76.

Reason for revert: partial revert to CP fix to b/404507270. The end state is the same as currently at ToT, but need to have cherry-pickable CLs for branches that are in CP mode.

Change-Id: I049b5de5065e4d877bfa835eaa3a97d724ada2cc
parent c912f0ab

packages/SystemUI/animation/src/com/android/systemui/animation/RemoteAnimationRunnerCompat.java

+19 −0
Original line number Diff line number Diff line
@@ -79,6 +79,15 @@ public abstract class RemoteAnimationRunnerCompat extends IRemoteAnimationRunner 


    /** Wraps a remote animation runner in a remote-transition. */

    public static RemoteTransitionStub wrap(IRemoteAnimationRunner runner) {

        return wrap(runner, false);

    }



    /**

     * Wraps a remote animation runner in a remote-transition, optionally reparenting the transition

     * leashes to null to explicitly clean them up once the animation is finished.

     */

    public static RemoteTransitionStub wrap(

            IRemoteAnimationRunner runner, boolean reparentLeashesOnFinish) {

        return new RemoteTransitionStub() {

            final ArrayMap<IBinder, Runnable> mFinishRunnables = new ArrayMap<>();
@@ -197,6 +206,16 @@ public abstract class RemoteAnimationRunnerCompat extends IRemoteAnimationRunner 
                    // Release surface references now. This is apparently to free GPU memory

                    // before GC would.

                    info.releaseAllSurfaces();

                    // Make sure that the transition leashes created are not leaked.

                    if (reparentLeashesOnFinish) {

                        for (SurfaceControl leash : leashMap.values()) {

                            try {

                                finishTransaction.reparent(leash, null);

                            } catch (Exception e) {

                                Log.e(TAG, "Failed to reparent leash", e);

                            }

                        }

                    }

                    // Don't release here since launcher might still be using them. Instead

                    // let launcher release them (eg. via RemoteAnimationTargets)

                    leashMap.clear();