Issue 2376 - Implement custom filters in new options page

background.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 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 13 matching lines @@
       var params = window.location.search.substr(1).split("&");
       for (var i = 0; i < params.length; i++)
       {
         var parts = params[i].split("=", 2);
         if (parts.length == 2 && parts[0] in data)
           data[parts[0]] = decodeURIComponent(parts[1]);
       }
     }
   }
 
+  var params = {
+    blockedURLs: "",
+    seenDataCorruption: false,
+    filterlistsReinitialized: false,
+    addSubscription: false,
+    filterError: false
+  };  
+  updateFromURL(params);
+
   var modules = {};
   global.require = function(module)
   {
     return modules[module];
   };
 
   modules.utils = {
     Utils: {
       getDocLink: function(link)
       {
@@ skipping 95 matching lines @@
       }
     }
   };
 
   modules.filterClasses = {
     BlockingFilter: function() {},
     Filter: function(text)
     {
       this.text = text;
       this.disabled = false;
-    },
-    WhitelistFilter: function(text) 
-    {
-      modules.filterClasses.Filter.call(this, text);
     }
   };
   modules.filterClasses.Filter.fromText = function(text)
   {
-    if (text.indexOf("@@") == 0)
-      return new modules.filterClasses.WhitelistFilter(text);
     return new modules.filterClasses.Filter(text);
   };
 
   modules.filterValidation = 
   {
-    FilterParsingError: function(type, details)
-    {
-      this.type = type;
-      if (details)
-      {
-        if ("reason" in details)
-          this.reason = details.reason;
-        if ("selector" in details)
-          this.selector = details.selector;
-      }

[Thomas Greiner, 2015/07/13 15:54:21]

You're not using either `FilterParsingError.reason` or
`FilterParsingError.selector` so no need to add them. You need, however,
`FilterParsingError.toString()` which translates the error type into an error
message.

[saroyanm, 2015/07/14 10:20:58]

Done.

-    },
     parseFilter: function(text) 
     {
-      if (text)
-      {
-        if (text[0] == "[")
-          return {error: new modules.filterValidation.FilterParsingError("unexpected-filter-list-header")};
-      }
+      
+      if (params.filterError)
+        return {error: "Invalid filter"};
       return {filter: modules.filterClasses.Filter.fromText(text)};
     },
     parseFilters: function(text)
     {
-      var lines = text.split("\n");
-      var filters = [];
-      var errors = [];
-
-      for (var i = 0; i < lines.length; i++)
-      {
-        var parseResult = modules.filterValidation.parseFilter(lines[i]);
-        if (parseResult.error)
-          errors.push(parseResult.error);
-        else
-          filters.push(parseResult.filter);
-      }
-
-      return {filters: filters, errors: errors};
+      if (params.filterError)
+        return {errors: ["Invalid filter"]};
+      return {filters: 
+              text.split("\n").map(modules.filterClasses.Filter.fromText)};
     }
   };
 
   modules.synchronizer = {
     Synchronizer: {}
   };
 
   modules.matcher = {
     defaultMatcher: {
       matchesAny: function(url, requestType, docDomain, thirdParty)
       {
-        var params = {blockedURLs: ""};
-        updateFromURL(params);
         var blocked = params.blockedURLs.split(",");
         if (blocked.indexOf(url) >= 0)
           return new modules.filterClasses.BlockingFilter();
         else
           return null;
       }
     }
   };
 
   var notifierListeners = [];
@@ skipping 69 matching lines @@
     "https://easylist-downloads.adblockplus.org/easylistgermany+easylist.txt",
     "https://easylist-downloads.adblockplus.org/exceptionrules.txt",
     "https://easylist-downloads.adblockplus.org/fanboy-social.txt",
     "~user~786254"
   ];
   var knownSubscriptions = Object.create(null);
   for (var subscriptionUrl of subscriptions)
     knownSubscriptions[subscriptionUrl] = modules.subscriptionClasses.Subscription.fromURL(subscriptionUrl);
   var customSubscription = knownSubscriptions["~user~786254"];
 
-  var issues = {seenDataCorruption: false, filterlistsReinitialized: false};
-  updateFromURL(issues);
-  global.seenDataCorruption = issues.seenDataCorruption;
-  global.filterlistsReinitialized = issues.filterlistsReinitialized;
+  global.seenDataCorruption = params.seenDataCorruption;
+  global.filterlistsReinitialized = params.filterlistsReinitialized;
   
-  var events = {addSubscription: false};
-  updateFromURL(events);
-  if (events.addSubscription)
+  if (params.addSubscription)
   {
     // We don't know how long it will take for the page to fully load
     // so we'll post the message after one second
     setTimeout(function()
     {
       window.postMessage({
         type: "message",
         payload: {
           title: "Custom subscription",
           url: "http://example.com/custom.txt",
           type: "add-subscription"
         }
       }, "*");
     }, 1000);
   }
 })(this);