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

Unified Diff: chrome/browser/ui/webui/options/options_browsertest.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: Changes per dbeam@'s comment. 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/options/options_browsertest.js
diff --git a/chrome/browser/ui/webui/options/options_browsertest.js b/chrome/browser/ui/webui/options/options_browsertest.js
index 895928aa8bdd79b1022b9667703a206dd10f4c1b..4de8c1936d25447f7584e29e15ed794d5f2191a0 100644
--- a/chrome/browser/ui/webui/options/options_browsertest.js
+++ b/chrome/browser/ui/webui/options/options_browsertest.js
@@ -79,6 +79,31 @@ OptionsWebUITest.prototype = {
this.mockHandler.stubs().observePrefs(ANYTHING);
this.mockHandler.stubs().coreOptionsUserMetricsAction(ANYTHING);
},
+
+ /** @override */
+ setUp: function() {
+ OptionsBrowsertestBase.prototype.setUp.call(this);
+
+ // Enable when failure is resolved.
+ // AX_ARIA_10: http://crbug.com/559329
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'unsupportedAriaAttribute',
+ '#profiles-list');
+
+ var linkWithUnclearPurposeSelectors = [
+ '#sync-overview > A',
+ '#privacy-explanation > A',
+ '#languages-section > .settings-row > A',
+ '#cloudprint-options-mdns > .settings-row > A',
+ '#do-not-track-confirm-overlay > .action-area > .hbox.stretch > A',
+ ];
+
+ // Enable when failure is resolved.
+ // AX_TEXT_04: http://crbug.com/559318
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'linkWithUnclearPurpose',
+ linkWithUnclearPurposeSelectors);
+ },
};
/**
@@ -378,6 +403,44 @@ OptionsWebUIExtendedTest.prototype = {
/** @override */
typedefCppFixture: 'OptionsBrowserTest',
+ /** @override */
+ setUp: function() {
+ OptionsWebUITest.prototype.setUp.call(this);
+
+ // Enable when failure is resolved.
+ // AX_ARIA_10: http://crbug.com/559329
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'unsupportedAriaAttribute',
+ '#profiles-list');
+
+ var controlsWithoutLabelSelectors = [
+ '#cookies-view-page > .content-area.cookies-list-content-area > *',
+ '#other-search-engine-list > .deletable-item > DIV > *',
+ ];
+
+ // Enable when failure is resolved.
+ // AX_TEXT_01: http://crbug.com/559330
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'controlsWithoutLabel',
+ controlsWithoutLabelSelectors);
+
+ var linkWithUnclearPurposeSelectors = [
+ '#sync-overview > A',
+ '#privacy-explanation > A',
+ '#languages-section > .settings-row > A',
+ '#cloudprint-options-mdns > .settings-row > A',
+ // Selectors below only affect ChromeOS tests.
+ '#privacy-section > DIV > DIV:nth-of-type(9) > A',
+ '#accessibility-learn-more',
+ ];
+
+ // Enable when failure is resolved.
+ // AX_TEXT_04: http://crbug.com/559326
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'linkWithUnclearPurpose',
+ linkWithUnclearPurposeSelectors);
+ },
+
testGenPreamble: function() {
// Start with no supervised users managed by this profile.
GEN(' ClearPref("' + SUPERVISED_USERS_PREF + '");');

Powered by Google App Engine
This is Rietveld 408576698