Merge remote-tracking branch 'upstream/master'

ricksbrown · ricksbrown · commit 00d8606f1842 · 2015-05-01T20:18:01.000+10:00
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -32,7 +32,7 @@ module.exports = function(grunt) {
           "summary_detail_level": 3,
           "warning_level": "VERBOSE",
           "compilation_level": "SIMPLE_OPTIMIZATIONS",
-          "output_wrapper": "'<%= grunt.file.read('scripts/output_wrapper.txt') %>'",
+          "output_wrapper": "<%= grunt.file.read('scripts/output_wrapper.txt') %>",
           "externs": "./src/js/externs/externs.js"
         }
       }
diff --git a/README.md b/README.md
@@ -76,6 +76,33 @@ The Accessibility Developer Tools project includes a command line runner for the
 
 The runner will load the specified file or URL in a headless browser, inject axs_testing.js, run the audit and output the report text.
 
+### Run audit from Selenium WebDriver (Scala):
+     val driver = org.openqa.selenium.firefox.FirefoxDriver //use driver of your choice
+     val jse = driver.asInstanceOf[JavascriptExecutor]
+     jse.executeScript(scala.io.Source.fromURL("https://raw.githubusercontent.com/GoogleChrome/" +
+       "accessibility-developer-tools/stable/dist/js/axs_testing.js").mkString)
+     val report = js.executeScript("var results = axs.Audit.run();return axs.Audit.createReport(results);")
+     println(report)
+    
+### Run audit from Selenium WebDriver (Scala)(with caching):
+     val cache = collection.mutable.Map[String, String]()
+     val driver = org.openqa.selenium.firefox.FirefoxDriver //use driver of your choice
+     val jse = driver.asInstanceOf[JavascriptExecutor]
+     def getUrlSource(arg: String): String = cache get arg match {
+        case Some(result) => result
+        case None =>
+          val result: String = scala.io.Source.fromURL(arg).mkString
+          cache(arg) = result
+          result
+      }
+     jse.executeScript(getUrlSource("https://raw.githubusercontent.com/GoogleChrome/" +
+       "accessibility-developer-tools/stable/dist/js/axs_testing.js"))
+     val report = js.executeScript("var results = axs.Audit.run();return axs.Audit.createReport(results);")
+     println(report)
+    
+If println() outputs nothing, check if you need to set DesiredCapabilities for your WebDriver (such as loggingPrefs):
+https://code.google.com/p/selenium/wiki/DesiredCapabilities
+
 ## Using the results
 
 ### Interpreting the result
