diff --git a/core/java/android/view/inputmethod/InputMethodManager.java b/core/java/android/view/inputmethod/InputMethodManager.java index 1ce804342c437..5a9a212538b68 100644 --- a/core/java/android/view/inputmethod/InputMethodManager.java +++ b/core/java/android/view/inputmethod/InputMethodManager.java @@ -39,6 +39,7 @@ import android.os.RemoteException; import android.os.ResultReceiver; import android.os.ServiceManager; import android.os.Trace; +import android.text.TextUtils; import android.text.style.SuggestionSpan; import android.util.Log; import android.util.Pools.Pool; @@ -553,8 +554,9 @@ public final class InputMethodManager { } @Override - protected void onReportFullscreenMode(boolean enabled) { - mParentInputMethodManager.setFullscreenMode(enabled); + protected void onReportFullscreenMode(boolean enabled, boolean calledInBackground) { + mParentInputMethodManager.onReportFullscreenMode(enabled, calledInBackground, + getInputMethodId()); } @Override @@ -563,6 +565,7 @@ public final class InputMethodManager { + "connection=" + getInputConnection() + " finished=" + isFinished() + " mParentInputMethodManager.mActive=" + mParentInputMethodManager.mActive + + " mInputMethodId=" + getInputMethodId() + "}"; } } @@ -718,8 +721,13 @@ public final class InputMethodManager { } /** @hide */ - public void setFullscreenMode(boolean fullScreen) { - mFullscreenMode = fullScreen; + public void onReportFullscreenMode(boolean fullScreen, boolean calledInBackground, + String inputMethodId) { + synchronized (mH) { + if (!calledInBackground || TextUtils.equals(mCurId, inputMethodId)) { + mFullscreenMode = fullScreen; + } + } } /** @hide */ @@ -746,9 +754,11 @@ public final class InputMethodManager { * your UI, else returns false. */ public boolean isFullscreenMode() { - return mFullscreenMode; + synchronized (mH) { + return mFullscreenMode; + } } - + /** * Return true if the given view is the currently active view for the * input method. @@ -1254,6 +1264,9 @@ public final class InputMethodManager { mCurId = res.id; mNextUserActionNotificationSequenceNumber = res.userActionNotificationSequenceNumber; + if (mServedInputConnectionWrapper != null) { + mServedInputConnectionWrapper.setInputMethodId(mCurId); + } } else { if (res.channel != null && res.channel != mCurChannel) { res.channel.dispose(); diff --git a/core/java/com/android/internal/view/IInputConnectionWrapper.java b/core/java/com/android/internal/view/IInputConnectionWrapper.java index 3a4afad3f3670..59b4b35fa24ef 100644 --- a/core/java/com/android/internal/view/IInputConnectionWrapper.java +++ b/core/java/com/android/internal/view/IInputConnectionWrapper.java @@ -71,6 +71,8 @@ public abstract class IInputConnectionWrapper extends IInputContext.Stub { private Object mLock = new Object(); @GuardedBy("mLock") private boolean mFinished = false; + @GuardedBy("mLock") + private String mInputMethodId; static class SomeArgs { Object arg1; @@ -109,6 +111,18 @@ public abstract class IInputConnectionWrapper extends IInputContext.Stub { } } + public String getInputMethodId() { + synchronized (mLock) { + return mInputMethodId; + } + } + + public void setInputMethodId(final String inputMethodId) { + synchronized (mLock) { + mInputMethodId = inputMethodId; + } + } + abstract protected boolean isActive(); /** @@ -119,9 +133,11 @@ public abstract class IInputConnectionWrapper extends IInputContext.Stub { /** * Called when the input method started or stopped full-screen mode. - * + * @param enabled {@code true} if the input method starts full-screen mode. + * @param calledInBackground {@code true} if this input connection is in a state when incoming + * events are usually ignored. */ - abstract protected void onReportFullscreenMode(boolean enabled); + abstract protected void onReportFullscreenMode(boolean enabled, boolean calledInBackground); public void getTextAfterCursor(int length, int flags, int seq, IInputContextCallback callback) { dispatchMessage(obtainMessageIISC(DO_GET_TEXT_AFTER_CURSOR, length, flags, seq, callback)); @@ -464,13 +480,19 @@ public abstract class IInputConnectionWrapper extends IInputContext.Stub { } case DO_REPORT_FULLSCREEN_MODE: { InputConnection ic = getInputConnection(); - if (ic == null) { + boolean isBackground = false; + if (ic == null || !isActive()) { Log.w(TAG, "reportFullscreenMode on inexistent InputConnection"); - return; + isBackground = true; } final boolean enabled = msg.arg1 == 1; - ic.reportFullscreenMode(enabled); - onReportFullscreenMode(enabled); + if (!isBackground) { + ic.reportFullscreenMode(enabled); + } + // Due to the nature of asynchronous event handling, currently InputMethodService + // has relied on the fact that #reportFullscreenMode() can be handled even when the + // InputConnection is inactive. We have to notify this event to InputMethodManager. + onReportFullscreenMode(enabled, isBackground); return; } case DO_PERFORM_PRIVATE_COMMAND: {