fix: ensure document is fetched from correct node

dequelabs · Jul 13, 2017 · b28597c · b28597c
1 parent f0fe551
commit b28597c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/rules/color-contrast-matches.js b/lib/rules/color-contrast-matches.js
@@ -1,8 +1,7 @@
 /* global document */
 
 var nodeName = node.nodeName.toUpperCase(),
-	nodeType = node.type,
-	doc = axe.commons.dom.getRootNode(node);
+	nodeType = node.type;
 
 if (node.getAttribute('aria-disabled') === 'true' || axe.commons.dom.findUp(node, '[aria-disabled="true"]')) {
 	return false;
@@ -40,6 +39,7 @@ if (nodeName === 'LABEL' || nodeParentLabel) {
 		relevantNode = nodeParentLabel;
 	}
 	// explicit label of disabled input
+	let doc = axe.commons.dom.getRootNode(relevantNode);
 	var candidate = relevantNode.htmlFor && doc.getElementById(relevantNode.htmlFor);
 	if (candidate && candidate.disabled) {
 		return false;
@@ -55,6 +55,7 @@ if (nodeName === 'LABEL' || nodeParentLabel) {
 
 // label of disabled control associated w/ aria-labelledby
 if (node.id) {
+	let doc = axe.commons.dom.getRootNode(node);
 	var candidate = doc.querySelector('[aria-labelledby~=' + axe.commons.utils.escapeSelector(node.id) + ']');
 	if (candidate && candidate.disabled) {
 		return false;