Skip IME initialization when received the token that already registered

When IME window switches from current display to another display and back quickly,
system will re-bind InputMethodService to re-attach window token for each moving,
the IME service lifecycle will be onBind -> onUnBind -> onDestroy -> onBind.

Ideally, system will deliver window token when onServiceConnected and system should
only receive one onServiceConnected when the last service bound.

But due to user switch display very quick, before unbind service callbacked from client,
the next bind service request comes, caused the connection exists in system side and
client will then callback 2 onServiceConnected to system.

Since CL [1] introduced InputMethodPrivilegedOperationsRegistry to deal with token
registraction singleton-ness, the exception will be thrown for above case due to
the duplicate window token passed to client.

Add InputMethodPrivilegedOperationsRegistry#isRegistered to check if the token
already registered, if so, skip this duplicate initialization request.

[1]: I4a61470f06ffac5f7a512536f8431489db0108f4

Fix: 133624278
Test: manual as below steps:
    1) Settings > Developer options, enable "Simulated display" & "force desktop mode"
    2) Reboot device.
    3) Launch app from simulated display.
    4) Launch app from primary display (i.e. contacts), focus EditText to show IME.
    5) Tapping primary & simulated display repeatly & quickly.
    6) See if any exception log from IME application. (i.e. Gboard)

Change-Id: Ie6bfbae735724fe744590e715124d2737d2b665d
This commit is contained in:
lumark
2019-05-29 17:11:57 +08:00
parent acd959e53f
commit e0af39403e
2 changed files with 22 additions and 1 deletions

View File

@@ -472,8 +472,12 @@ public class InputMethodService extends AbstractInputMethodService {
*/
@MainThread
@Override
public final void initializeInternal(IBinder token, int displayId,
public final void initializeInternal(@NonNull IBinder token, int displayId,
IInputMethodPrivilegedOperations privilegedOperations) {
if (InputMethodPrivilegedOperationsRegistry.isRegistered(token)) {
Log.w(TAG, "The token has already registered, ignore this initialization.");
return;
}
mPrivOps.set(privilegedOperations);
InputMethodPrivilegedOperationsRegistry.put(token, mPrivOps);
updateInputMethodDisplay(displayId);

View File

@@ -132,4 +132,21 @@ public final class InputMethodPrivilegedOperationsRegistry {
}
}
}
/**
* Check the given IME token registration status.
*
* @param token IME token
* @return {@code true} when the IME token has already registered
* {@link InputMethodPrivilegedOperations}, {@code false} otherwise.
*/
@AnyThread
public static boolean isRegistered(IBinder token) {
synchronized (sLock) {
if (sRegistry == null) {
return false;
}
return sRegistry.containsKey(token);
}
}
}