From df400939addf6dbc5f2a9e1d52a6219f356f82d8 Mon Sep 17 00:00:00 2001 From: Alice Boxhall Date: Fri, 4 Sep 2015 12:33:14 -0400 Subject: [PATCH] Revert #150 --- Changelog.md | 4 ++++ src/js/AuditRule.js | 7 ------- test/js/audit-rule-test.js | 10 ---------- 3 files changed, 4 insertions(+), 17 deletions(-) diff --git a/Changelog.md b/Changelog.md index 596fe376..4b2de32f 100644 --- a/Changelog.md +++ b/Changelog.md @@ -1,3 +1,7 @@ +### Bug fixes: + +* Revert #150 which was causing the extension not to work. + ## 2.9.0 - 2015-09-04 ## 2.9.0-rc.0 - 2015-08-21 diff --git a/src/js/AuditRule.js b/src/js/AuditRule.js index 0c0c8ee3..9912b1e8 100644 --- a/src/js/AuditRule.js +++ b/src/js/AuditRule.js @@ -183,13 +183,6 @@ axs.AuditRule.collectMatchingElements = function(node, matcher, collection, } } - // If it is a iframe, get the contentDocument - if (element && element.localName == 'iframe' && element.contentDocument) { - axs.AuditRule.collectMatchingElements(element.contentDocument, - matcher, - collection, - opt_shadowRoot); - } // If it is neither the parent of a ShadowRoot, a element, nor // a element recurse normally. var child = node.firstChild; diff --git a/test/js/audit-rule-test.js b/test/js/audit-rule-test.js index 1770594e..57562c91 100644 --- a/test/js/audit-rule-test.js +++ b/test/js/audit-rule-test.js @@ -41,16 +41,6 @@ equal(matched.length, DIV_COUNT); }); - test("Iframe with simple DOM", function () { - var ifrm = document.createElement("IFRAME"); - var container = document.getElementById('qunit-fixture'); - container.appendChild(ifrm); - ifrm.contentDocument.body.appendChild(buildTestDom()); - var matched = []; - axs.AuditRule.collectMatchingElements(container, matcher, matched); - equal(matched.length, DIV_COUNT); - }); - test("With shadow DOM with no content insertion point", function () { var container = document.getElementById('qunit-fixture'); container.appendChild(buildTestDom());