Unified Diff: lib/stats.js
Issue 29530598:
Issue 5588 - Explicitly clear badge text on tab update (Closed)
Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Add link to Firefox bug
Created Aug. 30, 2017, 10:23 a.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/stats.js |
| =================================================================== |
| --- a/lib/stats.js |
| +++ b/lib/stats.js |
| @@ -29,16 +29,24 @@ |
| /** |
| * Gets the number of requests blocked on the given page. |
| * |
| * @param {Page} page |
| * @return {Number} |
| */ |
| exports.getBlockedPerPage = page => blockedPerPage.get(page) || 0; |
| +// Chrome automatically clears the browser action badge text when the URL of |
| +// the tab is updated, but Firefox doesn't. |
| +// https://bugzilla.mozilla.org/show_bug.cgi?id=1395074 |
| +ext.pages.onLoading.addListener(page => |
| +{ |
| + page.browserAction.setBadge(); |
| +}); |
| + |
| FilterNotifier.on("filter.hitCount", (filter, newValue, oldValue, page) => |
| { |
| if (!(filter instanceof BlockingFilter) || !page) |
| return; |
| Prefs.blocked_total++; |
| let blocked = blockedPerPage.get(page) || 0; |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
