Issue 3916 - Supporting Adding filter lists via URL

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

 /*
  * 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 25 matching lines @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.TreeSet;
 import java.util.concurrent.LinkedBlockingQueue;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.locks.ReentrantLock;
 
 import org.adblockplus.adblockplussbrowser.R;
-import org.adblockplus.sbrowser.contentblocker.MoreBlockingPreferenceCategory;
 import org.adblockplus.sbrowser.contentblocker.util.SharedPrefsUtils;
 import org.adblockplus.sbrowser.contentblocker.util.SubscriptionUtils;
 
 import android.content.Context;
 import android.content.Intent;
 import android.content.pm.PackageInfo;
 import android.content.pm.PackageManager;
 import android.content.pm.ResolveInfo;
 import android.net.ConnectivityManager;
 import android.net.NetworkInfo;
@@ skipping 193 matching lines @@
   }
 
   public void subscriptionStateChanged()
   {
     if (this.subscriptionUpdateCallback != null)
     {
       subscriptionUpdateCallback.subscriptionUpdatedApplied();
     }
   }
 
-  public void createAndAddSubscriptionFromUrl(final String url, final SubscriptionAddedCallback callback) throws IOException
+  public void createAndAddSubscriptionFromUrl(final String url,
+      final SubscriptionAddedCallback callback) throws IOException
   {
     final Subscription sub = Subscription.create(url);
     sub.putMeta(Subscription.KEY_TITLE, url);
     sub.setEnabled(true);
     subscriptions.add(sub);
     subscriptions.persistSubscription(sub);
     callback.subscriptionAdded();
   }
 
   public void removeSubscriptionById(final String subscriptionId)
@@ skipping 147 matching lines @@
     engine.wasFirstRun = engine.subscriptions.wasUnitialized();
     if (engine.subscriptions.wasUnitialized())
     {
       Log.d(TAG, "Subscription storage was uninitialized, initializing...");
 
       try (final InputStream easylistTxt = context.getResources().openRawResource(R.raw.easylist))
       {
         final Subscription easylist = engine.subscriptions.add(Subscription
             // Use bundled EasyList as default and update it with locale specific list later
             // see: https://issues.adblockplus.org/ticket/5237
-            .create(SubscriptionUtils.chooseDefaultSubscriptionUrl(engine.defaultSubscriptions.getAdsSubscriptions()))
+            .create(SubscriptionUtils.chooseDefaultSubscriptionUrl(
+                engine.defaultSubscriptions.getAdsSubscriptions()))
             .parseLines(readLines(easylistTxt)));
         easylist.putMeta(Subscription.KEY_UPDATE_TIMESTAMP, "0");
         easylist.setEnabled(true);
       }
       Log.d(TAG, "Added and enabled bundled easylist");
 
       try (final InputStream exceptionsTxt = context.getResources()
           .openRawResource(R.raw.exceptionrules))
       {
         final Subscription exceptions = engine.subscriptions.add(Subscription
@@ skipping 181 matching lines @@
       sb.append(downloadCount);
     }
     else
     {
       sb.append("4%2B"); // "4+" URL encoded
     }
 
     return new URL(sb.toString());
   }
 
-  public List<SubscriptionInfo> getMoreBlockingPreferenceSubscriptions()
-  {
-    List<SubscriptionInfo> moreBlockingPreferenceSubscriptions = new ArrayList<>(5);
-    for(SubscriptionInfo sub : getListedSubscriptions())

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

it seems that space is required after `for`

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

Acknowledged.

-    {
-      final DefaultSubscriptionInfo info = getDefaultSubscriptionInfoForUrl(sub.getUrl());
-      Integer resInt = MoreBlockingPreferenceCategory.URL_TO_RES_ID_MAP.get(sub.getUrl());
-
-      if (sub.getType() == SubscriptionInfo.Type.CUSTOM)
-      {
-        moreBlockingPreferenceSubscriptions.add(sub);
-      }
-
-      if (info != null && !info.isComplete() && sub.isEnabled())
-      {
-        moreBlockingPreferenceSubscriptions.add(sub);
-      }
-
-      if ((!(isAcceptableAdsUrl(sub)) || sub.getTitle().startsWith("__"))
-          && resInt != null
-          && (info == null || info.getPrefixes().isEmpty() || sub.getType() != SubscriptionInfo.Type.ADS))
-      {
-        moreBlockingPreferenceSubscriptions.add(sub);
-      }
-    }
-
-    return moreBlockingPreferenceSubscriptions;
-  }
-
-  private boolean isAcceptableAdsUrl(final SubscriptionInfo subscriptionInfo)
+  public boolean isAcceptableAdsUrl(final SubscriptionInfo subscriptionInfo)
   {
     return getPrefsDefault(SUBSCRIPTIONS_EXCEPTIONSURL).equals(subscriptionInfo.getUrl());
   }
 
   private static class EventHandler implements Runnable
   {
     private static final String TAG = EventHandler.class.getSimpleName();
     private final Engine engine;
 
     public EventHandler(final Engine engine)
@@ skipping 166 matching lines @@
   {
     void subscriptionUpdateRequested(boolean enabled);
     void subscriptionUpdatedApplied();
   }
 
   public interface SubscriptionAddedCallback
   {
     void subscriptionAdded();
   }
 }