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 e63937fd94cd00c64407fe51eb113c41a9ee5cfe..2074fb3b44e21cb5a48676472aa35b0c3ce1265f 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 |
| @@ -174,6 +174,19 @@ public class ImeTest extends ContentShellTestBase { |
| @SmallTest |
| @Feature({"TextInput"}) |
| + public void testSelectActionBarShownOnLongPressingInput() throws Exception { |
| + DOMUtils.focusNode(mContentViewCore, "input_radio"); |
|
jdduke (slow)
2014/08/26 15:01:46
Why not put this test in ContentViewCoreFocusTest?
AKVT
2014/08/26 15:16:40
ContentViewCoreFocusTest does not holding environm
jdduke (slow)
2014/08/26 15:20:39
I don't have a strong opinion, so I'll defer to au
AKVT
2014/08/26 16:38:12
Thanks for the comments.
Without this line DOMUtil
|
| + DOMUtils.longPressNode(this, mContentViewCore, "input_text"); |
| + assertWaitForKeyboardStatus(false); |
| + assertFalse(mContentViewCore.isSelectActionBarShowing()); |
| + commitText(mConnection, "Sample Text", 1); |
| + DOMUtils.longPressNode(this, mContentViewCore, "input_text"); |
| + assertWaitForKeyboardStatus(true); |
| + assertTrue(mContentViewCore.isSelectActionBarShowing()); |
| + } |
| + |
| + @SmallTest |
| + @Feature({"TextInput"}) |
| public void testImeCut() throws Exception { |
| commitText(mConnection, "snarful", 1); |
| waitAndVerifyEditableCallback(mConnection.mImeUpdateQueue, 1, "snarful", 7, 7, -1, -1); |