Index: chrome/browser/ui/webui/extensions/extension_settings_browsertest.js |
diff --git a/chrome/browser/ui/webui/extensions/extension_settings_browsertest.js b/chrome/browser/ui/webui/extensions/extension_settings_browsertest.js |
index 8183dc84cac804e3b2b14191ee045603b73a5346..ada780243ecd89d88c4f542a809a5163cce9bd13 100644 |
--- a/chrome/browser/ui/webui/extensions/extension_settings_browsertest.js |
+++ b/chrome/browser/ui/webui/extensions/extension_settings_browsertest.js |
@@ -88,6 +88,7 @@ ExtensionSettingsWebUITest.prototype = { |
/** @protected */ |
verifyDeveloperModeWorks: function() { |
+ this.ignoreDevModeA11yFailures(); |
var extensionSettings = getRequiredElement('extension-settings'); |
assertFalse(extensionSettings.classList.contains('dev-mode')); |
$('toggle-dev-on').click(); |
@@ -114,7 +115,25 @@ ExtensionSettingsWebUITest.prototype = { |
this.verifyDeveloperModeWorks, |
testDone]; |
this.nextStep(); |
- } |
+ }, |
+ |
+ /** |
+ * TODO(hcarmona): Remove this as part of fixing crbug.com/463245. |
+ * Will ignore accessibility failures caused by the transition when developer |
+ * mode is enabled. |
Devlin
2015/04/27 15:49:24
nit: I'd comment on why we ignore it, and why it's
|
+ * @protected |
+ */ |
+ ignoreDevModeA11yFailures: function() { |
+ this.accessibilityAuditConfig.ignoreSelectors( |
+ 'focusableElementNotVisibleAndNotAriaHidden', |
+ '#load-unpacked'); |
+ this.accessibilityAuditConfig.ignoreSelectors( |
+ 'focusableElementNotVisibleAndNotAriaHidden', |
+ '#pack-extension'); |
+ this.accessibilityAuditConfig.ignoreSelectors( |
+ 'focusableElementNotVisibleAndNotAriaHidden', |
+ '#update-extensions-now'); |
+ }, |
}; |
// Verify that developer mode doesn't change behavior when the number of |