Issue 2376 - mockup causing error

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 157 matching lines @@
     {
       
       if (params.filterError)
         return {error: "Invalid filter"};
       return {filter: modules.filterClasses.Filter.fromText(text)};
     },
     parseFilters: function(text)
     {
       if (params.filterError)
         return {errors: ["Invalid filter"]};
-      return {filters: 
-              text.split("\n").map(modules.filterClasses.Filter.fromText), errors: []};

[Thomas Greiner, 2015/07/15 17:52:50]

Again, mind the 80 characters limit. I'd suggest using the following style:

return {
  filters: text.split("\n").map(modules.filterClasses.Filter.fromText),
  errors: []
};

[saroyanm, 2015/07/15 17:56:57]

Done.

+      return {
+        filters: text.split("\n").map(modules.filterClasses.Filter.fromText),
+        errors: []
+      };
     }
   };
 
   modules.synchronizer = {
     Synchronizer: {}
   };
 
   modules.matcher = {
     defaultMatcher: {
       matchesAny: function(url, requestType, docDomain, thirdParty)
@@ skipping 100 matching lines @@
         type: "message",
         payload: {
           title: "Custom subscription",
           url: "http://example.com/custom.txt",
           type: "add-subscription"
         }
       }, "*");
     }, 1000);
   }
 })(this);