Issue 6193 - Allow element hiding emulation exceptions

lib/filterValidation.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
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * 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/>.
  */
 
 /** @module filterValidation */
 
 "use strict";
 
-const {Filter, InvalidFilter, ElemHideBase, ElemHideEmulationFilter} =
-      require("filterClasses");
+const {Filter, InvalidFilter, ElemHideBase, ElemHideException,
+       ElemHideEmulationFilter} = require("filterClasses");
 
 /**
  * An error returned by
  * {@link module:filterValidation.parseFilter parseFilter()} or
  * {@link module:filterValidation.parseFilters parseFilters()}
  * indicating that a given filter cannot be parsed,
  * contains an invalid CSS selector or is a filter list header.
  *
  * @param {string} type See documentation in the constructor below.
  * @param {Object} [details] Contains the "reason" and / or "selector"
@@ skipping 69 matching lines @@
     document.querySelector(selector);
     sheet.insertRule(selector + "{}", 0);
   }
   catch (e)
   {
     return false;
   }
   return true;
 }
 
+function isValidFilterSelector(filter)
+{
+  // Only ElemHideBase has selectors.
+  if (!(filter instanceof ElemHideBase))
+    return true;
+
+  // We don't check the syntax of ElemHideEmulationFilter yet.
+  if (filter instanceof ElemHideEmulationFilter)
+    return true;
+
+  // If it is an ElemHideException, and it has an extended CSS
+  // selector we don't validate and assume it is valid.
+  if (filter instanceof ElemHideException &&
+      filter.selector.includes(":-abp-"))
+  {
+    return true;
+  }
+
+  return isValidCSSSelector(filter.selector);
+}
+
 /**
  * @typedef ParsedFilter
  * @property {?Filter} [filter]
  *   The parsed filter if it is valid. Or null if the given string is empty.
  * @property {FilterParsingError} [error]
  *   See {@link module:filterValidation~FilterParsingError FilterParsingError}
  */
 
 let parseFilter =
 /**
@@ skipping 12 matching lines @@
     if (text[0] == "[")
       return {error: new FilterParsingError("unexpected-filter-list-header")};
 
     filter = Filter.fromText(text);
 
     if (filter instanceof InvalidFilter)
     {
       return {error: new FilterParsingError("invalid-filter",
                                             {reason: filter.reason})};
     }
-    if (filter instanceof ElemHideBase &&
-        !(filter instanceof ElemHideEmulationFilter) &&
-        !isValidCSSSelector(filter.selector))
+    if (!isValidFilterSelector(filter))
     {
       return {error: new FilterParsingError("invalid-css-selector",
                                             {selector: filter.selector})};
     }
   }
 
   return {filter};
 };
 
 /**
@@ skipping 25 matching lines @@
 
     if (error)
     {
       error.lineno = i + 1;
       errors.push(error);
     }
   }
 
   return {filters, errors};
 };