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

Commit 80817dd4 authored by Cosmin Băieș's avatar Cosmin Băieș
Browse files

Consider transient-launch in waiting transitions 

Following from [1], this also considers the waiting transitions when
checking for transient-launch activities.

This would trigger an issue when going from an app with the IME shown to
recents, where the transition to launcher was moved to waiting, meaning
launcher is no longer considered in transient launch. Because of this,
launcher was now passing WindowState#canBeImeTarget, which triggered a
change in the IME layering target from the app to launcher. This allows
InputMonitor#updateInputFocusRequest to request hiding the IME, making
it disappear from app view in recents.

  [1]: Icaa21a6be8a28bfa5369b954e645f72cd47c937c

Bug: 333175068
Test: atest ShowImeWhileEnteringOverviewTest
Change-Id: I4624c5e6f5da0e69c1adaa308555901cb2ec04c8
parent 08df28f4

services/core/java/com/android/server/wm/TransitionController.java

+3 −0
Original line number Diff line number Diff line
@@ -565,6 +565,9 @@ class TransitionController { 
        if (isTransientCollect(ar)) {

            return true;

        }

        for (int i = mWaitingTransitions.size() - 1; i >= 0; --i) {

            if (mWaitingTransitions.get(i).isTransientLaunch(ar)) return true;

        }

        for (int i = mPlayingTransitions.size() - 1; i >= 0; --i) {

            if (mPlayingTransitions.get(i).isTransientLaunch(ar)) return true;

        }

services/tests/wmtests/src/com/android/server/wm/TransitionTests.java

+1 −0
Original line number Diff line number Diff line
@@ -1630,6 +1630,7 @@ public class TransitionTests extends WindowTestsBase { 
        assertTrue(controller.mWaitingTransitions.contains(transition));

        assertTrue(controller.isTransientHide(appTask));

        assertTrue(controller.isTransientVisible(appTask));

        assertTrue(controller.isTransientLaunch(recent));

    }



    @Test