Issue 6931 - Allow content filters for localhost

lib/filterClasses.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 1113 matching lines @@
   if (domains && /(^|,)~?(,|$)/.test(domains))
     return new InvalidFilter(text, "filter_invalid_domain");
 
   if (type == "@")
     return new ElemHideException(text, domains, body);
 
   if (type == "?" || type == "$")
   {
     // Element hiding emulation and snippet filters are inefficient so we need
     // to make sure that they're only applied if they specify active domains
-    if (!/,[^~][^,.]*\.[^,]/.test("," + domains))
+    if (!(/,[^~][^,.]*\.[^,]/.test("," + domains) ||
+          /,localhost,/.test("," + domains + ",")))

[Sebastian Noack, 2018/09/10 17:44:37]

It seems we don't need a regular expression here, but could use .includes().

[Manish Jethani, 2018/09/10 17:57:32]

Good point, done.

     {
       return new InvalidFilter(text, type == "?" ?
                                        "filter_elemhideemulation_nodomain" :
                                        "filter_snippet_nodomain");
     }
 
     if (type == "?")
       return new ElemHideEmulationFilter(text, domains, body);
 
     return new SnippetFilter(text, domains, body);
@@ skipping 102 matching lines @@
 
   /**
    * Script that should be executed
    * @type {string}
    */
   get script()
   {
     return this.body;
   }
 });