summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Louis Chang <louischang@google.com> 2025-02-10 18:59:27 -0800
committer Android (Google) Code Review <android-gerrit@google.com> 2025-02-10 18:59:27 -0800
commit997e3c8319cadb70915a8b51926335d1707f4ada (patch)
tree1609b0ffc75b62ec2ec6201ca0c4b2e6307b9a03
parent7f0b1598246017b0ec07bb8d36b16c682b5db383 (diff)
parent6b8dd93fcc22ae1cd170381d7686b5c81059c8d7 (diff)
Merge "Reset DisplayContent while a Task is removed from the hierarchy" into main
-rw-r--r--services/core/java/com/android/server/wm/Task.java12
1 files changed, 1 insertions, 11 deletions
diff --git a/services/core/java/com/android/server/wm/Task.java b/services/core/java/com/android/server/wm/Task.java
index f75e7175b4d2..9bfee8c698bc 100644
--- a/services/core/java/com/android/server/wm/Task.java
+++ b/services/core/java/com/android/server/wm/Task.java
@@ -1122,17 +1122,6 @@ class Task extends TaskFragment {
// already ran fully within super.onParentChanged
updateTaskOrganizerState();
- // TODO(b/168037178): The check for null display content and setting it to null doesn't
- // really make sense here...
-
- // TODO(b/168037178): This is mostly taking care of the case where the stask is removing
- // from the display, so we should probably consolidate it there instead.
-
- if (getParent() == null && mDisplayContent != null) {
- mDisplayContent = null;
- mWmService.mWindowPlacerLocked.requestTraversal();
- }
-
if (oldParent != null) {
final Task oldParentTask = oldParent.asTask();
if (oldParentTask != null) {
@@ -2770,6 +2759,7 @@ class Task extends TaskFragment {
}
super.removeImmediately();
+ mDisplayContent = null;
mRemoving = false;
}