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

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

Issue 508993002: Added test case for ime and select action bar check in case of cut operation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unwanted call to selectall Created 6 years, 4 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 8cf30bcab30ce8bba083a273cbb9979d874ac312..4fe7866df14eb8e2e7ce9a0e6e5e6ace8e6745a9 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
@@ -158,7 +158,20 @@ public class ImeTest extends ContentShellTestBase {
copy(mImeAdapter);
assertWaitForKeyboardStatus(true);
assertEquals(11, Selection.getSelectionEnd(mContentViewCore.getEditableForTest()));
- assertEquals(11, Selection.getSelectionEnd(mContentViewCore.getEditableForTest()));
+ }
+
+ @SmallTest
+ @Feature({"TextInput"})
+ public void testImeNotDismissedAfterCutSelection() throws Exception {
+ commitText(mConnection, "Sample Text", 1);
+ waitAndVerifyEditableCallback(mConnection.mImeUpdateQueue, 1,
jdduke (slow) 2014/08/28 14:59:08 Nit: This can all fit on a single line (Java style
ankit 2014/08/28 15:31:09 @jdduke Line width is 101 character. While submitt
jdduke (slow) 2014/08/28 15:33:22 Ah, right, I didn't include the indent.
+ "Sample Text", 11, 11, -1, -1);
+ DOMUtils.longPressNode(this, mContentViewCore, "input_text");
+ assertWaitForSelectActionBarStatus(true);
+ assertWaitForKeyboardStatus(true);
+ cut(mImeAdapter);
+ assertWaitForKeyboardStatus(true);
+ assertWaitForSelectActionBarStatus(false);
}
@SmallTest
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698