Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(729)

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java

Issue 24195023: Switch to sending IME selection updates early. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Set DEBUG to false Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
index 88785c177e4d7083df2badb0f07f756141072d1b..78ffa8c594be459cf217dcc2a06465f9f11c18f5 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
@@ -2421,11 +2421,10 @@ import java.util.Map;
getContentViewClient().onGeometryChanged(-1, null);
}
- @SuppressWarnings("unused")
@CalledByNative
private void updateImeAdapter(int nativeImeAdapterAndroid, int textInputType,
String text, int selectionStart, int selectionEnd,
- int compositionStart, int compositionEnd, boolean showImeIfNeeded) {
+ int compositionStart, int compositionEnd, boolean showImeIfNeeded, boolean requireAck) {
TraceEvent.begin();
mSelectionEditable = (textInputType != ImeAdapter.getTextInputTypeNone());
@@ -2436,20 +2435,13 @@ import java.util.Map;
if (mInputConnection != null) {
mInputConnection.setEditableText(text, selectionStart, selectionEnd,
- compositionStart, compositionEnd);
+ compositionStart, compositionEnd, requireAck);
}
TraceEvent.end();
}
@SuppressWarnings("unused")
@CalledByNative
- private void processImeBatchStateAck(boolean isBegin) {
- if (mInputConnection == null) return;
- mInputConnection.setIgnoreTextInputStateUpdates(isBegin);
- }
-
- @SuppressWarnings("unused")
- @CalledByNative
private void setTitle(String title) {
getContentViewClient().onUpdateTitle(title);
}
@@ -2949,6 +2941,7 @@ import java.util.Map;
if (mAccessibilityScriptInjectionObserver == null) {
ContentObserver contentObserver = new ContentObserver(new Handler()) {
+ @Override
public void onChange(boolean selfChange, Uri uri) {
setAccessibilityState(mAccessibilityManager.isEnabled());
}

Powered by Google App Engine
This is Rietveld 408576698