Issue 6959 - Use async/await in test/browser/elemHideEmulation.js

test/browser/elemHideEmulation.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present 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 34 matching lines @@
 };
 
 function timeout(delay)
 {
   return new Promise((resolve, reject) =>
   {
     window.setTimeout(resolve, delay);
   });
 }
 
-function unexpectedError(error)
+function unexpectedError(test, error)
 {
   console.error(error);
-  this.ok(false, "Unexpected error: " + error);
+  test.ok(false, "Unexpected error: " + error);
 }
 
 function expectHidden(test, element, id)
 {
   let withId = "";
   if (typeof id != "undefined")
     withId = ` with ID '${id}'`;
 
   test.equal(
     window.getComputedStyle(element).display, "none",
@@ skipping 72 matching lines @@
 // Insert a <div> with a unique id and a CSS rule
 // for the the selector matching the id.
 function createElementWithStyle(styleBlock, parent)
 {
   let element = createElement(parent);
   insertStyleRule("#" + element.id + " " + styleBlock);
   return element;
 }
 
 // Create a new ElemHideEmulation instance with @selectors.
-async function applyElemHideEmulation(selectors, test)
+async function applyElemHideEmulation(test, selectors)
 {
   await Promise.resolve();

[Manish Jethani, 2019/01/17 08:03:33]

This should not be necessary but I left it in for compatibility with the
previous version. We should be able to just create the ElemHideEmulation object
and apply the selectors synchronously.

 
-  let elemHideEmulation = null;
-
   try
   {
-    elemHideEmulation = new ElemHideEmulation(
+    let elemHideEmulation = new ElemHideEmulation(
       elems =>
       {
         for (let elem of elems)
           elem.style.display = "none";
       }
     );
 
     elemHideEmulation.document = testDocument;
     elemHideEmulation.MIN_INVOCATION_INTERVAL = REFRESH_INTERVAL / 2;
     elemHideEmulation.apply(selectors.map(
       selector => ({selector, text: selector})
     ));
+
+    return elemHideEmulation;
   }
   catch (error)
   {
-    if (!test)

[Manish Jethani, 2019/01/17 08:03:33]

This should not be necessary if every call to this function passes in the test
object.

[Manish Jethani, 2019/01/17 08:04:36]

Removed now.

-      throw error;
-
-    unexpectedError.call(test, error);
-    return null;
-  }
-
-  return elemHideEmulation;
+    unexpectedError(test, error);
+  }
 }
 
 exports.testVerbatimPropertySelector = async function(test)
 {
   let toHide = createElementWithStyle("{background-color: #000}");
-
-  if (await applyElemHideEmulation(
-        [":-abp-properties(background-color: rgb(0, 0, 0))"],
-        test
-      ))
-  {
+  let selectors = [":-abp-properties(background-color: rgb(0, 0, 0))"];
+
+  if (await applyElemHideEmulation(test, selectors))
     expectHidden(test, toHide);
-  }
 
   test.done();
 };
 
 exports.testVerbatimPropertySelectorWithPrefix = async function(test)
 {
   let parent = createElementWithStyle("{background-color: #000}");
   let toHide = createElementWithStyle("{background-color: #000}", parent);
 
-  if (await applyElemHideEmulation(
-        ["div > :-abp-properties(background-color: rgb(0, 0, 0))"],
-        test
-      ))
+  let selectors = ["div > :-abp-properties(background-color: rgb(0, 0, 0))"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectVisible(test, parent);
     expectHidden(test, toHide);
   }
 
   test.done();
 };
 
-exports.testVerbatimPropertySelectorWithPrefixNoMatch = function(test)
+exports.testVerbatimPropertySelectorWithPrefixNoMatch = async 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(() =>
+
+  let selectors = ["div > :-abp-properties(background-color: rgb(0, 0, 0))"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectVisible(test, parent);
     expectVisible(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testVerbatimPropertySelectorWithSuffix = function(test)
+  }
+
+  test.done();
+};
+
+exports.testVerbatimPropertySelectorWithSuffix = async 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(() =>
+
+  let selectors = [":-abp-properties(background-color: rgb(0, 0, 0)) > div"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectVisible(test, parent);
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testVerbatimPropertyPseudoSelectorWithPrefixAndSuffix = function(test)
+  }
+
+  test.done();
+};
+
+exports.testVerbatimPropertyPseudoSelectorWithPrefixAndSuffix = async 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(() =>
+
+  let selectors = ["div > :-abp-properties(background-color: rgb(0, 0, 0)) > div"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectVisible(test, parent);
     expectVisible(test, middle);
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
+  }
+
+  test.done();
 };
 
 // Add the style. Then add the element for that style.
 // This should retrigger the filtering and hide it.
-exports.testPropertyPseudoSelectorAddStyleAndElement = function(test)
+exports.testPropertyPseudoSelectorAddStyleAndElement = async function(test)
 {
   let styleElement;
   let toHide;
-  applyElemHideEmulation(
-    [":-abp-properties(background-color: rgb(0, 0, 0))"]
-  ).then(() =>
+
+  let selectors = [":-abp-properties(background-color: rgb(0, 0, 0))"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     styleElement = testDocument.createElement("style");
     testDocument.head.appendChild(styleElement);
     styleElement.sheet.insertRule("#toHide {background-color: #000}");
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     toHide = createElement();
     toHide.id = "toHide";
     expectVisible(test, toHide);
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPropertySelectorWithWildcard = function(test)
+  }
+
+  test.done();
+};
+
+exports.testPropertySelectorWithWildcard = async function(test)
 {
   let toHide = createElementWithStyle("{background-color: #000}");
-  applyElemHideEmulation(
-    [":-abp-properties(*color: rgb(0, 0, 0))"]
-  ).then(() =>
-  {
+  let selectors = [":-abp-properties(*color: rgb(0, 0, 0))"];
+
+  if (await applyElemHideEmulation(test, selectors))
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPropertySelectorWithRegularExpression = function(test)
+
+  test.done();
+};
+
+exports.testPropertySelectorWithRegularExpression = async function(test)
 {
   let toHide = createElementWithStyle("{background-color: #000}");
-  applyElemHideEmulation(
-    [":-abp-properties(/.*color: rgb\\(0, 0, 0\\)/)"]
-  ).then(() =>
-  {
+  let selectors = [":-abp-properties(/.*color: rgb\\(0, 0, 0\\)/)"];
+
+  if (await applyElemHideEmulation(test, selectors))
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPropertySelectorWithEscapedBrace = function(test)
+
+  test.done();
+};
+
+exports.testPropertySelectorWithEscapedBrace = async function(test)
 {
   let toHide = createElementWithStyle("{background-color: #000}");
-  applyElemHideEmulation(
-    [":-abp-properties(/background.\\7B 0,6\\7D : rgb\\(0, 0, 0\\)/)"]
-  ).then(() =>
-  {
+  let selectors = [":-abp-properties(/background.\\7B 0,6\\7D : rgb\\(0, 0, 0\\)/)"];
+
+  if (await applyElemHideEmulation(test, selectors))
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPropertySelectorWithImproperlyEscapedBrace = function(test)
+
+  test.done();
+};
+
+exports.testPropertySelectorWithImproperlyEscapedBrace = async function(test)
 {
   let toHide = createElementWithStyle("{background-color: #000}");
-  applyElemHideEmulation(
-    [":-abp-properties(/background.\\7B0,6\\7D: rgb\\(0, 0, 0\\)/)"]
-  ).then(() =>
-  {
+  let selectors = [":-abp-properties(/background.\\7B0,6\\7D: rgb\\(0, 0, 0\\)/)"];
+
+  if (await applyElemHideEmulation(test, selectors))
     expectVisible(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testDynamicallyChangedProperty = function(test)
+
+  test.done();
+};
+
+exports.testDynamicallyChangedProperty = async function(test)
 {
   let toHide = createElementWithStyle("{}");
-  applyElemHideEmulation(
-    [":-abp-properties(background-color: rgb(0, 0, 0))"]
-  ).then(() =>
+  let selectors = [":-abp-properties(background-color: rgb(0, 0, 0))"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectVisible(test, toHide);
     insertStyleRule("#" + toHide.id + " {background-color: #000}");
 
-    return timeout(0);
-  }).then(() =>
-  {
+    await timeout(0);
+
     // Re-evaluation will only happen after a delay
     expectVisible(test, toHide);
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassWithPropBeforeSelector = function(test)
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassWithPropBeforeSelector = async function(test)
 {
   let parent = createElementWithStyle("{}");
   let child = createElementWithStyle("{background-color: #000}", parent);
+
+  let selectors = ["div:-abp-properties(content: \"publicite\")"];
+
   insertStyleRule(`#${child.id}::before {content: "publicite"}`);
 
-  applyElemHideEmulation(
-    ["div:-abp-properties(content: \"publicite\")"]
-  ).then(() =>
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectHidden(test, child);
     expectVisible(test, parent);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassHasSelector = function(test)
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasSelector = async function(test)
 {
   let toHide = createElementWithStyle("{}");
-  applyElemHideEmulation(
-    ["div:-abp-has(div)"]
-  ).then(() =>
-  {
+
+  if (await applyElemHideEmulation(test, ["div:-abp-has(div)"]))
     expectVisible(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassHasSelectorWithPrefix = function(test)
+
+  test.done();
+};
+
+exports.testPseudoClassHasSelectorWithPrefix = async 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)
+
+  if (await applyElemHideEmulation(test, ["div:-abp-has(div)"]))
+  {
+    expectHidden(test, parent);
+    expectVisible(test, child);
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasSelectorWithSuffix = async function(test)
 {
   let parent = createElementWithStyle("{}");
   let middle = createElementWithStyle("{}", parent);
   let child = createElementWithStyle("{}", middle);
-  applyElemHideEmulation(
-    ["div:-abp-has(div) > div"]
-  ).then(() =>
+
+  if (await applyElemHideEmulation(test, ["div:-abp-has(div) > div"]))
   {
     expectVisible(test, parent);
     expectHidden(test, middle);
     expectHidden(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassHasSelectorWithSuffixSibling = function(test)
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasSelectorWithSuffixSibling = async function(test)
 {
   let parent = createElementWithStyle("{}");
   let middle = createElementWithStyle("{}", parent);
   let toHide = createElementWithStyle("{}");
-  applyElemHideEmulation(
-    ["div:-abp-has(div) + div"]
-  ).then(() =>
+
+  if (await applyElemHideEmulation(test, ["div:-abp-has(div) + div"]))
   {
     expectVisible(test, parent);
     expectVisible(test, middle);
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassHasSelectorWithSuffixSiblingChild = function(test)
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasSelectorWithSuffixSiblingChild = async 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(() =>
+
+  if (await applyElemHideEmulation(test, ["div:-abp-has(div) + div > div"]))
   {
     expectVisible(test, parent);
     expectVisible(test, middle);
     expectVisible(test, sibling);
     expectHidden(test, toHide);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
+  }
+
+  test.done();
 };
 
 function compareExpectations(test, elems, expectations)
 {
   for (let elem in expectations)
   {
     if (elems[elem])
     {
       if (expectations[elem])
         expectVisible(test, elems[elem], elem);
       else
         expectHidden(test, elems[elem], elem);
     }
   }
 }
 
-function runTestPseudoClassHasSelectorWithHasAndWithSuffixSibling(test, selector, expectations)
+async function runTestPseudoClassHasSelectorWithHasAndWithSuffixSibling(test, selector, expectations)
 {
   testDocument.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"><span>to hide</span></div>
       </div>
       <div id="sibling2">
         <div id="sibling21"><div id="sibling211" class="inside"></div></div>
       </div>
     </div>`;
   let elems = {
     parent: testDocument.getElementById("parent"),
     middle: testDocument.getElementById("middle"),
     inside: testDocument.getElementById("inside"),
     sibling: testDocument.getElementById("sibling"),
     sibling2: testDocument.getElementById("sibling2"),
     toHide: testDocument.getElementById("tohide")
   };
 
   insertStyleRule(".inside {}");
 
-  applyElemHideEmulation(
-    [selector]
-  ).then(() =>
-  {
+  if (await applyElemHideEmulation(test, [selector]))
     compareExpectations(test, elems, expectations);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
+
+  test.done();
 }
 
 exports.testPseudoClassHasSelectorWithHasAndWithSuffixSibling = function(test)
 {
   let expectations = {
     parent: true,
     middile: true,
     inside: true,
     sibling: true,
     sibling2: true,
@@ skipping 52 matching lines @@
     middile: true,
     inside: true,
     sibling: true,
     sibling2: true,
     toHide: true
   };
   runTestPseudoClassHasSelectorWithHasAndWithSuffixSibling(
     test, "div:-abp-has(> span:-abp-contains(Advertisment))", expectations);
 };
 
-function runTestPseudoClassContains(test, selector, expectations)
+async function runTestPseudoClassContains(test, selector, expectations)
 {
   testDocument.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 \ud83d\ude42!</div>
       </div>
       <div id="sibling2">
         <div id="sibling21"><div id="sibling211" class="inside">Ad*</div></div>
       </div>
     </div>`;
   let elems = {
     parent: testDocument.getElementById("parent"),
     middle: testDocument.getElementById("middle"),
     inside: testDocument.getElementById("inside"),
     sibling: testDocument.getElementById("sibling"),
     sibling2: testDocument.getElementById("sibling2"),
     toHide: testDocument.getElementById("tohide")
   };
 
-  applyElemHideEmulation(
-    [selector]
-  ).then(
-    () => compareExpectations(test, elems, expectations)
-  ).catch(unexpectedError.bind(test)).then(() => test.done());
+  if (await applyElemHideEmulation(test, [selector]))
+    compareExpectations(test, elems, expectations);
+
+  test.done();
 }
 
 exports.testPseudoClassContainsText = function(test)
 {
   let expectations = {
     parent: true,
     middle: true,
     inside: true,
     sibling: false,
     sibling2: true,
@@ skipping 67 matching lines @@
     middle: true,
     inside: true,
     sibling: true,
     sibling2: false,
     toHide: true
   };
   runTestPseudoClassContains(
     test, "#parent div:-abp-contains(Ad*)", expectations);
 };
 
-exports.testPseudoClassHasSelectorWithPropSelector = function(test)
+exports.testPseudoClassHasSelectorWithPropSelector = async 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());
-};
-
-exports.testPseudoClassHasSelectorWithPropSelector2 = function(test)
+
+  let selectors = ["div:-abp-has(:-abp-properties(background-color: rgb(0, 0, 0)))"];
+
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    expectVisible(test, child);
+    expectHidden(test, parent);
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasSelectorWithPropSelector2 = async function(test)
 {
   let parent = createElementWithStyle("{}");
   let child = createElementWithStyle("{}", parent);
+
+  let selectors = ["div:-abp-has(:-abp-properties(background-color: rgb(0, 0, 0)))"];
+
   insertStyleRule("body #" + parent.id + " > div { background-color: #000}");
-  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());
-};
-
-exports.testDomUpdatesStyle = function(test)
+
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    expectVisible(test, child);
+    expectHidden(test, parent);
+  }
+
+  test.done();
+};
+
+exports.testDomUpdatesStyle = async function(test)
 {
   let parent = createElementWithStyle("{}");
   let child = createElementWithStyle("{}", parent);
-  applyElemHideEmulation(
-    ["div:-abp-has(:-abp-properties(background-color: rgb(0, 0, 0)))"]
-  ).then(() =>
+
+  let selectors = ["div:-abp-has(:-abp-properties(background-color: rgb(0, 0, 0)))"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectVisible(test, child);
     expectVisible(test, parent);
 
     insertStyleRule("body #" + parent.id + " > div { background-color: #000}");
-    return timeout(0);
-  }).then(() =>
-  {
-    expectVisible(test, child);
-    expectVisible(test, parent);
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
-    expectVisible(test, child);
-    expectHidden(test, parent);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testDomUpdatesContent = function(test)
+    await timeout(0);
+
+    expectVisible(test, child);
+    expectVisible(test, parent);
+    await timeout(REFRESH_INTERVAL);
+
+    expectVisible(test, child);
+    expectHidden(test, parent);
+  }
+
+  test.done();
+};
+
+exports.testDomUpdatesContent = async function(test)
 {
   let parent = createElementWithStyle("{}");
   let child = createElementWithStyle("{}", parent);
-  applyElemHideEmulation(
-    ["div > div:-abp-contains(hide me)"]
-  ).then(() =>
+
+  if (await applyElemHideEmulation(test, ["div > div:-abp-contains(hide me)"]))
   {
     expectVisible(test, parent);
     expectVisible(test, child);
 
     child.textContent = "hide me";
-    return timeout(0);
-  }).then(() =>
-  {
-    expectVisible(test, parent);
-    expectVisible(test, child);
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(0);
+
+    expectVisible(test, parent);
+    expectVisible(test, child);
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
     expectHidden(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testDomUpdatesNewElement = function(test)
+  }
+
+  test.done();
+};
+
+exports.testDomUpdatesNewElement = async function(test)
 {
   let parent = createElementWithStyle("{}");
   let child = createElementWithStyle("{ background-color: #000}", parent);
   let sibling;
   let child2;
-  applyElemHideEmulation(
-    ["div:-abp-has(:-abp-properties(background-color: rgb(0, 0, 0)))"]
-  ).then(() =>
+
+  let selectors = ["div:-abp-has(:-abp-properties(background-color: rgb(0, 0, 0)))"];
+
+  if (await applyElemHideEmulation(test, selectors))
   {
     expectHidden(test, parent);
     expectVisible(test, child);
 
     sibling = createElementWithStyle("{}");
-    return timeout(0);
-  }).then(() =>
-  {
+    await timeout(0);
+
     expectHidden(test, parent);
     expectVisible(test, child);
     expectVisible(test, sibling);
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     expectHidden(test, parent);
     expectVisible(test, child);
     expectVisible(test, sibling);
 
     child2 = createElementWithStyle("{ background-color: #000}",
                                     sibling);
-    return timeout(0);
-  }).then(() =>
-  {
+    await timeout(0);
+
     expectVisible(test, child2);
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     expectHidden(test, parent);
     expectVisible(test, child);
     expectHidden(test, sibling);
     expectVisible(test, child2);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassPropertiesOnStyleSheetLoad = function(test)
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassPropertiesOnStyleSheetLoad = async function(test)
 {
   let parent = createElement();
   let child = createElement(parent);
-  applyElemHideEmulation(
-    ["div:-abp-properties(background-color: rgb(0, 0, 0))",
-     "div:-abp-contains(hide me)",
-     "div:-abp-has(> div.hideMe)"]
-  ).then(() => timeout(REFRESH_INTERVAL)
-  ).then(() =>
-  {
+
+  let selectors = [
+    "div:-abp-properties(background-color: rgb(0, 0, 0))",
+    "div:-abp-contains(hide me)",
+    "div:-abp-has(> div.hideMe)"
+  ];
+
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
     expectVisible(test, child);
 
     // Load a style sheet that targets the parent element. This should run only
     // the "div:-abp-properties(background-color: rgb(0, 0, 0))" pattern.
     insertStyleRule("#" + parent.id + " {background-color: #000}");
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
-    expectHidden(test, parent);
-    expectVisible(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPlainAttributeOnDomMutation = function(test)
+    await timeout(REFRESH_INTERVAL);
+
+    expectHidden(test, parent);
+    expectVisible(test, child);
+  }
+
+  test.done();
+};
+
+exports.testPlainAttributeOnDomMutation = async function(test)
 {
   let parent = createElement();
   let child = createElement(parent);
-  applyElemHideEmulation(
-    ["div:-abp-properties(background-color: rgb(0, 0, 0))",
-     "div[data-hide-me]",
-     "div:-abp-contains(hide me)",
-     "div:-abp-has(> div.hideMe)"]
-  ).then(() => timeout(REFRESH_INTERVAL)
-  ).then(() =>
-  {
+
+  let selectors = [
+    "div:-abp-properties(background-color: rgb(0, 0, 0))",
+    "div[data-hide-me]",
+    "div:-abp-contains(hide me)",
+    "div:-abp-has(> div.hideMe)"
+  ];
+
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
     expectVisible(test, child);
 
     // Set the "data-hide-me" attribute on the child element.
     //
     // Note: Since the "div[data-hide-me]" pattern has already been processed
     // and the selector added to the document's style sheet, this will in fact
     // not do anything at our end, but the browser will just match the selector
     // and hide the element.
     child.setAttribute("data-hide-me", "");
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
     expectHidden(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassContainsOnDomMutation = function(test)
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassContainsOnDomMutation = async function(test)
 {
   let parent = createElement();
   let child = createElement(parent);
 
+  let selectors = [
+    "div:-abp-properties(background-color: rgb(0, 0, 0))",
+    "div[data-hide-me]",
+    "div:-abp-contains(hide me)",
+    "div:-abp-has(> div.hideMe)"
+  ];
+
   child.innerText = "do nothing";
 
-  applyElemHideEmulation(
-    ["div:-abp-properties(background-color: rgb(0, 0, 0))",
-     "div[data-hide-me]",
-     "div:-abp-contains(hide me)",
-     "div:-abp-has(> div.hideMe)"]
-  ).then(() => timeout(REFRESH_INTERVAL)
-  ).then(() =>
-  {
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
     expectVisible(test, child);
 
     // Set the child element's text to "hide me". This should run only the
     // "div:-abp-contains(hide me)" pattern.
     //
     // Note: We need to set Node.innerText here in order to trigger the
     // "characterData" DOM mutation on Chromium. If we set Node.textContent
     // instead, it triggers the "childList" DOM mutation instead.
     child.innerText = "hide me";
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
-    expectHidden(test, parent);
-    expectVisible(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassHasOnDomMutation = function(test)
+    await timeout(REFRESH_INTERVAL);
+
+    expectHidden(test, parent);
+    expectVisible(test, child);
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasOnDomMutation = async function(test)
 {
   let parent = createElement();
   let child = null;
-  applyElemHideEmulation(
-    ["div:-abp-properties(background-color: rgb(0, 0, 0))",
-     "div[data-hide-me]",
-     "div:-abp-contains(hide me)",
-     "div:-abp-has(> div)"]
-  ).then(() => timeout(REFRESH_INTERVAL)
-  ).then(() =>
-  {
+
+  let selectors = [
+    "div:-abp-properties(background-color: rgb(0, 0, 0))",
+    "div[data-hide-me]",
+    "div:-abp-contains(hide me)",
+    "div:-abp-has(> div)"
+  ];
+
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
 
     // Add the child element. This should run all the DOM-dependent patterns
     // ("div:-abp-contains(hide me)" and "div:-abp-has(> div)").
     child = createElement(parent);
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
-    expectHidden(test, parent);
-    expectVisible(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassHasWithClassOnDomMutation = function(test)
+    await timeout(REFRESH_INTERVAL);
+
+    expectHidden(test, parent);
+    expectVisible(test, child);
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasWithClassOnDomMutation = async function(test)
 {
   let parent = createElement();
   let child = createElement(parent);
-  applyElemHideEmulation(
-    ["div:-abp-properties(background-color: rgb(0, 0, 0))",
-     "div[data-hide-me]",
-     "div:-abp-contains(hide me)",
-     "div:-abp-has(> div.hideMe)"]
-  ).then(() => timeout(REFRESH_INTERVAL)
-  ).then(() =>
-  {
+
+  let selectors = [
+    "div:-abp-properties(background-color: rgb(0, 0, 0))",
+    "div[data-hide-me]",
+    "div:-abp-contains(hide me)",
+    "div:-abp-has(> div.hideMe)"
+  ];
+
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
     expectVisible(test, child);
 
     // Set the child element's class to "hideMe". This should run only the
     // "div:-abp-has(> div.hideMe)" pattern.
     child.className = "hideMe";
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
-    expectHidden(test, parent);
-    expectVisible(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testPseudoClassHasWithPseudoClassContainsOnDomMutation = function(test)
+    await timeout(REFRESH_INTERVAL);
+
+    expectHidden(test, parent);
+    expectVisible(test, child);
+  }
+
+  test.done();
+};
+
+exports.testPseudoClassHasWithPseudoClassContainsOnDomMutation = async function(test)
 {
   let parent = createElement();
   let child = createElement(parent);
 
+  let selectors = [
+    "div:-abp-properties(background-color: rgb(0, 0, 0))",
+    "div[data-hide-me]",
+    "div:-abp-contains(hide me)",
+    "div:-abp-has(> div:-abp-contains(hide me))"
+  ];
+
   child.innerText = "do nothing";
 
-  applyElemHideEmulation(
-    ["div:-abp-properties(background-color: rgb(0, 0, 0))",
-     "div[data-hide-me]",
-     "div:-abp-contains(hide me)",
-     "div:-abp-has(> div:-abp-contains(hide me))"]
-  ).then(() => timeout(REFRESH_INTERVAL)
-  ).then(() =>
-  {
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    await timeout(REFRESH_INTERVAL);
+
     expectVisible(test, parent);
     expectVisible(test, child);
 
     // Set the child element's text to "hide me". This should run only the
     // "div:-abp-contains(hide me)" and
     // "div:-abp-has(> div:-abp-contains(hide me))" patterns.
     child.innerText = "hide me";
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     // Note: Even though it runs both the :-abp-contains() patterns, it only
     // hides the parent element because of revision d7d51d29aa34.
     expectHidden(test, parent);
     expectVisible(test, child);
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
-
-exports.testOnlyRelevantElementsProcessed = function(test)
+  }
+
+  test.done();
+};
+
+exports.testOnlyRelevantElementsProcessed = async function(test)
 {
   // <body>
   //   <div id="n1">
   //     <p id="n1_1"></p>
   //     <p id="n1_2"></p>
   //     <p id="n1_4">Hello</p>
   //   </div>
   //   <div id="n2">
   //     <p id="n2_1"></p>
   //     <p id="n2_2"></p>
@@ skipping 10 matching lines @@
   //     <p id="n4_4">Hello</p>
   //   </div>
   // </body>
   for (let i of [1, 2, 3, 4])
   {
     let n = createElement(null, "div", `n${i}`);
     for (let [j, text] of [[1], [2], [4, "Hello"]])
       createElement(n, "p", `n${i}_${j}`, text);
   }
 
-  applyElemHideEmulation(
-    ["p:-abp-contains(Hello)",
-     "div:-abp-contains(Try me!)",
-     "div:-abp-has(p:-abp-contains(This is good))"]
-  ).then(() => timeout(REFRESH_INTERVAL)
-  ).then(() =>
-  {
+  let selectors = [
+    "p:-abp-contains(Hello)",
+    "div:-abp-contains(Try me!)",
+    "div:-abp-has(p:-abp-contains(This is good))"
+  ];
+
+  if (await applyElemHideEmulation(test, selectors))
+  {
+    await timeout(REFRESH_INTERVAL);
+
     // This is only a sanity check to make sure everything else is working
     // before we do the actual test.
     for (let i of [1, 2, 3, 4])
     {
       for (let j of [1, 2, 4])
       {
         let id = `n${i}_${j}`;
         if (j == 4)
           expectHidden(test, testDocument.getElementById(id), id);
         else
           expectVisible(test, testDocument.getElementById(id), id);
       }
     }
 
     // All <div> and <p> elements should be processed initially.
     for (let element of [...testDocument.getElementsByTagName("div"),
                          ...testDocument.getElementsByTagName("p")])
     {
       expectProcessed(test, element, element.id);
     }
 
     // Modify the text in <p id="n4_1">
     testDocument.getElementById("n4_1").innerText = "Try me!";
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     // When an element's text is modified, only the element or one of its
     // ancestors matching any selector is processed for :-abp-has() and
     // :-abp-contains()
     for (let element of [...testDocument.getElementsByTagName("div"),
                          ...testDocument.getElementsByTagName("p")])
     {
       if (element.id == "n4" || element.id == "n4_1")
         expectProcessed(test, element, element.id);
       else
         expectNotProcessed(test, element, element.id);
     }
 
     // Create a new <p id="n2_3"> element with no text.
     createElement(testDocument.getElementById("n2"), "p", "n2_3");
 
-    return timeout(REFRESH_INTERVAL);
-  }).then(() =>
-  {
+    await timeout(REFRESH_INTERVAL);
+
     // When a new element is added, only the element or one of its ancestors
     // matching any selector is processed for :-abp-has() and :-abp-contains()
     for (let element of [...testDocument.getElementsByTagName("div"),
                          ...testDocument.getElementsByTagName("p")])
     {
       if (element.id == "n2" || element.id == "n2_3")
         expectProcessed(test, element, element.id);
       else
         expectNotProcessed(test, element, element.id);
     }
-  }).catch(unexpectedError.bind(test)).then(() => test.done());
-};
+  }
+
+  test.done();
+};