diff options
| author | 2024-03-08 08:07:01 +0000 | |
|---|---|---|
| committer | 2024-03-08 22:57:46 +0000 | |
| commit | 966c7cb65afa28d9903e30a185e3f9f5dc3cc4b7 (patch) | |
| tree | d118326d3d81550df57429f7b4a6320c77a05485 | |
| parent | 4faa2d19f61e4076d40cb21f4dec6d9013318c6c (diff) | |
Add some more logging for why a change might not be passed to Launcher
- The bug report in b/324435197 indicates that we are starting a recents
transition, but the app going TO_BACK is not being passed to launcher.
This CL adds a bit more logging to see if the task is incorrectly
being considered as a non-leaf task, or something else.
Bug: 324435197
Test: Just adding logging
Change-Id: I9f692a49950cc859f0f7227b79c8e4c6ddbfeeb0
| -rw-r--r-- | core/java/android/window/TransitionInfo.java | 4 | ||||
| -rw-r--r-- | libs/WindowManager/Shell/src/com/android/wm/shell/recents/RecentsTransitionHandler.java | 9 |
2 files changed, 12 insertions, 1 deletions
diff --git a/core/java/android/window/TransitionInfo.java b/core/java/android/window/TransitionInfo.java index 15b9b788bca9..3685bbabf4d3 100644 --- a/core/java/android/window/TransitionInfo.java +++ b/core/java/android/window/TransitionInfo.java @@ -1021,6 +1021,10 @@ public final class TransitionInfo implements Parcelable { sb.append(" component="); sb.append(mActivityComponent.flattenToShortString()); } + if (mTaskInfo != null) { + sb.append(" taskParent="); + sb.append(mTaskInfo.parentTaskId); + } sb.append('}'); return sb.toString(); } diff --git a/libs/WindowManager/Shell/src/com/android/wm/shell/recents/RecentsTransitionHandler.java b/libs/WindowManager/Shell/src/com/android/wm/shell/recents/RecentsTransitionHandler.java index b5ea1b1b43ea..2f68145c08b7 100644 --- a/libs/WindowManager/Shell/src/com/android/wm/shell/recents/RecentsTransitionHandler.java +++ b/libs/WindowManager/Shell/src/com/android/wm/shell/recents/RecentsTransitionHandler.java @@ -531,12 +531,18 @@ public class RecentsTransitionHandler implements Transitions.TransitionHandler { // Put into the "below" layer space. t.setLayer(change.getLeash(), layer); mOpeningTasks.add(new TaskState(change, null /* leash */)); + } else { + ProtoLog.v(ShellProtoLogGroup.WM_SHELL_RECENTS_TRANSITION, + " unhandled root taskId=%d", taskInfo.taskId); } } else if (TransitionUtil.isDividerBar(change)) { final RemoteAnimationTarget target = TransitionUtil.newTarget(change, belowLayers - i, info, t, mLeashMap); // Add this as a app and we will separate them on launcher side by window type. apps.add(target); + } else { + ProtoLog.v(ShellProtoLogGroup.WM_SHELL_RECENTS_TRANSITION, + " unhandled change taskId=%d", taskInfo.taskId); } } t.apply(); @@ -545,7 +551,8 @@ public class RecentsTransitionHandler implements Transitions.TransitionHandler { mRecentTasksController.getSplitBoundsForTaskId(closingSplitTaskId)); try { ProtoLog.v(ShellProtoLogGroup.WM_SHELL_RECENTS_TRANSITION, - "[%d] RecentsController.start: calling onAnimationStart", mInstanceId); + "[%d] RecentsController.start: calling onAnimationStart with %d apps", + mInstanceId, apps.size()); mListener.onAnimationStart(this, apps.toArray(new RemoteAnimationTarget[apps.size()]), wallpapers.toArray(new RemoteAnimationTarget[wallpapers.size()]), |