Unified Diff: lib/antiadblockInit.js
Issue 5501516697829376:
Issue 1161 - Anti-adblock notifications were triggered on similar domains (Closed)
Patch Set:
Created July 29, 2014, 3:41 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
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/antiadblockInit.js |
| =================================================================== |
| --- a/lib/antiadblockInit.js |
| +++ b/lib/antiadblockInit.js |
| @@ -46,10 +46,17 @@ |
| { |
| let urlFilters = []; |
| for (let filter of subscription.filters) |
| + { |
| if (filter instanceof ActiveFilter) |
| + { |
| for (let domain in filter.domains) |
| - if (domain && urlFilters.indexOf(domain) == -1) |
| - urlFilters.push(domain); |
| + { |
| + let urlFilter = "||" + domain + "^"; |
| + if (domain && filter.domains[domain] && urlFilters.indexOf(urlFilter) == -1) |
| + urlFilters.push(urlFilter); |
| + } |
| + } |
| + } |
| notification.urlFilters = urlFilters; |
| Notification.addNotification(notification); |
| Notification.addQuestionListener(notification.id, notificationListener); |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
