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

Commit b49da778 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Fix re-entrant issue in Keyguard / Mixed handler" into udc-dev am: b4b4f204 am: 13ab0ed6 

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



Change-Id: Ic8361003992cc571d4ed71baf249d97270e1baf1
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents f133499f 13ab0ed6

libs/WindowManager/Shell/src/com/android/wm/shell/keyguard/KeyguardTransitionHandler.java

+5 −4
Original line number Diff line number Diff line
@@ -156,6 +156,8 @@ public class KeyguardTransitionHandler implements Transitions.TransitionHandler 
                "start keyguard %s transition, info = %s", description, info);



        try {

            mStartedTransitions.put(transition,

                    new StartedTransition(info, finishTransaction, remoteHandler));

            remoteHandler.startAnimation(transition, info, startTransaction,

                    new IRemoteTransitionFinishedCallback.Stub() {

                        @Override
@@ -164,14 +166,13 @@ public class KeyguardTransitionHandler implements Transitions.TransitionHandler 
                            if (sct != null) {

                                finishTransaction.merge(sct);

                            }

                            mMainExecutor.execute(() -> {

                            // Post our finish callback to let startAnimation finish first.

                            mMainExecutor.executeDelayed(() -> {

                                mStartedTransitions.remove(transition);

                                finishCallback.onTransitionFinished(wct, null);

                            });

                            }, 0);

                        }

                    });

            mStartedTransitions.put(transition,

                    new StartedTransition(info, finishTransaction, remoteHandler));

        } catch (RemoteException e) {

            Log.wtf(TAG, "RemoteException thrown from local IRemoteTransition", e);

            return false;

libs/WindowManager/Shell/src/com/android/wm/shell/transition/DefaultMixedHandler.java

+2 −1
Original line number Diff line number Diff line
@@ -631,11 +631,12 @@ public class DefaultMixedHandler implements Transitions.TransitionHandler, 
                finishCallback.onTransitionFinished(wct, wctCB);

            }

        };

        mixed.mInFlightSubAnimations++;

        if (!mKeyguardHandler.startAnimation(

                mixed.mTransition, info, startTransaction, finishTransaction, finishCB)) {

            mixed.mInFlightSubAnimations--;

            return false;

        }

        mixed.mInFlightSubAnimations++;

        // Sync pip state.

        if (mPipHandler != null) {

            // We don't know when to apply `startTransaction` so use a separate transaction here.