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

Commit a047fa1f authored by Todd Lee's avatar Todd Lee
Browse files

Ensure that window states (i.e. bounds etc) are committed pre-anim 

Bug: b/378975686
Test: visual validation
Flag: NONE exempt minor fix
Change-Id: I90fdd7a5dc16f115d2a65d62b4bbf306f9e64fa4
parent b0b90cca

packages/SystemUI/animation/lib/src/com/android/systemui/animation/OriginRemoteTransition.java

+4 −4
Original line number Diff line number Diff line
@@ -478,10 +478,6 @@ public class OriginRemoteTransition extends IRemoteTransition.Stub implements 
            }

        }



        // Intentionally use the existing transaction since we want to ensure that the window

        // states are applied along with the other surface preparations prior to the animation.

        // The existing transaction is expected to be committed by the

        // onStart() client callback together with client's custom transformation.

        if (!maxClosingBounds.isEmpty()) {

            logD("Applying closing window bounds: " + maxClosingBounds);

            transactions.setBounds(closingApp, maxClosingBounds);
@@ -556,6 +552,10 @@ public class OriginRemoteTransition extends IRemoteTransition.Stub implements 
            applyWindowAnimationStates(

                    transitionInfo, states, transactions, closingApp, openingApp);



            // ensure that we apply any transactions (e.g. bounds/cropping/corners etc) that might

            // be required prior to starting the actual animation.

            transactions.commit();



            // Start.

            onStart(animationController, transactions, origin, closingApp, openingApp);

        }