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

Unified Diff: chrome/browser/ui/webui/options/options_browsertest.js

Issue 1973823002: Disable failing accessibility audit in OptionsWebUITest tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update bugref Created 4 years, 7 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 1b65090dd13c16140ec991b9a65b798b4a6afe3a..c951d5279b0f52dc6ce4495ae36e26ed537e9761 100644
--- a/chrome/browser/ui/webui/options/options_browsertest.js
+++ b/chrome/browser/ui/webui/options/options_browsertest.js
@@ -103,6 +103,18 @@ OptionsWebUITest.prototype = {
this.accessibilityAuditConfig.ignoreSelectors(
'linkWithUnclearPurpose',
linkWithUnclearPurposeSelectors);
+
+ // Causes testDefaultZoomFactor to flake. See http://crbug.com/611233.
+ var requiredOwnedAriaRoleMissingSelectors = [
+ '#default-search-engine-list',
+ '#other-search-engine-list',
+ ];
+
+ // Enable when failure is resolved.
+ // AX_ARIA_08: http://crbug.com/606657
+ this.accessibilityAuditConfig.ignoreSelectors(
+ 'requiredOwnedAriaRoleMissing',
+ requiredOwnedAriaRoleMissingSelectors);
},
};
@@ -170,7 +182,7 @@ TEST_F('OptionsWebUITest', 'DISABLED_testRefreshStaysOnCurrentPage',
/**
* Test the default zoom factor select element.
*/
-TEST_F('OptionsWebUITest', 'testDefaultZoomFactor', function() {
+TEST_F('OptionsWebUITest', 'MAYBE_testDefaultZoomFactor', function() {
michaelpg 2016/05/12 03:36:24 remove MAYBE_
tapted 2016/05/12 03:41:00 Done. (whoops)
// The expected minimum length of the |defaultZoomFactor| element.
var defaultZoomFactorMinimumLength = 10;
// Verify that the zoom factor element exists.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698