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

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

Issue 1278593004: Introduce ThreadedInputConnection behind a switch (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: adding missing ImeTestUtils.java Created 4 years, 10 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 89414b4ab13612e3db29577d84c7e4f425d16c15..fd55e5cae1156e7c73515bb0bb51cc2c634ce35f 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
@@ -334,6 +334,7 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Screen
contentViewCore.mIsMobileOptimizedHint = false;
contentViewCore.hidePopupsAndClearSelection();
contentViewCore.resetScrollInProgress();
+ contentViewCore.resetImeAdapter();
}
private void determinedProcessVisibility() {
@@ -934,6 +935,7 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Screen
mWebContentsObserver = null;
setSmartClipDataListener(null);
setZoomControlsDelegate(null);
+ mImeAdapter.reset();
// TODO(igsolla): address TODO in ContentViewClient because ContentViewClient is not
// currently a real Null Object.
//
@@ -3109,6 +3111,11 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Screen
}
}
+ @VisibleForTesting
+ public WebContentsObserver getWebContentsObserverForTest() {
+ return mWebContentsObserver;
Ted C 2016/02/17 19:09:33 Instead of exposing this, can you use the list fro
Changwan Ryu 2016/02/18 06:03:26 Done.
+ }
+
/**
* Offer a long press gesture to the embedding View, primarily for WebView compatibility.
*
@@ -3134,6 +3141,10 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Screen
if (potentiallyActiveFlingCount > 0) updateGestureStateListener(GestureEventType.FLING_END);
}
+ private void resetImeAdapter() {
+ mImeAdapter.reset();
+ }
+
private float getWheelScrollFactorInPixels() {
if (mWheelScrollFactorInPixels == 0) {
TypedValue outValue = new TypedValue();

Powered by Google App Engine
This is Rietveld 408576698