Issue 3829 - Merge add-subscriptions and subscriptions.add messages

messageResponder.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 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 114 matching lines @@
         if (!listenerFilters)
         {
           listenerFilters = Object.create(null);
           changeListeners.set(sender.page, listenerFilters);
         }
         break;
     }
 
     switch (message.type)
     {
-      case "add-subscription":
-        ext.showOptions(function()
-        {
-          var subscription = Subscription.fromURL(message.url);
-          subscription.title = message.title;
-          onFilterChange("addSubscription", subscription);
-        });
-        break;
       case "app.get":
         if (message.what == "issues")
         {
           var subscriptionInit;
           try
           {
             subscriptionInit = require("subscriptionInit");
           }
           catch (e)
           {
@@ skipping 139 matching lines @@
 
         if (!subscription)
           FilterStorage.removeFilter(filter);
         else
           FilterStorage.removeFilter(filter, subscription, message.index);
         break;
       case "prefs.get":
         callback(Prefs[message.key]);
         break;
       case "subscriptions.add":
-        if (message.url in FilterStorage.knownSubscriptions)
-          return;
-
         var subscription = Subscription.fromURL(message.url);
-        if (!subscription)
-          return;
-
-        subscription.disabled = false;
         if ("title" in message)
           subscription.title = message.title;
         if ("homepage" in message)
           subscription.homepage = message.homepage;
-        FilterStorage.addSubscription(subscription);
 
-        if (subscription instanceof DownloadableSubscription && !subscription.lastDownload)
-          Synchronizer.execute(subscription);
+        if (message.confirm)
+        {
+          ext.showOptions(function()
+          {
+            onFilterChange("addSubscription", subscription);
+          });
+        }
+        else
+        {
+          subscription.disabled = false;
+          FilterStorage.addSubscription(subscription);
+
+          if (subscription instanceof DownloadableSubscription && !subscription.lastDownload)
+            Synchronizer.execute(subscription);
+        }
         break;
       case "subscriptions.get":
         var subscriptions = FilterStorage.subscriptions.filter(function(s)
         {
           if (message.ignoreDisabled && s.disabled)
             return false;
           if (s instanceof DownloadableSubscription && message.downloadable)
             return true;
           if (s instanceof SpecialSubscription && message.special)
             return true;
@@ skipping 44 matching lines @@
           if (subscription instanceof DownloadableSubscription)
             Synchronizer.execute(subscription, true);
         }
         break;
       case "subscriptions.isDownloading":
         callback(Synchronizer.isExecuting(message.url));
         break;
     }
   });
 })(this);