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

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

Issue 1052343004: [Files.app] Make menus accessible (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix test Created 5 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
« no previous file with comments | « ui/file_manager/file_manager/main.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/integration_tests/file_manager/tab_index.js
diff --git a/ui/file_manager/integration_tests/file_manager/tab_index.js b/ui/file_manager/integration_tests/file_manager/tab_index.js
index dcf8032e0f683dba73db9201c6d4ddfd03fb616b..1f43dd165e26d3fe37418f391f3f020adef65ba0 100644
--- a/ui/file_manager/integration_tests/file_manager/tab_index.js
+++ b/ui/file_manager/integration_tests/file_manager/tab_index.js
@@ -41,10 +41,11 @@ testcase.searchBoxFocus = function() {
// Check that the file list has the focus.
function(result) {
chrome.test.assertTrue(result);
- remoteCall.waitForElement(appId, ['#file-list:focus']).then(this.next);
+ remoteCall.checkNextTabFocus(appId, 'file-list').then(this.next);
},
// Check for errors.
- function(element) {
+ function(result) {
+ chrome.test.assertTrue(result);
checkIfNoErrorsOccured(this.next);
}
]);
@@ -67,10 +68,7 @@ testcase.tabindexFocus = function() {
},
// Press the Tab key.
function(element) {
- remoteCall.callRemoteTestUtil('getActiveElement',
- appId,
- [],
- this.next);
+ remoteCall.callRemoteTestUtil('getActiveElement', appId, [], this.next);
}, function(element) {
chrome.test.assertEq('list', element.attributes['class']);
remoteCall.checkNextTabFocus(appId, 'search-button').then(this.next);
@@ -114,10 +112,7 @@ testcase.tabindexFocusDownloads = function() {
},
// Press the Tab key.
function(element) {
- remoteCall.callRemoteTestUtil('getActiveElement',
- appId,
- [],
- this.next);
+ remoteCall.callRemoteTestUtil('getActiveElement', appId, [], this.next);
}, function(element) {
chrome.test.assertEq('list', element.attributes['class']);
remoteCall.checkNextTabFocus(appId, 'search-button').then(this.next);
@@ -157,10 +152,7 @@ testcase.tabindexFocusDirectorySelected = function() {
},
// Press the Tab key.
function(element) {
- remoteCall.callRemoteTestUtil('getActiveElement',
- appId,
- [],
- this.next);
+ remoteCall.callRemoteTestUtil('getActiveElement', appId, [], this.next);
}, function(element) {
chrome.test.assertEq('list', element.attributes['class']);
// Select the directory named 'photos'.
« no previous file with comments | « ui/file_manager/file_manager/main.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698