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

Side by Side Diff: chrome/test/data/webui/media_router/media_router_elements_browsertest.js

Issue 1807803002: [Media Router WebUI] Handle on dialog load focus on Mac. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changes per imcheng@'s comments. Created 4 years, 9 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/resources/media_router/elements/media_router_header/media_router_header.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** @fileoverview Runs the Media Router Polymer elements tests. */ 5 /** @fileoverview Runs the Media Router Polymer elements tests. */
6 6
7 /** @const {string} Path to source root. */ 7 /** @const {string} Path to source root. */
8 var ROOT_PATH = '../../../../../'; 8 var ROOT_PATH = '../../../../../';
9 9
10 // Polymer BrowserTest fixture. 10 // Polymer BrowserTest fixture.
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 66
67 var requiredOwnedAriaRoleMissingSelectors = [ 67 var requiredOwnedAriaRoleMissingSelectors = [
68 '#cast-mode-list', 68 '#cast-mode-list',
69 '#sink-list' 69 '#sink-list'
70 ]; 70 ];
71 71
72 // Enable when failure is resolved. 72 // Enable when failure is resolved.
73 // AX_ARIA_08: http://crbug.com/591552 73 // AX_ARIA_08: http://crbug.com/591552
74 this.accessibilityAuditConfig.ignoreSelectors( 74 this.accessibilityAuditConfig.ignoreSelectors(
75 'requiredOwnedAriaRoleMissing', requiredOwnedAriaRoleMissingSelectors); 75 'requiredOwnedAriaRoleMissing', requiredOwnedAriaRoleMissingSelectors);
76
77 // This element is used as a focus placeholder on dialog open, then
78 // deleted. The user will be unable to tab to it. Remove when there is a
79 // long term fix.
80 this.accessibilityAuditConfig.ignoreSelectors(
81 'focusableElementNotVisibleAndNotAriaHidden', '#focus-placeholder');
76 }, 82 },
77 }; 83 };
78 84
79 TEST_F('MediaRouterElementsBrowserTest', 'IssueBanner', function() { 85 TEST_F('MediaRouterElementsBrowserTest', 'IssueBanner', function() {
80 issue_banner.registerTests(); 86 issue_banner.registerTests();
81 mocha.run(); 87 mocha.run();
82 }); 88 });
83 89
84 // The media-router-container tests are being split into multiple parts due to 90 // The media-router-container tests are being split into multiple parts due to
85 // timeout issues on bots. 91 // timeout issues on bots.
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 'MediaRouterSearchHighlighter', 133 'MediaRouterSearchHighlighter',
128 function() { 134 function() {
129 media_router_search_highlighter.registerTests(); 135 media_router_search_highlighter.registerTests();
130 mocha.run(); 136 mocha.run();
131 }); 137 });
132 138
133 TEST_F('MediaRouterElementsBrowserTest', 'MediaRouterRouteDetails', function() { 139 TEST_F('MediaRouterElementsBrowserTest', 'MediaRouterRouteDetails', function() {
134 route_details.registerTests(); 140 route_details.registerTests();
135 mocha.run(); 141 mocha.run();
136 }); 142 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/media_router/elements/media_router_header/media_router_header.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698