Issue 7096 - Make it possible to lazy initialize a filter's subscriptions

lib/filterStorage.js

Index: lib/filterStorage.js
===================================================================
--- a/lib/filterStorage.js
+++ b/lib/filterStorage.js
@@ -123,16 +123,31 @@
    * @type {number}
    */
   get subscriptionCount()
   {
     return this.knownSubscriptions.size;
   }
 
   /**
+   * Adds any subscriptions to which a filter belongs to the filter.
+   * @param {Filter} filter
+   */
+  addSubscriptionsToFilter(filter)
+  {
+    for (let subscription of this.subscriptions())
+    {
+      if (subscription.filters.indexOf(filter) != -1)

[hub, 2018/11/10 04:33:21]

this is quite an expensive search, isn't it?

[Manish Jethani, 2018/11/15 22:58:52]

Yes, so here's the thing:

 1.  This function will not be executed on startup, because the initial filter
objects will already have their subscriptions assigned in INIParser; once those
objects are discarded and then when a subscription is updated, it will be called
for any newly created filter objects

 2.  In
https://codereview.adblockplus.org/29934588/diff/29934599/lib/subscriptionCla...
I intend to add a Set object containing the filter text, so this will become a
set lookup and it will be faster (see searchFilter method)

[hub, 2018/11/16 17:04:39]

So shall we make sure the review 29934588 lands first?

[Manish Jethani, 2018/11/17 19:49:17]

OK, let's do this one first instead, it's easier:

https://codereview.adblockplus.org/29935564/

I'll prepare the other one.

+        filter.addSubscription(subscription);
+    }
+
+    filter.subscriptionsAssigned = true;
+  }
+
+  /**
    * Finds the filter group that a filter should be added to by default. Will
    * return <code>null</code> if this group doesn't exist yet.
    * @param {Filter} filter
    * @returns {?SpecialSubscription}
    */
   getGroupForFilter(filter)
   {
     let generalSubscription = null;