Do not keep focus on IME dialog if its target is invisible
Otherwise the input method target won't have a change to be
updated from IMMS for new focus window.
And if input method target doesn't update, the IME and IME
Switcher Menu will be invisible because they are put under
ImeContainer which is put on invisible app's surface.
Fix: 359711526
Flag: EXEMPT bugfix
Test: atest DisplayContentTests# \
testImeMenuDialogFocusWhenImeLayeringTargetChanges
Change-Id: Ia5ea318b2a92ccc003f3192a198bf041365f70cf
diff --git a/services/core/java/com/android/server/wm/DisplayContent.java b/services/core/java/com/android/server/wm/DisplayContent.java
index 9c8c759..9264ad1 100644
--- a/services/core/java/com/android/server/wm/DisplayContent.java
+++ b/services/core/java/com/android/server/wm/DisplayContent.java
@@ -857,8 +857,8 @@
return false;
}
if (w.mAttrs.type == TYPE_INPUT_METHOD_DIALOG && mImeLayeringTarget != null
- && !mImeLayeringTarget.isRequestedVisible(ime())
- && !mImeLayeringTarget.isVisibleRequested()) {
+ && !(mImeLayeringTarget.isRequestedVisible(ime())
+ && mImeLayeringTarget.isVisibleRequested())) {
return false;
}
diff --git a/services/tests/wmtests/src/com/android/server/wm/DisplayContentTests.java b/services/tests/wmtests/src/com/android/server/wm/DisplayContentTests.java
index ab0c8d4..205d1249 100644
--- a/services/tests/wmtests/src/com/android/server/wm/DisplayContentTests.java
+++ b/services/tests/wmtests/src/com/android/server/wm/DisplayContentTests.java
@@ -2794,15 +2794,17 @@
final WindowState imeAppTarget =
createWindow(null, TYPE_BASE_APPLICATION, mDisplayContent, "imeAppTarget");
mDisplayContent.setImeLayeringTarget(imeAppTarget);
- spyOn(imeAppTarget);
- doReturn(true).when(imeAppTarget).isRequestedVisible(ime());
+ imeAppTarget.setRequestedVisibleTypes(ime());
assertEquals(imeMenuDialog, mDisplayContent.findFocusedWindow());
// Verify imeMenuDialog doesn't be focused window if the next IME target is closing.
final WindowState nextImeAppTarget =
createWindow(null, TYPE_BASE_APPLICATION, mDisplayContent, "nextImeAppTarget");
makeWindowVisibleAndDrawn(nextImeAppTarget);
- nextImeAppTarget.mActivityRecord.commitVisibility(false, false);
+ // Even if the app still requests IME, the ime dialog should not gain focus if the target
+ // app is invisible.
+ nextImeAppTarget.setRequestedVisibleTypes(ime());
+ nextImeAppTarget.mActivityRecord.setVisibility(false);
mDisplayContent.setImeLayeringTarget(nextImeAppTarget);
assertNotEquals(imeMenuDialog, mDisplayContent.findFocusedWindow());
}