Chromium Code Reviews| Index: content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java |
| diff --git a/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java b/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java |
| index 3a422176f6bc7d7761812ef566fc378d2f5fecc3..465aee593956a3f362dbe54cc106eefcc8b24a84 100644 |
| --- a/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java |
| +++ b/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java |
| @@ -140,9 +140,6 @@ public class ImeTest extends ContentShellTestBase { |
| assertTextsAroundCursor("he", "", "llo"); |
| } |
|
yabinh
2016/07/06 11:53:55
I checked, and found it works for ReplicaInputConn
|
| - // When newCursorPosition != 1, setComposingText doesn't work for ReplicaInputConnection |
| - // because there is a bug in BaseInputConnection. |
| - @CommandLineFlags.Add("enable-features=ImeThread") |
| @SmallTest |
| @Feature({"TextInput", "Main"}) |
| public void testSetComposingTextForDifferentnewCursorPositions() throws Throwable { |
| @@ -191,6 +188,24 @@ public class ImeTest extends ContentShellTestBase { |
| @SmallTest |
| @Feature({"TextInput", "Main"}) |
| + public void testSetComposingTextWithEmptyText() throws Throwable { |
| + commitText("hello", 1); |
| + waitAndVerifyUpdateSelection(0, 5, 5, -1, -1); |
| + |
| + setComposingText("AB", 1); |
| + waitAndVerifyUpdateSelection(1, 7, 7, 5, 7); |
| + |
| + // With previous composition. |
| + setComposingText("", -3); |
| + waitAndVerifyUpdateSelection(2, 2, 2, -1, -1); |
| + |
| + // Without previous composition. |
| + setComposingText("", 3); |
| + waitAndVerifyUpdateSelection(3, 4, 4, -1, -1); |
| + } |
| + |
| + @SmallTest |
| + @Feature({"TextInput", "Main"}) |
| public void testCommitWhileComposingText() throws Throwable { |
| setComposingText("h", 1); |
| waitAndVerifyUpdateSelection(0, 1, 1, 0, 1); |