diff --git a/services/core/java/com/android/server/wm/ActivityTaskManagerService.java b/services/core/java/com/android/server/wm/ActivityTaskManagerService.java index 25827cfe6b03d..19cf14f70e0a3 100644 --- a/services/core/java/com/android/server/wm/ActivityTaskManagerService.java +++ b/services/core/java/com/android/server/wm/ActivityTaskManagerService.java @@ -1908,15 +1908,15 @@ public class ActivityTaskManagerService extends IActivityTaskManager.Stub { if (r.moveFocusableActivityToTop("setFocusedTask")) { mRootWindowContainer.resumeFocusedTasksTopActivities(); - } else if (touchedActivity != null && touchedActivity != r - && touchedActivity.getTask() == r.getTask() - && touchedActivity.getTaskFragment() != r.getTaskFragment()) { - // Set the focused app directly since the focused window is not on the - // top-most TaskFragment of the top-most Task - final DisplayContent displayContent = touchedActivity.getDisplayContent(); - displayContent.setFocusedApp(touchedActivity); - mWindowManager.updateFocusedWindowLocked(UPDATE_FOCUS_NORMAL, - true /* updateInputWindows */); + } else if (touchedActivity != null && touchedActivity.isFocusable()) { + final TaskFragment parent = touchedActivity.getTaskFragment(); + if (parent != null && parent.isEmbedded()) { + // Set the focused app directly if the focused window is currently embedded + final DisplayContent displayContent = touchedActivity.getDisplayContent(); + displayContent.setFocusedApp(touchedActivity); + mWindowManager.updateFocusedWindowLocked(UPDATE_FOCUS_NORMAL, + true /* updateInputWindows */); + } } } diff --git a/services/core/java/com/android/server/wm/DisplayContent.java b/services/core/java/com/android/server/wm/DisplayContent.java index b2e7b8f8b298b..9d95ffa69136c 100644 --- a/services/core/java/com/android/server/wm/DisplayContent.java +++ b/services/core/java/com/android/server/wm/DisplayContent.java @@ -771,10 +771,19 @@ class DisplayContent extends RootDisplayArea implements WindowManagerPolicy.Disp return true; } - if (focusedApp.getTask() == activity.getTask() - && focusedApp.getTaskFragment() != activity.getTaskFragment()) { - // Do not use the activity window of another TaskFragment in the same leaf Task - return false; + // If the candidate activity is currently being embedded in the focused task, the + // activity cannot be focused unless it is on the same TaskFragment as the focusedApp's. + TaskFragment parent = activity.getTaskFragment(); + if (parent != null && parent.isEmbedded()) { + Task hostTask = focusedApp.getTask(); + if (hostTask.isEmbedded()) { + // Use the hosting task if the current task is embedded. + hostTask = hostTask.getParent().asTaskFragment().getTask(); + } + if (activity.isDescendantOf(hostTask) + && activity.getTaskFragment() != focusedApp.getTaskFragment()) { + return false; + } } }