Unified Diff: lib/filterStorage.js
Issue 29853574:
Issue 6855 - Release all references to Subscription object once removed (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set:
Created Aug. 11, 2018, 3:21 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
lib/subscriptionClasses.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/filterStorage.js |
| =================================================================== |
| --- a/lib/filterStorage.js |
| +++ b/lib/filterStorage.js |
| @@ -144,16 +144,18 @@ |
| { |
| if (FilterStorage.subscriptions[i].url == subscription.url) |
| { |
| removeSubscriptionFilters(subscription); |
| FilterStorage.subscriptions.splice(i--, 1); |
| FilterStorage.knownSubscriptions.delete(subscription.url); |
| FilterNotifier.triggerListeners("subscription.removed", subscription); |
| + |
| + subscription.free(); |
|
Manish Jethani
2018/08/11 15:28:17
At this point all the other references to this obj
|
| return; |
| } |
| } |
| }, |
| /** |
| * Moves a subscription in the list to a new position. |
| * @param {Subscription} subscription filter subscription to be moved |
« no previous file with comments
|
« no previous file
|
lib/subscriptionClasses.js » ('j')
|
no next file with comments »
