Issue 3143 - Filter elements with :-abp-has()

test/browser/elemHideEmulation.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2017 eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 14 matching lines @@
 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);
+    child.parentNode.removeChild(child);
 
   callback();
 };
 
 function unexpectedError(error)
 {
   console.error(error);
   this.ok(false, "Unexpected error: " + error);
 }
 
@@ skipping 24 matching lines @@
   if (styleElements.length)
     styleElement = styleElements[0];
   else
   {
     styleElement = document.createElement("style");
     document.head.appendChild(styleElement);
   }
   styleElement.sheet.insertRule(rule, styleElement.sheet.cssRules.length);
 }
 
-// insert a <div> with a unique id and and empty CSS rule
+// insert a <div> with a unique id and a CSS rule
 // for the the selector matching the id.
 function createElementWithStyle(styleBlock, parent)
 {
   let element = document.createElement("div");
   element.id = findUniqueId();
   if (!parent)
     document.body.appendChild(element);
   else
     parent.appendChild(element);
   insertStyleRule("#" + element.id + " " + styleBlock);
   return element;
 }
 
-// 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
+// Will ensure the class ElemHideEmulation is loaded.
+// NOTE: if it never loads, this will probably hang.
 function loadElemHideEmulation()
 {
   if (typeof ElemHideEmulation == "undefined")
   {
     return loadScript(myUrl + "/../../../lib/common.js").then(() =>
     {
       return loadScript(myUrl + "/../../../chrome/content/elemHideEmulation.js");
     }).then(() =>
     {
       return loadElemHideEmulation();
     });
   }
 
   return Promise.resolve();
 }
 
-// instantiate a ElemHideEmulation with @selectors.
+// Create a new ElemHideEmulation instance 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;}");
+      },
+      elems =>
+      {
+        for (let elem of elems)
+          elem.style.display = "none";
       }
     );
 
     elemHideEmulation.apply();
-    return Promise.resolve();
+    return Promise.resolve(elemHideEmulation);
   });
 }
 
+// internals testing
+
 exports.testParseSelectorContent = function(test)
 {
   loadElemHideEmulation().then(() =>
   {
-    let parsed = parseSelectorContent("> div) > div");
+    let parsed = parseSelectorContent(":-abp-has(> div) > div", 10);
+    test.equal(parsed.text, "> div");
+    test.equal(parsed.end, 15);
+
+    parsed = parseSelectorContent("> div) > div", 0);
     test.equal(parsed.text, "> div");
     test.equal(parsed.end, 5);
 
-    parsed = parseSelectorContent("\"> div\") > div");
-    test.equal(parsed.text, "\"> div\"");
-    test.equal(parsed.end, 7);
-    parsed = parseSelectorContent(" \"> div\" ) > div");
-    test.equal(parsed.text, " \"> div\" ");
-    test.equal(parsed.end, 9);
-
-    // simple quoted content
-    parsed = parseSelectorContent("\"> div\") > div", true);
-    test.equal(parsed.text, "> div");
-    test.equal(parsed.end, 7);
-    // quoted content with whitespace around.
-    parsed = parseSelectorContent(" \"> div\" ) > div", true);
-    test.equal(parsed.text, "> div");
-    test.equal(parsed.end, 9);
-
     // parens not closed.
-    parsed = parseSelectorContent("> div > div");
+    parsed = parseSelectorContent("> div > div", 0);
     test.equal(parsed, null);
   }).catch(unexpectedError.bind(test)).then(() => test.done());
 };
 
 exports.testParseSelector = function(test)
 {
   loadElemHideEmulation().then(() =>
   {
     let selectors = parseSelector("");
     test.equal(selectors.length, 0);
@@ skipping 12 matching lines @@
     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)')";
+    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)')";
+    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());
 };
 
@@ skipping 76 matching lines @@
 
     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.testNestedHasSelector = function(test)
-{
-  buildDom(document);
-
-  loadElemHideEmulation().then(() =>
-  {
-    let selector = new HasSelector(":-abp-has(> div.inside)");
-
-    test.ok(!selector._innerSelectors);
+
+    selector = new HasSelector(":-abp-has(> div.inside)");
+
+    test.ok(selector._innerSelectors);
   }).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());
 };
 
+// API testing
+
 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)')"]
+    ["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)')"]
+    ["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"]
+    [":-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"]
+    ["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();
@@ skipping 72 matching lines @@
     ["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)\"))"]
+    ["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());
 };
-