Add filtersLoaded and filterLoadListener to AdblockPlusApi

adblockplus/Api.jsm

Index: adblockplus/Api.jsm
===================================================================
--- a/adblockplus/Api.jsm
+++ b/adblockplus/Api.jsm
@@ -32,12 +32,18 @@
   return result.exports;
 }
 
+let {FilterNotifier} = require("filterNotifier");
 let {FilterStorage} = require("filterStorage");
 let {Prefs} = require("prefs");
 let {UI} = require("ui");
 
 var AdblockPlusApi =
 {
+  get filtersLoaded()
+  {
+    return !FilterStorage._loading;
+  },
+  filterLoadListener: null,
   get acceptableAdsEnabled()
   {
     return FilterStorage.subscriptions.some(
@@ -49,3 +55,13 @@
       UI.toggleAcceptableAds();
   }
 };
+
+FilterNotifier.addListener(function filterListener(action)

[René Jeschke, 2015/03/22 21:01:46]

Oh, and from my side: we do not even really need this listener, anyway. I really
only need 'get filtersLoaded()' above.

[Felix Dahlke, 2015/03/22 21:13:28]

Right, we _can't_ use it there, actually - interaction between Java and JS is
worse than I thought.

In the spirit of not adding unused code, I'll close this review. René can add
the filtersLoaded getter above (as it is there) to his patch.

+{
+  if (action != "load")
+    return;
+
+  FilterNotifier.removeListener(filterListener);
+  if (typeof AdblockPlusApi.filterLoadListener == "function")
+    AdblockPlusApi.filterLoadListener();
+});