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

Commit aef1f023 authored by Gustav Sennton's avatar Gustav Sennton
Browse files

Use alpha instead of hide() to hide Home task in Recents transition 

Bug: 394074421
Bug: 430559197
Flag: com.android.window.flags.enable_desktop_splitscreen_transition_bugfix
Test: RecentsTransitionHandlerTest, ShowImeWhileEnteringOverviewTest

Change-Id: Ib1089ca786087a10c41ab4de9cac80c02fad503e
parent a6188aac

libs/WindowManager/Shell/src/com/android/wm/shell/recents/RecentsTransitionHandler.java

+1 −1
Original line number Diff line number Diff line
@@ -850,7 +850,7 @@ public class RecentsTransitionHandler implements Transitions.TransitionHandler, 
                            // appearing before the animation itself starts.

                            // TODO: b/399160023 remove this when we stop using transition-type

                            //  checks in transition utils.

                            t.hide(target.leash);

                            t.setAlpha(target.leash, 0f);

                        } else {

                            ProtoLog.v(ShellProtoLogGroup.WM_SHELL_RECENTS_TRANSITION,

                                    "  not handling home taskId=%d", taskInfo.taskId);

libs/WindowManager/Shell/tests/unittest/src/com/android/wm/shell/recents/RecentsTransitionHandlerTest.java

+1 −1
Original line number Diff line number Diff line
@@ -283,7 +283,7 @@ public class RecentsTransitionHandlerTest extends ShellTestCase { 
                mock(Transitions.TransitionFinishCallback.class));

        mMainExecutor.flushAll();



        verify(startT).hide(controller.getLeashMapForTesting().get(homeLeash));

        verify(startT).setAlpha(controller.getLeashMapForTesting().get(homeLeash), 0);

    }



    @Test