Issue 3835 - Update dependency on adblockpluscore to revision 2d4888611240

chrome/content/ui/filters-subscriptionview.js

Index: chrome/content/ui/filters-subscriptionview.js
===================================================================
--- a/chrome/content/ui/filters-subscriptionview.js
+++ b/chrome/content/ui/filters-subscriptionview.js
@@ -319,9 +319,24 @@ ListManager.allowAcceptableAds = functio
     FilterStorage.addSubscription(subscription);
     if (subscription instanceof DownloadableSubscription && !subscription.lastDownload)
       Synchronizer.execute(subscription);
   }
   else
     FilterStorage.removeSubscription(subscription);
 };
 
+/**
+ * Checks whether Adblock Plus needs to be upgraded in order to support filters
+ * in a particular subscription.
+ */
+ListManager.isUpgradeRequired = function(/**Subscription*/ subscription)

[Thomas Greiner, 2016/03/18 16:43:22]

Where is this being used? In filters.xul (line 249) it's still checking
`hidden="{!subscription.upgradeRequired}"`.

[Wladimir Palant, 2016/03/19 18:59:13]

Ouch, not sure what happened here - part of the patch got lost. Fixed.

+{
+  if (subscription instanceof DownloadableSubscription && subscription.requiredVersion)
+  {
+    let {addonVersion} = require("info");
+    if (Services.vc.compare(subscription.requiredVersion, addonVersion) > 0)
+      return true;
+  }
+  return false;
+};
+
 window.addEventListener("load", ListManager.init, false);