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

Commit 736c6a50 authored by Cairn Overturf's avatar Cairn Overturf
Browse files

Use PipSurfaceTransactionHelper instead of Transaction API directly 

Bug: b/367464660
Flag: EXEMPT refactor
Test: Open and close pip window
Change-Id: If5ad3e0c6aeea56c5b73b032c9e272111938f5d6
parent 6cb9e197

libs/WindowManager/Shell/src/com/android/wm/shell/pip/PipTaskOrganizer.java

+1 −1
Original line number Diff line number Diff line
@@ -1884,7 +1884,7 @@ public class PipTaskOrganizer implements ShellTaskOrganizer.TaskListener, 
            return;

        }

        final SurfaceControl.Transaction tx = mSurfaceControlTransactionFactory.getTransaction();

        tx.setShadowRadius(mLeash, 0f);

        mSurfaceTransactionHelper.shadow(tx, mLeash, false /* applyShadowRadius */);

        tx.apply();

    }

libs/WindowManager/Shell/src/com/android/wm/shell/pip/tv/TvPipTransition.java

+4 −2
Original line number Diff line number Diff line
@@ -343,7 +343,8 @@ public class TvPipTransition extends PipTransitionController { 


        final SurfaceControl.Transaction transaction = mTransactionFactory.getTransaction();

        for (SurfaceControl leash : closeLeashes) {

            startTransaction.setShadowRadius(leash, 0f);

            mSurfaceTransactionHelper.shadow(startTransaction, leash,

                    false /* applyShadowRadius */);

        }



        ValueAnimator closeFadeOutAnimator = createAnimator();
@@ -361,7 +362,8 @@ public class TvPipTransition extends PipTransitionController { 
                ProtoLog.v(ShellProtoLogGroup.WM_SHELL_PICTURE_IN_PICTURE,

                        "%s: close animation: start", TAG);

                for (SurfaceControl leash : closeLeashes) {

                    startTransaction.setShadowRadius(leash, 0f);

                    mSurfaceTransactionHelper.shadow(startTransaction, leash,

                            false /* applyShadowRadius */);

                }

                startTransaction.apply();