commit a4b48f4da8035b5a8671df2cf715863612498516
author Tony Huang <tonyychuang@google.com> Wed Dec 21 07:59:56 2022 +0000
committer Tony Huang <tonyychuang@google.com> Wed Dec 21 09:46:53 2022 +0000
tree a760e8dda2fe8c4f0738f16ac07e9fc6d23f9f23
parent 02358a560e158fbe27502c3f3aef7f7e226af5ca

Initialize Task surface as invisible

While app cold launch and combine to split, sometimes race happend
and cause child task surface still hidden.
Fix this issue by initializing task as invisible so it would not
set surface as hide when first prepareSurface.

Fix: 262478804
Test: manual
Test: pass existing tests
Change-Id: Iaa93fee162fc8bc4b60aac9ff8faea6943cd01d5

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

diff --git a/services/core/java/com/android/server/wm/Task.java b/services/core/java/com/android/server/wm/Task.java
index ea6f244..7d59947 100644
--- a/services/core/java/com/android/server/wm/Task.java
+++ b/services/core/java/com/android/server/wm/Task.java
@@ -609,7 +609,7 @@
      */
     ActivityRecord mChildPipActivity;
 
-    boolean mLastSurfaceShowing = true;
+    boolean mLastSurfaceShowing;
 
     /**
      * Tracks if a back gesture is in progress.
@@ -4170,13 +4170,7 @@
 
     @Override
     boolean showSurfaceOnCreation() {
-        if (mCreatedByOrganizer) {
-            // Tasks created by the organizer are default visible because they can synchronously
-            // update the leash before new children are added to the task.
-            return true;
-        }
-        // Organized tasks handle their own surface visibility
-        return !canBeOrganized();
+        return false;
     }
 
     @Override