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

Unified Diff: chrome/test/data/webui/net_internals/net_internals_test.js

Issue 21944004: Enable the accessibility audit for chrome://net-internals (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 4 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 | « chrome/browser/resources/net_internals/top_bar_view.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/net_internals/net_internals_test.js
diff --git a/chrome/test/data/webui/net_internals/net_internals_test.js b/chrome/test/data/webui/net_internals/net_internals_test.js
index ff5421bad6eb71ab60e5133a720d01c6d3482082..b107e308fe683f6485dfa71c7a5909ad5475118d 100644
--- a/chrome/test/data/webui/net_internals/net_internals_test.js
+++ b/chrome/test/data/webui/net_internals/net_internals_test.js
@@ -56,6 +56,24 @@ var NetInternalsTest = (function() {
isAsync: true,
setUp: function() {
+ // Enforce accessibility auditing, but suppress some false positives.
+ this.accessibilityIssuesAreErrors = true;
+ // False positive because a unicode character is used to draw a square.
+ // If it was actual text it'd be too low-contrast, but a square is fine.
+ this.accessibilityAuditConfig().ignoreSelectors(
+ 'lowContrastElements', '#timeline-view-selection-ul label');
+ // Suppress this error; the black-on-gray button is readable.
+ this.accessibilityAuditConfig().ignoreSelectors(
+ 'lowContrastElements', '#export-view-save-log-file');
+ // False positive because the background color highlights and then
+ // fades out with a transition when there's an error.
+ this.accessibilityAuditConfig().ignoreSelectors(
+ 'lowContrastElements', '#hsts-view-query-output span');
+ // False positives for unknown reason.
+ this.accessibilityAuditConfig().ignoreSelectors(
+ 'focusableElementNotVisibleAndNotAriaHidden',
+ '#hsts-view-tab-content *');
+
// Wrap g_browser.receive around a test function so that assert and expect
// functions can be called from observers.
g_browser.receive =
« no previous file with comments | « chrome/browser/resources/net_internals/top_bar_view.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698