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

Commit ad41e139 authored by Winson Chung's avatar Winson Chung
Browse files

Restore existing logic to keep restoring-transient-launch tasks as CHANGE transition 

- This doesn't seem to be an issue anymore (perhaps with ag/30166499),
  things that are already visible should be CHANGE transitions otherwise
  we will run the surface setup code which can reset their position
  and other properties.

  The existing mRestoringTransientHide is still necessary to ensure that
  the right transient hide tasks are included in the transition (and
  only used when the flag is on)

Bug: 30166499
Flag: com.android.wm.shell.enable_recents_bookend_transition
Test: atest  WmTests:TransitionTests  WmTests:WindowOrganizerTests  TaplTestsTaskbarSearch#testLaunchSearchWebSuggestionInSplitscreen  TaplTestsTaskbar#testLaunchShortcutInSplitscreen  TaplTestsSplitscreen#testSaveAppPairMenuItemOrActionExistsOnSplitPair
Change-Id: Id8631b29b36d36c109643ce8680c42b99dbbf642
parent 81abb3be

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

+0 −5
Original line number Diff line number Diff line
@@ -3630,11 +3630,6 @@ class Transition implements BLASTSyncEngine.TransactionReadyListener { 
            }

            final boolean nowVisible = wc.isVisibleRequested();

            if (nowVisible == mVisible) {

                if (mRestoringTransientHide) {

                    // The requested visibility has not changed for transient-hide containers, but

                    // we are restoring them so we should considering them moving to front again

                    return TRANSIT_TO_FRONT;

                }

                return TRANSIT_CHANGE;

            }

            if (mExistenceChanged) {