Add a mechanism to ignore certain elements for accessibility audit on a per-test basis.

chrome/test/data/webui/accessibility_audit_browsertest.js

Index: chrome/test/data/webui/accessibility_audit_browsertest.js
diff --git a/chrome/test/data/webui/accessibility_audit_browsertest.js b/chrome/test/data/webui/accessibility_audit_browsertest.js
index 9af61c0f2980b72c31409f67ac2e0171230116c4..407116159586b7deb137b5ff58ccd9de9a100e3b 100644
--- a/chrome/test/data/webui/accessibility_audit_browsertest.js
+++ b/chrome/test/data/webui/accessibility_audit_browsertest.js
@@ -38,6 +38,9 @@ WebUIAccessibilityAuditBrowserTest.prototype = {
    */
   expectedErrors: null,
 
+  isAsync: false,
+  accessibilityAuditConfig: new axs.AuditConfiguration(),
+
   tearDown: function() {
     if (this.expectedErrors != null)
       expectEquals(this.expectedErrors, this.getAccessibilityErrors().length);
@@ -119,7 +122,7 @@ function expectAuditWillNotRun() {
 function expectAuditWillRun(times) {
   var audit = createMockAudit();
   var realAudit = axs.Audit;
-  var expectedInvocation = audit.expects(exactly(times)).run();
+  var expectedInvocation = audit.expects(exactly(times)).run(ANYTHING);
   var willArgs = [];
   for (var i = 0; i < times; i++)
     willArgs.push(callFunction(realAudit.run));
@@ -188,7 +191,6 @@ TEST_F('WebUIAccessibilityAuditBrowserTest_TestsDisabledInFixture',
 TEST_F('WebUIAccessibilityAuditBrowserTest_TestsDisabledInFixture',
        'testWithAuditFailures_shouldFail',
        function() {
-         console.log(axs.Audit);
   expectAuditWillRun(1);
   this.enableAccessibilityChecks();
   addAuditFailures();
@@ -282,3 +284,22 @@ TEST_F('WebUIAccessibilityAuditBrowserTest_IssuesAreWarnings',
 
   addAuditFailures();
 });
+
+// Tests that the accessibility audit will run twice if expectAccessibilityOk()

[dmazzoni, 2012/12/21 20:00:35]

Update this comment

[aboxhall, 2012/12/21 23:12:04]

Done.

+// is called during the test function and |runAccessibilityChecks| is true in
+// the test fixture.
+TEST_F('WebUIAccessibilityAuditBrowserTest_IssuesAreWarnings',
+       'testCanIgnoreSelectors',
+        function() {
+  expectAuditWillRun(1);
+  // lowContrastElements should pass as the failing element is ignored.

[dmazzoni, 2012/12/21 20:00:35]

Would it be possible to make this test self-contained,
rather than relative to another test?

I'd prefer something like:
1. Run the test normally, expect 1 warning
2. Ignore selectors
3. Run the test again, this time expect 0 warnings

[aboxhall, 2012/12/21 23:12:04]

Done.

+  this.expectedWarnings = 0;
+  this.expectedErrors = 2;
+  expectReportConsoleWarning();
+
+  this.enableAccessibilityChecks();
+
+  this.accessibilityAuditConfig.ignoreSelectors('lowContrastElements', 'P');
+
+  addAuditFailures();
+});