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

Unified Diff: content/public/android/javatests/src/org/chromium/content/browser/input/ReplicaInputConnectionTest.java

Issue 1278593004: Introduce ThreadedInputConnection behind a switch (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: handles render crash and navigation Created 4 years, 11 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/javatests/src/org/chromium/content/browser/input/ReplicaInputConnectionTest.java
diff --git a/content/public/android/javatests/src/org/chromium/content/browser/input/AdapterInputConnectionTest.java b/content/public/android/javatests/src/org/chromium/content/browser/input/ReplicaInputConnectionTest.java
similarity index 76%
rename from content/public/android/javatests/src/org/chromium/content/browser/input/AdapterInputConnectionTest.java
rename to content/public/android/javatests/src/org/chromium/content/browser/input/ReplicaInputConnectionTest.java
index 8ea6e17cd0f138d7612626cf179d67f3340960bf..5b0354a0e1c1c3f1feb72d83cd5a941be1801c21 100644
--- a/content/public/android/javatests/src/org/chromium/content/browser/input/AdapterInputConnectionTest.java
+++ b/content/public/android/javatests/src/org/chromium/content/browser/input/ReplicaInputConnectionTest.java
@@ -5,25 +5,30 @@
package org.chromium.content.browser.input;
import android.content.Context;
+import android.os.Handler;
import android.os.IBinder;
+import android.os.Looper;
import android.os.ResultReceiver;
import android.test.suitebuilder.annotation.MediumTest;
import android.test.suitebuilder.annotation.SmallTest;
+import android.text.Editable;
import android.view.View;
import android.view.inputmethod.EditorInfo;
import org.chromium.base.test.util.Feature;
-import org.chromium.content.browser.input.AdapterInputConnection.ImeState;
+import org.chromium.content.browser.input.ChromiumBaseInputConnection.ThreadManager;
+import org.chromium.content.browser.input.ReplicaInputConnection.ImeState;
import org.chromium.content_shell_apk.ContentShellTestBase;
+import org.chromium.ui.base.ime.TextInputType;
import java.util.ArrayList;
/**
* Tests AdapterInputConnection class and its callbacks to ImeAdapter.
*/
-public class AdapterInputConnectionTest extends ContentShellTestBase {
+public class ReplicaInputConnectionTest extends ContentShellTestBase {
- private AdapterInputConnection mConnection;
+ private ReplicaInputConnection mConnection;
private TestInputMethodManagerWrapper mWrapper;
private TestImeAdapter mImeAdapter;
@@ -36,8 +41,12 @@ public class AdapterInputConnectionTest extends ContentShellTestBase {
TestImeAdapterDelegate imeAdapterDelegate =
new TestImeAdapterDelegate(getContentViewCore().getContainerView());
mImeAdapter = new TestImeAdapter(mWrapper, imeAdapterDelegate);
- mConnection = new AdapterInputConnection(
- getContentViewCore().getContainerView(), mImeAdapter, 0, 0, new EditorInfo());
+ Editable editable = Editable.Factory.getInstance().newEditable("");
+ ThreadManager threadManager = new ChromiumBaseInputConnection.ThreadManager(
+ new Handler(Looper.getMainLooper()));
+
+ mConnection = new ReplicaInputConnection(getContentViewCore().getContainerView(),
+ mImeAdapter, threadManager, editable, TextInputType.TEXT, 0, new EditorInfo());
}
@SmallTest
@@ -47,35 +56,43 @@ public class AdapterInputConnectionTest extends ContentShellTestBase {
final String smiley = "\uD83D\uDE0A"; // multi character codepoint
// No need to adjust length.
- assertFalse(AdapterInputConnection.isIndexBetweenUtf16SurrogatePair("a", 0));
- assertFalse(AdapterInputConnection.isIndexBetweenUtf16SurrogatePair(ga, 0));
- assertFalse(AdapterInputConnection.isIndexBetweenUtf16SurrogatePair("aa", 1));
- assertFalse(AdapterInputConnection.isIndexBetweenUtf16SurrogatePair("a" + smiley + "a", 1));
+ assertFalse(ReplicaInputConnection.isIndexBetweenUtf16SurrogatePair("a", 0));
+ assertFalse(ReplicaInputConnection.isIndexBetweenUtf16SurrogatePair(ga, 0));
+ assertFalse(ReplicaInputConnection.isIndexBetweenUtf16SurrogatePair("aa", 1));
+ assertFalse(ReplicaInputConnection.isIndexBetweenUtf16SurrogatePair("a" + smiley + "a", 1));
// Needs to adjust length.
- assertTrue(AdapterInputConnection.isIndexBetweenUtf16SurrogatePair(smiley, 1));
- assertTrue(AdapterInputConnection.isIndexBetweenUtf16SurrogatePair(smiley + "a", 1));
+ assertTrue(ReplicaInputConnection.isIndexBetweenUtf16SurrogatePair(smiley, 1));
+ assertTrue(ReplicaInputConnection.isIndexBetweenUtf16SurrogatePair(smiley + "a", 1));
}
@MediumTest
@Feature({"TextInput", "Main"})
- @RerunWithUpdatedContainerView
public void testSetComposingText() throws Throwable {
+ assertEquals(0, mWrapper.getUpdateSelectionCallCount());
+
mConnection.setComposingText("t", 1);
assertCorrectState("t", 1, 1, 0, 1, mConnection.getImeStateForTesting());
mWrapper.verifyUpdateSelectionCall(0, 1, 1, 0, 1);
+ // BaseInputConnection calls updateSelection() one more time because getEditable() does
+ // not return a correct value here.
+ mWrapper.verifyUpdateSelectionCall(1, 1, 1, 0, 1);
+ assertEquals(2, mWrapper.getUpdateSelectionCallCount());
mConnection.setComposingText("te", 1);
assertCorrectState("te", 2, 2, 0, 2, mConnection.getImeStateForTesting());
- mWrapper.verifyUpdateSelectionCall(1, 2, 2, 0, 2);
+ mWrapper.verifyUpdateSelectionCall(2, 2, 2, 0, 2);
+ mWrapper.verifyUpdateSelectionCall(3, 2, 2, 0, 2);
mConnection.setComposingText("tes", 1);
assertCorrectState("tes", 3, 3, 0, 3, mConnection.getImeStateForTesting());
- mWrapper.verifyUpdateSelectionCall(2, 3, 3, 0, 3);
+ mWrapper.verifyUpdateSelectionCall(4, 3, 3, 0, 3);
+ mWrapper.verifyUpdateSelectionCall(5, 3, 3, 0, 3);
mConnection.setComposingText("test", 1);
assertCorrectState("test", 4, 4, 0, 4, mConnection.getImeStateForTesting());
- mWrapper.verifyUpdateSelectionCall(3, 4, 4, 0, 4);
+ mWrapper.verifyUpdateSelectionCall(6, 4, 4, 0, 4);
+ mWrapper.verifyUpdateSelectionCall(7, 4, 4, 0, 4);
}
@MediumTest

Powered by Google App Engine
This is Rietveld 408576698