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

Unified Diff: ui/file_manager/integration_tests/file_manager/folder_shortcuts.js

Issue 2046143002: Send 'key' from fake keyboard events generated by file_manager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix additional tests I missed Created 4 years, 6 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: ui/file_manager/integration_tests/file_manager/folder_shortcuts.js
diff --git a/ui/file_manager/integration_tests/file_manager/folder_shortcuts.js b/ui/file_manager/integration_tests/file_manager/folder_shortcuts.js
index e3f5f81c708200c0d1b2a701df5a039d6b3fb3f7..3f010c1bdadf9b1a29c0c1bb4656b64680507a43 100644
--- a/ui/file_manager/integration_tests/file_manager/folder_shortcuts.js
+++ b/ui/file_manager/integration_tests/file_manager/folder_shortcuts.js
@@ -299,7 +299,7 @@ testcase.traverseFolderShortcuts = function() {
// Shortcut to C should be selected.
function() {
remoteCall.callRemoteTestUtil('fakeKeyDown', windowId,
- ['#file-list', 'U+0034', true, false, false], this.next);
+ ['#file-list', '4', 'U+0034', true, false, false], this.next);
},
function(result) {
chrome.test.assertTrue(result);
@@ -311,7 +311,7 @@ testcase.traverseFolderShortcuts = function() {
// Shortcut to C should be selected.
function() {
remoteCall.callRemoteTestUtil('fakeKeyDown', windowId,
- ['#directory-tree', 'Up', false, false, false], this.next);
+ ['#directory-tree', 'ArrowUp', 'Up', false, false, false], this.next);
},
function(result) {
chrome.test.assertTrue(result);
@@ -322,7 +322,8 @@ testcase.traverseFolderShortcuts = function() {
// Current directory should be C.
function() {
remoteCall.callRemoteTestUtil('fakeKeyDown', windowId,
- ['#directory-tree', 'Enter', false, false, false], this.next);
+ ['#directory-tree', 'Enter', 'Enter', false, false, false],
+ this.next);
},
function(result) {
chrome.test.assertTrue(result);

Powered by Google App Engine
This is Rietveld 408576698