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

Unified Diff: chrome/browser/ui/webui/downloads_ui_browsertest_base.js

Issue 1447693002: [a11y] Bring accessibility-audit up to date: v2.10.0 release. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix the rest of the ChromeOS tests. File crbugs and update existing ones. Created 5 years, 1 month 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/browser/ui/webui/downloads_ui_browsertest_base.js
diff --git a/chrome/browser/ui/webui/downloads_ui_browsertest_base.js b/chrome/browser/ui/webui/downloads_ui_browsertest_base.js
index 4af9e7e6a568e9b6e1105efdfeca88a1c9743f3c..24ec2791ee1927769e8d8c6265016272955ef6aa 100644
--- a/chrome/browser/ui/webui/downloads_ui_browsertest_base.js
+++ b/chrome/browser/ui/webui/downloads_ui_browsertest_base.js
@@ -47,6 +47,8 @@ BaseDownloadsWebUITest.prototype = {
* @override
*/
setUp: function() {
+ testing.Test.prototype.setUp.call(this);
+
this.createdDownloads = [];
// The entries will begin at 1:00 AM on Sept 2, 2008, and will be spaced
@@ -58,6 +60,26 @@ BaseDownloadsWebUITest.prototype = {
}
downloads.Manager.updateAll(this.createdDownloads);
expectEquals(downloads.Manager.size(), TOTAL_RESULT_COUNT);
+
+ this.updateAccessibilityAuditConfig();
+ },
+
+ /**
+ * Disables failing accessibility audits. This should be removed when all
+ * audit issues have been resolved.
+ */
+ updateAccessibilityAuditConfig: function() {
+ // Enable when failure is resolved.
+ // AX_TEXT_01: http://crbug.com/559217
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'controlsWithoutLabel',
+ '#term');
+
+ // Enable when failure is resolved.
+ // AX_FOCUS_03: http://crbug.com/559219
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'tabIndexGreaterThanZero',
+ '#term');
},
/**

Powered by Google App Engine
This is Rietveld 408576698