diff --git a/src/audits/AriaOnReservedElement.js b/src/audits/AriaOnReservedElement.js
@@ -22,7 +22,7 @@ goog.require('axs.properties');
 axs.AuditRules.addRule({
     name: 'ariaOnReservedElement',
     heading: 'This element does not support ARIA roles, states and properties',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_12--this-element-does-not-support-aria-roles-states-and-properties',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_12',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return !axs.properties.canTakeAriaAttributes(element);
diff --git a/src/audits/AriaOwnsDescendant.js b/src/audits/AriaOwnsDescendant.js
@@ -27,7 +27,7 @@ axs.AuditRules.addRule({
     // Also: other "bad hierarchy" tests - e.g. active-descendant owning a non-descendant...
     name: 'ariaOwnsDescendant',
     heading: 'aria-owns should not be used if ownership is implicit in the DOM',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_06--aria-owns-should-not-be-used-if-ownership-is-implicit-in-the-dom',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_06',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, '[aria-owns]');
diff --git a/src/audits/AriaRoleNotScoped.js b/src/audits/AriaRoleNotScoped.js
@@ -25,7 +25,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'ariaRoleNotScoped',
     heading: 'Elements with ARIA roles must be in the correct scope',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_09--elements-with-aria-roles-must-be-in-the-correct-scope',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_09',
     severity: axs.constants.Severity.SEVERE,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, '[role]');
diff --git a/src/audits/AudioWithoutControls.js b/src/audits/AudioWithoutControls.js
@@ -22,7 +22,7 @@ goog.require('axs.constants.Severity');
 axs.AuditRules.addRule({
     name: 'audioWithoutControls',
     heading: 'Audio elements should have controls',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_audio_01--audio-elements-should-have-controls',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_audio_01',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, 'audio[autoplay]');
diff --git a/src/audits/BadAriaAttribute.js b/src/audits/BadAriaAttribute.js
@@ -33,7 +33,7 @@ goog.require('axs.constants');
     var badAriaAttribute = {
         name: 'badAriaAttribute',
         heading: 'This element has an invalid ARIA attribute',
-        url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_11--this-element-has-an-invalid-aria-attribute',
+        url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_11',
         severity: axs.constants.Severity.WARNING,
         relevantElementMatcher: function(element) {
             var attributes = element.attributes;
diff --git a/src/audits/BadAriaAttributeValue.js b/src/audits/BadAriaAttributeValue.js
@@ -23,7 +23,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'badAriaAttributeValue',
     heading: 'ARIA state and property values must be valid',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_04--aria-state-and-property-values-must-be-valid',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_04',
     severity: axs.constants.Severity.SEVERE,
     relevantElementMatcher: function(element) {
         var selector = axs.utils.getSelectorForAriaProperties(axs.constants.ARIA_PROPERTIES);
diff --git a/src/audits/BadAriaRole.js b/src/audits/BadAriaRole.js
@@ -23,7 +23,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'badAriaRole',
     heading: 'Elements with ARIA roles must use a valid, non-abstract ARIA role',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_01--elements-with-aria-roles-must-use-a-valid-non-abstract-aria-role',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_01',
     severity: axs.constants.Severity.SEVERE,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, '[role]');
diff --git a/src/audits/ControlsWithoutLabel.js b/src/audits/ControlsWithoutLabel.js
@@ -23,7 +23,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'controlsWithoutLabel',
     heading: 'Controls and media elements should have labels',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_text_01--controls-and-media-elements-should-have-labels',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_text_01',
     severity: axs.constants.Severity.SEVERE,
     relevantElementMatcher: function(element) {
         var controlsSelector = ['input:not([type="hidden"]):not([disabled])',
diff --git a/src/audits/DuplicateId.js b/src/audits/DuplicateId.js
@@ -22,7 +22,7 @@ goog.require('axs.constants.Severity');
 axs.AuditRules.addRule({
     name: 'duplicateId',
     heading: 'An element\'s ID must be unique in the DOM',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_html_02--an-elements-id-must-be-unique-in-the-dom',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_html_02',
     severity: axs.constants.Severity.SEVERE,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, '[id]');
diff --git a/src/audits/FocusableElementNotVisibleAndNotAriaHidden.js b/src/audits/FocusableElementNotVisibleAndNotAriaHidden.js
@@ -23,7 +23,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'focusableElementNotVisibleAndNotAriaHidden',
     heading: 'These elements are focusable but either invisible or obscured by another element',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_focus_01--these-elements-are-focusable-but-either-invisible-or-obscured-by-another-element',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_focus_01',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         var isFocusable = axs.browserUtils.matchSelector(
diff --git a/src/audits/HumanLangMissing.js b/src/audits/HumanLangMissing.js
@@ -18,7 +18,7 @@ goog.require('axs.constants.Severity');
 axs.AuditRules.addRule({
     name: 'humanLangMissing',
     heading: 'The web page should have the content\'s human language indicated in the markup',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_html_01--the-web-page-should-have-the-contents-human-language-indicated-in-the-markup',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_html_01',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return element instanceof element.ownerDocument.defaultView.HTMLHtmlElement;
diff --git a/src/audits/ImageWithoutAltText.js b/src/audits/ImageWithoutAltText.js
@@ -20,7 +20,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'imagesWithoutAltText',
     heading: 'Images should have an alt attribute',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_text_02--images-should-have-an-alt-attribute-unless-they-have-an-aria-role-of-presentation',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_text_02',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, 'img') &&
diff --git a/src/audits/LinkWithUnclearPurpose.js b/src/audits/LinkWithUnclearPurpose.js
@@ -22,7 +22,7 @@ goog.require('axs.constants.Severity');
 axs.AuditRules.addRule({
     name: 'linkWithUnclearPurpose',
     heading: 'The purpose of each link should be clear from the link text',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_text_04--the-purpose-of-each-link-should-be-clear-from-the-link-text',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_text_04',
     severity: axs.constants.Severity.WARNING,
     /**
      * @param {Element} element
diff --git a/src/audits/LowContrast.js b/src/audits/LowContrast.js
@@ -23,7 +23,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'lowContrastElements',
     heading: 'Text elements should have a reasonable contrast ratio',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_color_01--text-elements-should-have-a-reasonable-contrast-ratio',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_color_01',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return axs.properties.hasDirectTextDescendant(element);
diff --git a/src/audits/MainRoleOnInappropriateElement.js b/src/audits/MainRoleOnInappropriateElement.js
@@ -23,7 +23,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'mainRoleOnInappropriateElement',
     heading: 'role=main should only appear on significant elements',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_05--rolemain-should-only-appear-on-significant-elements',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_05',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, '[role~=main]');
diff --git a/src/audits/MeaningfulBackgroundImage.js b/src/audits/MeaningfulBackgroundImage.js
@@ -26,7 +26,7 @@ axs.AuditRules.addRule({
         return !axs.utils.isElementOrAncestorHidden(element);
     },
     heading: 'Meaningful images should not be used in element backgrounds',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_image_01--meaningful-images-should-not-be-used-in-element-backgrounds',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_image_01',
     test: function(el) {
         if (el.textContent && el.textContent.length > 0) {
           return false;
diff --git a/src/audits/MultipleAriaOwners.js b/src/audits/MultipleAriaOwners.js
@@ -23,7 +23,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'multipleAriaOwners',
     heading: 'An element\'s ID must not be present in more that one aria-owns attribute at any time',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_07--an-element-can-have-only-one-explicit-aria-owner',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_07',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         /*
diff --git a/src/audits/NonExistentAriaRelatedElement.js b/src/audits/NonExistentAriaRelatedElement.js
@@ -25,7 +25,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'nonExistentAriaRelatedElement',
     heading: 'ARIA attributes which refer to other elements by ID should refer to elements which exist in the DOM',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_02--aria-labelledby-attributes-should-refer-to-an-element-which-exists-in-the-dom',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_02',
     severity: axs.constants.Severity.SEVERE,
     relevantElementMatcher: function(element) {
         var idrefTypes = ['idref', 'idref_list'];
diff --git a/src/audits/PageWithoutTitle.js b/src/audits/PageWithoutTitle.js
@@ -18,7 +18,7 @@ goog.require('axs.constants.Severity');
 axs.AuditRules.addRule({
     name: 'pageWithoutTitle',
     heading: 'The web page should have a title that describes topic or purpose',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_title_01--the-web-page-should-have-a-title-that-describes-topic-or-purpose',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_title_01',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return element.tagName.toLowerCase() == "html";
diff --git a/src/audits/RequiredAriaAttributeMissing.js b/src/audits/RequiredAriaAttributeMissing.js
@@ -21,7 +21,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'requiredAriaAttributeMissing',
     heading: 'Elements with ARIA roles must have all required attributes for that role',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_03--elements-with-aria-roles-must-have-all-required-attributes-for-that-role',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_03',
     severity: axs.constants.Severity.SEVERE,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, '[role]');
diff --git a/src/audits/RequiredOwnedAriaRoleMissing.js b/src/audits/RequiredOwnedAriaRoleMissing.js
@@ -24,7 +24,7 @@ goog.require('axs.utils');
     var spec = {
         name: 'requiredOwnedAriaRoleMissing',
         heading: 'Elements with ARIA roles must ensure required owned elements are present',
-        url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_08--elements-with-aria-roles-must-ensure-required-owned-elements-are-present',
+        url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_08',
         severity: axs.constants.Severity.SEVERE,
         relevantElementMatcher: function(element) {
             if (!axs.browserUtils.matchSelector(element, '[role]'))
@@ -83,4 +83,3 @@ goog.require('axs.utils');
     }
     axs.AuditRules.addRule(spec);
 })();
-
diff --git a/src/audits/TabIndexGreaterThanZero.js b/src/audits/TabIndexGreaterThanZero.js
@@ -19,7 +19,7 @@ goog.require('axs.constants.Severity');
 axs.AuditRules.addRule({
     name: "tabIndexGreaterThanZero",
     heading: "Avoid positive integer values for tabIndex",
-    url: "https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_focus_03--avoid-positive-integer-values-for-tabindex",
+    url: "https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_focus_03",
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
       var selector = "[tabindex]";
diff --git a/src/audits/UnfocusableElementsWithOnClick.js b/src/audits/UnfocusableElementsWithOnClick.js
@@ -19,7 +19,7 @@ goog.require('axs.utils');
 axs.AuditRules.addRule({
     name: 'unfocusableElementsWithOnClick',
     heading: 'Elements with onclick handlers must be focusable',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_focus_02--elements-with-onclick-handlers-must-be-focusable',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_focus_02',
     severity: axs.constants.Severity.WARNING,
     opt_requiresConsoleAPI: true,
     relevantElementMatcher: function(element) {
diff --git a/src/audits/UnsupportedAriaAttribute.js b/src/audits/UnsupportedAriaAttribute.js
@@ -35,7 +35,7 @@ goog.require('axs.utils');
     var unsupportedAriaAttribute = {
         name: 'unsupportedAriaAttribute',
         heading: 'This element has an unsupported ARIA attribute',
-        url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_aria_10--this-element-has-an-unsupported-aria-attribute',
+        url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_10',
         severity: axs.constants.Severity.SEVERE,
         relevantElementMatcher: function(element) {
             return axs.browserUtils.matchSelector(element, selector);
diff --git a/src/audits/VideoWithoutCaptions.js b/src/audits/VideoWithoutCaptions.js
@@ -19,7 +19,7 @@ goog.require('axs.constants.Severity');
 axs.AuditRules.addRule({
     name: 'videoWithoutCaptions',
     heading: 'Video elements should use <track> elements to provide captions',
-    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#-ax_video_01--video-elements-should-use-track-elements-to-provide-captions',
+    url: 'https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_video_01',
     severity: axs.constants.Severity.WARNING,
     relevantElementMatcher: function(element) {
         return axs.browserUtils.matchSelector(element, 'video');
diff --git a/src/js/externs/externs.js b/src/js/externs/externs.js
@@ -25,28 +25,6 @@ HTMLLabelElement.prototype.control;
  */
 ShadowRoot.prototype.olderShadowRoot;
 
-/**
- * @constructor
- * @extends {HTMLElement}
- */
-function HTMLContentElement() {}
-
-/**
- * @return {Array.<Node>}
- */
-HTMLContentElement.prototype.getDistributedNodes = function() {};
-
-/**
- * @constructor
- * @extends {HTMLElement}
- */
-function HTMLShadowElement() {}
-
-/**
- * @return {Array.<Node>}
- */
-HTMLShadowElement.prototype.getDistributedNodes = function() {};
-
 /**
  * Note: will be deprecated at some point; prefer shadowRoot if it exists.
  * @type {HTMLShadowElement}

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ module.exports = function(grunt) {`
`32`	`32`	`"summary_detail_level": 3,`
`33`	`33`	`"warning_level": "VERBOSE",`
`34`	`34`	`"compilation_level": "SIMPLE_OPTIMIZATIONS",`
`35`		`- "output_wrapper": "'<%= grunt.file.read('scripts/output_wrapper.txt') %>'",`
	`35`	`+ "output_wrapper": "<%= grunt.file.read('scripts/output_wrapper.txt') %>",`
`36`	`36`	`"externs": "./src/js/externs/externs.js"`
`37`	`37`	`}`
`38`	`38`	`}`