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

Commit 966c7cb6 authored by Winson Chung's avatar Winson Chung
Browse files

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
parent 4faa2d19

core/java/android/window/TransitionInfo.java

+4 −0
Original line number Diff line number Diff line
@@ -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();

        }

libs/WindowManager/Shell/src/com/android/wm/shell/recents/RecentsTransitionHandler.java

+8 −1
Original line number Diff line number Diff line
@@ -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()]),