| OLD | NEW |
| 1 /* | 1 /* |
| 2 * This file is part of Adblock Plus <https://adblockplus.org/>, | 2 * This file is part of Adblock Plus <https://adblockplus.org/>, |
| 3 * Copyright (C) 2006-present eyeo GmbH | 3 * Copyright (C) 2006-present eyeo GmbH |
| 4 * | 4 * |
| 5 * Adblock Plus is free software: you can redistribute it and/or modify | 5 * Adblock Plus is free software: you can redistribute it and/or modify |
| 6 * it under the terms of the GNU General Public License version 3 as | 6 * it under the terms of the GNU General Public License version 3 as |
| 7 * published by the Free Software Foundation. | 7 * published by the Free Software Foundation. |
| 8 * | 8 * |
| 9 * Adblock Plus is distributed in the hope that it will be useful, | 9 * Adblock Plus is distributed in the hope that it will be useful, |
| 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of | 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of |
| (...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 142 { | 142 { |
| 143 for (let i = 0; i < FilterStorage.subscriptions.length; i++) | 143 for (let i = 0; i < FilterStorage.subscriptions.length; i++) |
| 144 { | 144 { |
| 145 if (FilterStorage.subscriptions[i].url == subscription.url) | 145 if (FilterStorage.subscriptions[i].url == subscription.url) |
| 146 { | 146 { |
| 147 removeSubscriptionFilters(subscription); | 147 removeSubscriptionFilters(subscription); |
| 148 | 148 |
| 149 FilterStorage.subscriptions.splice(i--, 1); | 149 FilterStorage.subscriptions.splice(i--, 1); |
| 150 FilterStorage.knownSubscriptions.delete(subscription.url); | 150 FilterStorage.knownSubscriptions.delete(subscription.url); |
| 151 FilterNotifier.triggerListeners("subscription.removed", subscription); | 151 FilterNotifier.triggerListeners("subscription.removed", subscription); |
| 152 |
| 153 // This should be the last remaining reference to the Subscription |
| 154 // object. |
| 155 Subscription.knownSubscriptions.delete(subscription.url); |
| 152 return; | 156 return; |
| 153 } | 157 } |
| 154 } | 158 } |
| 155 }, | 159 }, |
| 156 | 160 |
| 157 /** | 161 /** |
| 158 * Moves a subscription in the list to a new position. | 162 * Moves a subscription in the list to a new position. |
| 159 * @param {Subscription} subscription filter subscription to be moved | 163 * @param {Subscription} subscription filter subscription to be moved |
| 160 * @param {Subscription} [insertBefore] filter subscription to insert before | 164 * @param {Subscription} [insertBefore] filter subscription to insert before |
| 161 * (if omitted the subscription will be put at the end of the list) | 165 * (if omitted the subscription will be put at the end of the list) |
| (...skipping 611 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 773 else if (this.wantObj === false && val) | 777 else if (this.wantObj === false && val) |
| 774 this.curObj.push(val.replace(/\\\[/g, "[")); | 778 this.curObj.push(val.replace(/\\\[/g, "[")); |
| 775 } | 779 } |
| 776 finally | 780 finally |
| 777 { | 781 { |
| 778 Filter.knownFilters = origKnownFilters; | 782 Filter.knownFilters = origKnownFilters; |
| 779 Subscription.knownSubscriptions = origKnownSubscriptions; | 783 Subscription.knownSubscriptions = origKnownSubscriptions; |
| 780 } | 784 } |
| 781 } | 785 } |
| 782 }; | 786 }; |
| OLD | NEW |