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

Unified Diff: chrome/test/data/chromeos/virtual_keyboard/inputview/typing_test.js

Issue 259603002: Add touch and basic longpress typing tests for the IME keyboard. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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: chrome/test/data/chromeos/virtual_keyboard/inputview/typing_test.js
diff --git a/chrome/test/data/chromeos/virtual_keyboard/inputview/typing_test.js b/chrome/test/data/chromeos/virtual_keyboard/inputview/typing_test.js
index 5f3d69afd1793c5143ef5a5563252812742ef169..379736180d57abc821fd04886ac130c7f47d7e16 100644
--- a/chrome/test/data/chromeos/virtual_keyboard/inputview/typing_test.js
+++ b/chrome/test/data/chromeos/virtual_keyboard/inputview/typing_test.js
@@ -5,11 +5,32 @@
*/
/**
- * Tests typing in the lowecase keyset.
+ * Tests typing in the lowercase keyset.
*/
function testLowercaseKeyset() {
+ // Mouse events.
bshe 2014/04/24 18:36:25 We probably dont need to test the same case 5 time
rsadam 2014/04/24 21:23:25 Done.
mockMouseType('l');
mockMouseType('p');
mockMouseType('e');
mockMouseType('f');
+ mockTouchType('e');
+
+ // Touch events.
+ mockTouchType('l');
+ mockTouchType('p');
+ mockTouchType('e');
+ mockTouchType('f');
+ mockTouchType('e');
+}
+
+/**
+ * Tests longpress on the keyboard.
+ */
+function testLongpress() {
+ // TODO(rsadam@): Add support for mock longpressing the non-default selection.
+ mockLongpress('e', ['e', 'E'], 'e');
+ mockLongpress('a', ['a', 'A'], 'a');
+ mockLongpress('b', ['b', 'B'], 'b');
+ mockLongpress('c', ['c', 'C'], 'c');
+ // TODO(rsadam@): Test cancelling longpress.
}

Powered by Google App Engine
This is Rietveld 408576698