Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: test/browser/elemHideEmulation.js

Issue 29383960: Issue 3143 - Filter elements with :-abp-has() (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore
Patch Set: Added validate of element id, and fixed an infinite recursion in parsing. Created May 15, 2017, 6:10 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/browser/elemHideEmulation.js
===================================================================
--- a/test/browser/elemHideEmulation.js
+++ b/test/browser/elemHideEmulation.js
@@ -12,25 +12,32 @@
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
*/
"use strict";
-/* globals ElemHideEmulation */
+/* globals ElemHideEmulation, splitSelector,
+ parseSelector,
+ PlainSelector, HasSelector, PropsSelector */
let myUrl = document.currentScript.src;
exports.tearDown = function(callback)
{
let styleElements = document.head.getElementsByTagName("style");
while (styleElements.length)
styleElements[0].parentNode.removeChild(styleElements[0]);
+
+ let child;
+ while (child = document.body.firstChild)
+ document.body.removeChild(child);
+
callback();
};
function unexpectedError(error)
{
console.error(error);
this.ok(false, "Unexpected error: " + error);
}
@@ -64,127 +71,492 @@
else
{
styleElement = document.createElement("style");
document.head.appendChild(styleElement);
}
styleElement.sheet.insertRule(rule, styleElement.sheet.cssRules.length);
}
-function createElementWithStyle(styleBlock)
+// insert a <div> with a unique id and and empty CSS rule
+// for the the selector matching the id.
+function createElementWithStyle(styleBlock, parent)
{
let element = document.createElement("div");
element.id = findUniqueId();
- document.body.appendChild(element);
+ if (!parent)
+ document.body.appendChild(element);
+ else
+ parent.appendChild(element);
insertStyleRule("#" + element.id + " " + styleBlock);
return element;
}
-function applyElemHideEmulation(selectors)
+// Will ensure the class ElemHideEmulation is loaded
+// and then will call the callback.
+// NOTE: if it never loads, this will probably hang in an infinite
+// loop
+function loadElemHideEmulation()
{
if (typeof ElemHideEmulation == "undefined")
{
return loadScript(myUrl + "/../../../lib/common.js").then(() =>
{
return loadScript(myUrl + "/../../../chrome/content/elemHideEmulation.js");
}).then(() =>
{
- return applyElemHideEmulation(selectors);
+ return loadElemHideEmulation();
});
}
- let elemHideEmulation = new ElemHideEmulation(
- window,
- callback =>
- {
- let patterns = [];
- selectors.forEach(selector =>
- {
- patterns.push({selector});
- });
- callback(patterns);
- }, newSelectors =>
- {
- if (!newSelectors.length)
- return;
- let selector = newSelectors.join(", ");
- insertStyleRule(selector + "{display: none !important;}");
- }
- );
-
- elemHideEmulation.apply();
return Promise.resolve();
}
+// instantiate a ElemHideEmulation with @selectors.
+function applyElemHideEmulation(selectors)
+{
+ return loadElemHideEmulation().then(() =>
+ {
+ let elemHideEmulation = new ElemHideEmulation(
+ window,
+ callback =>
+ {
+ let patterns = [];
+ selectors.forEach(selector =>
+ {
+ patterns.push({selector});
+ });
+ callback(patterns);
+ },
+ newSelectors =>
+ {
+ if (!newSelectors.length)
+ return;
+ let selector = newSelectors.join(", ");
+ insertStyleRule(selector + "{display: none !important;}");
+ },
+ elements =>
+ {
+ if (!elements.length)
+ return;
+ for (let element of elements)
+ element.style.display = "none";
+ }
+ );
+
+ elemHideEmulation.apply();
+ return Promise.resolve();
+ });
+}
+
+exports.testValidId = function(test)
+{
+ loadElemHideEmulation().then(() =>
+ {
+ test.ok(!idValid(null));
+ test.ok(!idValid(undefined));
+ test.ok(!idValid(""));
+ test.ok(idValid("abcdef-u_"));
+ test.ok(idValid("-abdsc"));
+ test.ok(!idValid("--abdsc"));
+ test.ok(!idValid("0abcsd"));
+ test.ok(!idValid("-0abcsd"));
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+}
+
+exports.testParseSelector = function(test)
+{
+ loadElemHideEmulation().then(() =>
+ {
+ let selectors = parseSelector("");
+ test.equal(selectors.length, 0);
+
+ let selector = "div > :-abp-properties('background-color: rgb(0, 0, 0)')";
+ selectors = parseSelector(selector);
+ test.equal(selectors.length, 2);
+ test.ok(selectors[0] instanceof PlainSelector);
+ test.ok(selectors[1] instanceof PropsSelector);
+
+ selector = "div > :-abp-has(> div.inside) > div";
+ selectors = parseSelector(selector);
+ test.equal(selectors.length, 3);
+ test.ok(selectors[0] instanceof PlainSelector);
+ test.ok(selectors[1] instanceof HasSelector);
+ test.ok(selectors[2] instanceof PlainSelector);
+
+ selector = "div > div:-abp-has(> div.inside) > div";
+ selectors = parseSelector(selector);
+
+ test.equal(selectors.length, 3);
+ test.ok(selectors[0] instanceof PlainSelector);
+ test.ok(selectors[1] instanceof HasSelector);
+ test.ok(selectors[2] instanceof PlainSelector);
+
+ selector = "div > :-abp-has(> div.inside) > :-abp-properties('background-color: rgb(0, 0, 0)')";
+ selectors = parseSelector(selector);
+
+ test.equal(selectors.length, 4);
+ test.ok(selectors[0] instanceof PlainSelector);
+ test.ok(selectors[1] instanceof HasSelector);
+ test.ok(selectors[2] instanceof PlainSelector);
+ test.ok(selectors[3] instanceof PropsSelector);
+
+ selector = "div > :-abp-has(> div.inside > :-abp-properties('background-color: rgb(0, 0, 0)')";
+ selectors = parseSelector(selector);
+ test.equal(selectors, null);
+
+ // -abp-has-unsupported() is unknown. Ensure we fail parsing.
+ selector = 'div[arial-label="Story"]:-abp-has(> div > div > span > span:-abp-unsupported("Suggested Post"))';
+ selectors = parseSelector(selector);
+ test.equal(selectors, null);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+function buildDom(doc)
+{
+ doc.body.innerHTML = `<div id="parent">
+ <div id="middle">
+ <div id="middle1"><div id="inside" class="inside"></div></div>
+ </div>
+ <div id="sibling">
+ <div id="tohide">to hide</div>
+ </div>
+ <div id="sibling2">
+ <div id="sibling21"><div id="sibling211" class="inside"></div></div>
+ </div>
+ </div>`;
+ let parent = document.getElementById("parent");
+ let middle = document.getElementById("middle");
+ let inside = document.getElementById("inside");
+ let sibling = document.getElementById("sibling");
+ let sibling2 = document.getElementById("sibling2");
+ let toHide = document.getElementById("tohide");
+ return {parent, middle, inside, sibling, sibling2, toHide};
+}
+
+exports.testPlainSelector = function(test)
+{
+ let nodes = buildDom(document);
+
+ loadElemHideEmulation().then(() =>
+ {
+ let selector = new PlainSelector("div > div");
+
+ let iter = selector.getSelectors("foo > ");
+ let value = iter.next();
+ test.equal(value.value[0], "foo > div > div");
+ test.ok(iter.next().done);
+
+ iter = selector.getElements("", document, [document.sheet]);
+ value = iter.next();
+ test.ok(!value.done);
+ test.equal(value.value, nodes.middle);
+ value = iter.next();
+ test.ok(!value.done);
+ test.equal(value.value.id, "middle1");
+ value = iter.next();
+ test.ok(!value.done);
+ test.equal(value.value, nodes.inside);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testHasSelector = function(test)
+{
+ buildDom(document);
+
+ loadElemHideEmulation().then(() =>
+ {
+ let selector = new HasSelector("> div.inside");
+
+ let iter = selector.getSelectors("", document, document.sheet);
+ let value = iter.next();
+ test.ok(!value.done);
+ test.equal(value.value[0], "#middle1");
+
+ iter = selector.getElements("", document, document.sheet);
+ value = iter.next();
+ test.ok(!value.done);
+ test.equal(value.value.id, "middle1");
+ value = iter.next();
+ test.ok(!value.done);
+ test.equal(value.value.id, "sibling21");
+ value = iter.next();
+ test.ok(value.done);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testSplitStyleRule = function(test)
+{
+ loadElemHideEmulation().then(() =>
+ {
+ let selectors = splitSelector("div:-abp-has(div) > [-abp-properties='background-color: rgb(0, 0, 0)'] > span");
+ test.ok(selectors);
+ test.equal(selectors.length, 1, "There is only one selector");
+
+ selectors = splitSelector("div:-abp-has(div), [-abp-properties='background-color: rgb(0, 0, 0)']");
+ test.ok(selectors);
+ test.equal(selectors.length, 2, "There are two selectors");
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
exports.testVerbatimPropertySelector = function(test)
{
let toHide = createElementWithStyle("{background-color: #000}");
applyElemHideEmulation(
- ["[-abp-properties='background-color: rgb(0, 0, 0)']"]
+ [":-abp-properties('background-color: rgb(0, 0, 0)')"]
+ ).then(() =>
+ {
+ expectHidden(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testVerbatimPropertySelectorWithPrefix = function(test)
+{
+ let parent = createElementWithStyle("{background-color: #000}");
+ let toHide = createElementWithStyle("{background-color: #000}", parent);
+ applyElemHideEmulation(
+ ["div > :-abp-properties('background-color: rgb(0, 0, 0)')"]
+ ).then(() =>
+ {
+ expectVisible(test, parent);
+ expectHidden(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testVerbatimPropertySelectorWithPrefixNoMatch = function(test)
+{
+ let parent = createElementWithStyle("{background-color: #000}");
+ let toHide = createElementWithStyle("{background-color: #fff}", parent);
+ applyElemHideEmulation(
+ ["div > :-abp-properties('background-color: rgb(0, 0, 0)')"]
).then(() =>
{
+ expectVisible(test, parent);
+ expectVisible(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testVerbatimPropertySelectorWithSuffix = function(test)
+{
+ let parent = createElementWithStyle("{background-color: #000}");
+ let toHide = createElementWithStyle("{background-color: #000}", parent);
+ applyElemHideEmulation(
+ [":-abp-properties('background-color: rgb(0, 0, 0)') > div"]
+ ).then(() =>
+ {
+ expectVisible(test, parent);
+ expectHidden(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testVerbatimPropertyPseudoSelectorWithPrefixAndSuffix = function(test)
+{
+ let parent = createElementWithStyle("{background-color: #000}");
+ let middle = createElementWithStyle("{background-color: #000}", parent);
+ let toHide = createElementWithStyle("{background-color: #000}", middle);
+ applyElemHideEmulation(
+ ["div > :-abp-properties('background-color: rgb(0, 0, 0)') > div"]
+ ).then(() =>
+ {
+ expectVisible(test, parent);
+ expectVisible(test, middle);
expectHidden(test, toHide);
}).catch(unexpectedError.bind(test)).then(() => test.done());
};
exports.testPropertySelectorWithWildcard = function(test)
{
let toHide = createElementWithStyle("{background-color: #000}");
applyElemHideEmulation(
- ["[-abp-properties='*color: rgb(0, 0, 0)']"]
+ [":-abp-properties('*color: rgb(0, 0, 0)')"]
).then(() =>
{
expectHidden(test, toHide);
}).catch(unexpectedError.bind(test)).then(() => test.done());
};
exports.testPropertySelectorWithRegularExpression = function(test)
{
let toHide = createElementWithStyle("{background-color: #000}");
applyElemHideEmulation(
- ["[-abp-properties='/.*color: rgb\\(0, 0, 0\\)/']"]
+ [":-abp-properties('/.*color: rgb\\(0, 0, 0\\)/')"]
).then(() =>
{
expectHidden(test, toHide);
}).catch(unexpectedError.bind(test)).then(() => test.done());
};
exports.testPropertySelectorWithEscapedBrace = function(test)
{
let toHide = createElementWithStyle("{background-color: #000}");
applyElemHideEmulation(
- ["[-abp-properties='/background.\\x7B 0,6\\x7D : rgb\\(0, 0, 0\\)/']"]
+ [":-abp-properties('/background.\\x7B 0,6\\x7D : rgb\\(0, 0, 0\\)/')"]
).then(() =>
{
expectHidden(test, toHide);
}).catch(unexpectedError.bind(test)).then(() => test.done());
};
exports.testPropertySelectorWithImproperlyEscapedBrace = function(test)
{
let toHide = createElementWithStyle("{background-color: #000}");
applyElemHideEmulation(
- ["[-abp-properties='/background.\\x7B0,6\\x7D: rgb\\(0, 0, 0\\)/']"]
+ [":-abp-properties('/background.\\x7B0,6\\x7D: rgb\\(0, 0, 0\\)/')"]
).then(() =>
{
expectVisible(test, toHide);
}).catch(unexpectedError.bind(test)).then(() => test.done());
};
exports.testDynamicallyChangedProperty = function(test)
{
let toHide = createElementWithStyle("{}");
applyElemHideEmulation(
- ["[-abp-properties='background-color: rgb(0, 0, 0)']"]
+ [":-abp-properties('background-color: rgb(0, 0, 0)')"]
).then(() =>
{
expectVisible(test, toHide);
insertStyleRule("#" + toHide.id + " {background-color: #000}");
return new Promise((resolve, reject) =>
{
window.setTimeout(() =>
{
expectHidden(test, toHide);
resolve();
}, 0);
});
}).catch(unexpectedError.bind(test)).then(() => test.done());
};
+
+exports.testPseudoClassHasSelector = function(test)
+{
+ let toHide = createElementWithStyle("{}");
+ applyElemHideEmulation(
+ ["div:-abp-has(div)"]
+ ).then(() =>
+ {
+ expectVisible(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testPseudoClassHasSelectorWithPrefix = function(test)
+{
+ let parent = createElementWithStyle("{}");
+ let child = createElementWithStyle("{}", parent);
+ applyElemHideEmulation(
+ ["div:-abp-has(div)"]
+ ).then(() =>
+ {
+ expectHidden(test, parent);
+ expectVisible(test, child);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testPseudoClassHasSelectorWithSuffix = function(test)
+{
+ let parent = createElementWithStyle("{}");
+ let middle = createElementWithStyle("{}", parent);
+ let child = createElementWithStyle("{}", middle);
+ applyElemHideEmulation(
+ ["div:-abp-has(div) > div"]
+ ).then(() =>
+ {
+ expectVisible(test, parent);
+ expectHidden(test, middle);
+ expectHidden(test, child);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testPseudoClassHasSelectorWithSuffixSibling = function(test)
+{
+ let parent = createElementWithStyle("{}");
+ let middle = createElementWithStyle("{}", parent);
+ let toHide = createElementWithStyle("{}");
+ applyElemHideEmulation(
+ ["div:-abp-has(div) + div"]
+ ).then(() =>
+ {
+ expectVisible(test, parent);
+ expectVisible(test, middle);
+ expectHidden(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+exports.testPseudoClassHasSelectorWithSuffixSiblingChild = function(test)
+{
+ // <div>
+ // <div></div>
+ // <div>
+ // <div>to hide</div>
+ // </div>
+ // </div>
+ let parent = createElementWithStyle("{}");
+ let middle = createElementWithStyle("{}", parent);
+ let sibling = createElementWithStyle("{}");
+ let toHide = createElementWithStyle("{}", sibling);
+ applyElemHideEmulation(
+ ["div:-abp-has(div) + div > div"]
+ ).then(() =>
+ {
+ expectVisible(test, parent);
+ expectVisible(test, middle);
+ expectVisible(test, sibling);
+ expectHidden(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
+
+function runTestPseudoClassHasSelectorWithHasAndWithSuffixSibling(test, selector)
+{
+ document.body.innerHTML = `<div id="parent">
+ <div id="middle">
+ <div id="middle1"><div id="inside" class="inside"></div></div>
+ </div>
+ <div id="sibling">
+ <div id="tohide">to hide</div>
+ </div>
+ <div id="sibling2">
+ <div id="sibling21"><div id="sibling211" class="inside"></div></div>
+ </div>
+ </div>`;
+ let parent = document.getElementById("parent");
+ let middle = document.getElementById("middle");
+ let inside = document.getElementById("inside");
+ let sibling = document.getElementById("sibling");
+ let sibling2 = document.getElementById("sibling2");
+ let toHide = document.getElementById("tohide");
+
+ insertStyleRule(".inside {}");
+
+ applyElemHideEmulation(
+ [selector]
+ ).then(() =>
+ {
+ expectVisible(test, parent);
+ expectVisible(test, middle);
+ expectVisible(test, inside);
+ expectVisible(test, sibling);
+ expectVisible(test, sibling2);
+ expectHidden(test, toHide);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+}
+
+exports.testPseudoClassHasSelectorWithHasAndWithSuffixSibling = function(test)
+{
+ runTestPseudoClassHasSelectorWithHasAndWithSuffixSibling(test, "div:-abp-has(:-abp-has(div.inside)) + div > div");
+};
+
+exports.testPseudoClassHasSelectorWithHasAndWithSuffixSibling2 = function(test)
+{
+ runTestPseudoClassHasSelectorWithHasAndWithSuffixSibling(test, "div:-abp-has(:-abp-has(> div.inside)) + div > div");
+};
+
+exports.testPseudoClassHasSelectorWithPropSelector = function(test)
+{
+ let parent = createElementWithStyle("{}");
+ let child = createElementWithStyle("{background-color: #000}", parent);
+ applyElemHideEmulation(
+ ["div:-abp-has(:-abp-properties(\"background-color: rgb(0, 0, 0)\"))"]
+ ).then(() =>
+ {
+ expectVisible(test, child);
+ expectHidden(test, parent);
+ }).catch(unexpectedError.bind(test)).then(() => test.done());
+};
« lib/filterClasses.js ('K') | « lib/filterClasses.js ('k') | test/filterClasses.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld