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

Unified Diff: content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestInputMethodManagerWrapper.java

Issue 1388283002: Fix OSK flickering issue (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix more tests except for contextualsearchmanagertests Created 5 years, 2 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/test/android/javatests/src/org/chromium/content/browser/test/util/TestInputMethodManagerWrapper.java
diff --git a/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestInputMethodManagerWrapper.java b/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestInputMethodManagerWrapper.java
index 674bf5850754dc08056dec03c32c9915f9457b31..20ad2219232b2a17aad95c20c957f4992391b1fa 100644
--- a/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestInputMethodManagerWrapper.java
+++ b/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestInputMethodManagerWrapper.java
@@ -64,11 +64,14 @@ public class TestInputMethodManagerWrapper extends InputMethodManagerWrapper {
private final ContentViewCore mContentViewCore;
private InputConnection mInputConnection;
- private int mShowSoftInputCounter = 0;
- private int mUpdateSelectionCounter = 0;
+ private int mRestartInputCounter;
+ private int mShowSoftInputCounter;
+ private int mHideSoftInputCounter;
+ private int mUpdateSelectionCounter;
private EditorInfo mEditorInfo;
private final Range mSelection = new Range(0, 0);
private final Range mComposition = new Range(-1, -1);
+ private boolean mIsShowWithoutHideOutstanding;
public TestInputMethodManagerWrapper(ContentViewCore contentViewCore) {
super(null);
@@ -77,12 +80,14 @@ public class TestInputMethodManagerWrapper extends InputMethodManagerWrapper {
@Override
public void restartInput(View view) {
+ mRestartInputCounter++;
mEditorInfo = new EditorInfo();
mInputConnection = mContentViewCore.onCreateInputConnection(mEditorInfo);
}
@Override
public void showSoftInput(View view, int flags, ResultReceiver resultReceiver) {
+ mIsShowWithoutHideOutstanding = true;
mShowSoftInputCounter++;
if (mInputConnection != null) return;
mEditorInfo = new EditorInfo();
@@ -98,6 +103,8 @@ public class TestInputMethodManagerWrapper extends InputMethodManagerWrapper {
@Override
public boolean hideSoftInputFromWindow(IBinder windowToken, int flags,
ResultReceiver resultReceiver) {
+ mIsShowWithoutHideOutstanding = false;
+ mHideSoftInputCounter++;
boolean retVal = mInputConnection == null;
mInputConnection = null;
return retVal;
@@ -111,14 +118,29 @@ public class TestInputMethodManagerWrapper extends InputMethodManagerWrapper {
mComposition.set(candidatesStart, candidatesEnd);
}
+ public int getRestartInputCounter() {
+ return mRestartInputCounter;
+ }
+
public int getShowSoftInputCounter() {
return mShowSoftInputCounter;
}
+ public int getHideSoftInputCounter() {
+ return mHideSoftInputCounter;
+ }
+
public int getUpdateSelectionCounter() {
return mUpdateSelectionCounter;
}
+ public void resetCounters() {
+ mRestartInputCounter = 0;
+ mShowSoftInputCounter = 0;
+ mHideSoftInputCounter = 0;
+ mUpdateSelectionCounter = 0;
+ }
+
public EditorInfo getEditorInfo() {
return mEditorInfo;
}
@@ -130,4 +152,8 @@ public class TestInputMethodManagerWrapper extends InputMethodManagerWrapper {
public Range getComposition() {
return mComposition;
}
+
+ public boolean isShowWithoutHideOutstanding() {
+ return mIsShowWithoutHideOutstanding;
+ }
}

Powered by Google App Engine
This is Rietveld 408576698