Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix SystemUI crash from continues launch task with setLaunchRootTask

Browse Source 
In StageCoordinator#addActivityOptions, there will use
KEY_LAUNCH_ROOT_TASK_TOKEN to specify the "parent task" of current
launch, but that isn't align the definition of "root task" after
single split root was introduced. So the method "getOrCreateRootTask"
also seems weird now.
WindowContainer#reparent will throw IllegalArgumentException when no
parent change.

Bug: 286028934
Test: create a split pair, continues drag chrome to the same stage.
Change-Id: Ic165eca258a8ef1f365b4f86f98211453243a465
This commit is contained in:
wilsonshih
2023-06-06 11:28:37 +00:00
parent a9025bc558
commit 99ccc8786e
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
services/core/java/com/android/server/wm/RootWindowContainer.java
View File

@@ -3310,9 +3310,14 @@ class RootWindowContainer extends WindowContainer<DisplayContent>
if (aOptions != null) {
// Resolve the root task the task should be placed in now based on options
// and reparent if needed.
// TODO(b/229927851) For split-screen, setLaunchRootTask is no longer the "root"
// task, consider to rename methods like "parentTask" instead of "rootTask".
final Task targetRootTask =
getOrCreateRootTask(null, aOptions, task, onTop);
if (targetRootTask != null && task.getRootTask() != targetRootTask) {
// When launch with ActivityOptions#getLaunchRootTask, the "root task" just mean the
// parent of current launch, not the "root task" in hierarchy.
if (targetRootTask != null && task.getRootTask() != targetRootTask
&& task.getParent() != targetRootTask) {
final int reparentMode = onTop
? REPARENT_MOVE_ROOT_TASK_TO_FRONT : REPARENT_LEAVE_ROOT_TASK_IN_PLACE;
task.reparent(targetRootTask, onTop, reparentMode, ANIMATE, DEFER_RESUME,