Issue 3916 - Supporting Adding filter lists via URL

adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java

Index: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java
===================================================================
--- a/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java
+++ b/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java
@@ -215,16 +215,26 @@ final class Subscriptions
     if (!this.subscriptions.containsKey(id))
     {
       this.subscriptions.put(id, sub);
       return sub;
     }
     return this.subscriptions.get(id);
   }
 
+  public boolean remove(final String id)
+  {
+    if (this.subscriptions.containsKey(id))

[anton, 2017/08/25 10:40:40]

the whole method could be replaced with `return this.subscriptions.remove(id) !=
null;`, is it?
BTW is return value needed somewhere?

[jens, 2017/08/25 11:35:07]

Acknowledged.
The return values is not needed yet but I thought it makes sense to confirm if
the subscription was removed or not.

+    {
+      this.subscriptions.remove(id);
+      return true;
+    }
+    return false;
+  }
+
   public static Subscriptions initialize(final Engine engine, final File appFolder,
       final File cacheFolder)
   {
     final Subscriptions subs = new Subscriptions(engine, appFolder, cacheFolder);
 
     subs.subscriptionFolder.mkdirs();
     subs.cacheFolder.mkdirs();