Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix NPE when IMM#invalidateInput() is called too early" into tm-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2022-02-19 00:26:47 +00:00
committed by Android (Google) Code Review
parent 286aed555d a4a6960b62
commit 92d4c63228
2 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
core/java/android/view/inputmethod/InputMethodManager.java
View File

@@ -2031,14 +2031,20 @@ public final class InputMethodManager {
* @param inputConnection the connection to be invalidated.
* @param textSnapshot {@link TextSnapshot} to be used to update {@link EditorInfo}.
* @param sessionId the session ID to be sent.
* @return {@code true} if the operation is done. {@code false} if the caller needs to fall back
* to {@link InputMethodManager#restartInput(View)}.
* @hide
*/
public void doInvalidateInput(@NonNull RemoteInputConnectionImpl inputConnection,
public boolean doInvalidateInput(@NonNull RemoteInputConnectionImpl inputConnection,
@NonNull TextSnapshot textSnapshot, int sessionId) {
synchronized (mH) {
if (mServedInputConnection != inputConnection || mCurrentTextBoxAttribute == null) {
// OK to ignore because the calling InputConnection is already abandoned.
return;
return true;
}
if (mCurrentInputMethodSession == null) {
// IME is not yet bound to the client. Need to fall back to the restartInput().
return false;
}
final EditorInfo editorInfo = mCurrentTextBoxAttribute.createCopyInternal();
editorInfo.initialSelStart = mCursorSelStart = textSnapshot.getSelectionStart();
@@ -2051,6 +2057,7 @@ public final class InputMethodManager {
sessionId);
forAccessibilitySessions(wrapper -> wrapper.invalidateInput(editorInfo,
mServedInputConnection, sessionId));
return true;
}
}

5
core/java/com/android/internal/inputmethod/RemoteInputConnectionImpl.java
View File

@@ -282,9 +282,8 @@ public final class RemoteInputConnectionImpl extends IInputContext.Stub {
if (!alwaysTrueEndBatchEditDetected) {
final TextSnapshot textSnapshot = ic.takeSnapshot();
if (textSnapshot != null) {
mParentInputMethodManager.doInvalidateInput(this, textSnapshot,
nextSessionId);
if (textSnapshot != null && mParentInputMethodManager.doInvalidateInput(
this, textSnapshot, nextSessionId)) {
return;
}
}