Issue 5053 - Release JsValues explicitly

libadblockplus-android-webview/src/org/adblockplus/libadblockplus/android/webview/AdblockWebView.java

Index: libadblockplus-android-webview/src/org/adblockplus/libadblockplus/android/webview/AdblockWebView.java
diff --git a/libadblockplus-android-webview/src/org/adblockplus/libadblockplus/android/webview/AdblockWebView.java b/libadblockplus-android-webview/src/org/adblockplus/libadblockplus/android/webview/AdblockWebView.java
index cac1ec3163b97f07a5ab7faa7614e3b8740f1d82..0b434f89ff10e3e8538ae1edcdecc54f4869c0be 100644
--- a/libadblockplus-android-webview/src/org/adblockplus/libadblockplus/android/webview/AdblockWebView.java
+++ b/libadblockplus-android-webview/src/org/adblockplus/libadblockplus/android/webview/AdblockWebView.java
@@ -990,12 +990,22 @@ public class AdblockWebView extends WebView
             };
 
           List<Subscription> subscriptions = adblockEngine.getFilterEngine().getListedSubscriptions();
-          d("Listed subscriptions: " + subscriptions.size());
-          if (debugMode)
+          try
+          {
+            d("Listed subscriptions: " + subscriptions.size());
+            if (debugMode)

[diegocarloslima, 2017/03/30 14:19:29]

Isn't this check redundant here, since d() already checks for debugMode?

[anton, 2017/03/30 14:34:15]

Acknowledged. Bu it's scope of scope of this task/review.
Create separate issue for it if necessary.

+            {
+              for (Subscription eachSubscription : subscriptions)
+              {
+                d("Subscribed to " + eachSubscription);
+              }
+            }
+          }
+          finally
           {
             for (Subscription eachSubscription : subscriptions)
             {
-              d("Subscribed to " + eachSubscription);
+              eachSubscription.dispose();
             }
           }