Issue 6848 - Add searchSelector parameter to hide-if-contains snippet

lib/content/snippets.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 79 matching lines @@
  *
  * @returns {function} The generated injector.
  */
 function makeInjector(injectable, ...dependencies)
 {
   return (...args) => injectCode(stringifyFunctionCall(injectable, ...args),
                                  dependencies);
 }
 
 /**
- * Hides an HTML element by settings its <code>style</code> attribute to
+ * Hides an HTML element by setting its <code>style</code> attribute to
  * <code>display: none !important</code>.
  *
  * @param {HTMLElement} element The HTML element to hide.
  */
 function hideElement(element)
 {
   element.style.setProperty("display", "none", "important");
 
   // Listen for changes to the style property and if our values are unset
   // then reset them.
@@ skipping 99 matching lines @@
     // Since it's a weak map, it's possible that either the element or its
     // shadow has been removed.
     if (!host || !root)
       return;
 
     // If the shadow contains the given text, check if the host or one of its
     // ancestors matches the selector; if a matching element is found, hide
     // it.
     if (root.textContent.includes(search))
     {
-      let element = host;
-
-      do
-      {
-        if (element.matches(selector))
-        {
-          hideElement(element);
-          break;
-        }
-      }
-      while (element = element.parentElement);
+      let closest = host.closest(selector);
+      if (closest)
+        hideElement(closest);
     }
   }
 
   Object.defineProperty(Element.prototype, "attachShadow", {
     value(...args)
     {
       // Create the shadow root first. It doesn't matter if it's a closed
       // shadow root, we keep the reference in a weak map.
       let root = originalAttachShadow.apply(this, args);
 
@@ skipping 13 matching lines @@
 
       return root;
     }
   });
 }
 
 exports["hide-if-shadow-contains"] = makeInjector(hideIfShadowContains,
                                                   hideElement);
 
 /**
- * Hides any HTML element if the text content of the element contains a given
- * string.
+ * Hides any HTML element or one of its ancestors matching a CSS selector if
+ * the text content of the element contains a given string.
  *
- * @param {string} search The string to look for in every HTML element.
+ * @param {string} search The string to look for in HTML elements.
  * @param {string} selector The CSS selector that an HTML element must match
  *   for it to be hidden.
+ * @param {string?} [searchSelector] The CSS selector that an HTML element
+ *   containing the given string must match. Defaults to the value of the
+ *   <code>selector</code> argument.
  */
-function hideIfContains(search, selector = "*")
+function hideIfContains(search, selector = "*", searchSelector = null)
 {
+  if (searchSelector == null)
+    searchSelector = selector;
+
   new MutationObserver(() =>
   {
-    for (let element of document.querySelectorAll(selector))
+    for (let element of document.querySelectorAll(searchSelector))
     {
       if (element.textContent.includes(search))
-        hideElement(element);
+      {
+        let closest = element.closest(selector);
+        if (closest)
+          hideElement(closest);
+      }
     }
   })
   .observe(document, {childList: true, characterData: true, subtree: true});
 }
 
 exports["hide-if-contains"] = hideIfContains;
 
 /**
  * Readds to the document any removed HTML elements that match a CSS selector.
  *
@@ skipping 17 matching lines @@
           // We don't have the location of the element in its former parent,
           // but it's usually OK to just add it at the end.
           mutation.target.appendChild(node);
         }
       }
     }
   });
 }
 
 exports.readd = readd;