Unified Diff: ext/background.js
Issue 29995555:
Issue 7253 - Pre-render icons for badge on Chromium (Closed)
Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set:
Created Feb. 1, 2019, 2:28 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/icon.js » ('j')
|
lib/icon.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: ext/background.js |
| =================================================================== |
| --- a/ext/background.js |
| +++ b/ext/background.js |
| @@ -325,25 +325,40 @@ |
| { |
| // Firefox for Android displays the browser action not as an icon but |
| // as a menu item. There is no icon, but such an option may be added |
| // in the future. |
| // https://bugzilla.mozilla.org/show_bug.cgi?id=1331746 |
| if (change == "iconPath" && "setIcon" in browser.browserAction) |
| { |
| let path = { |
| - 16: this._changes.iconPath.replace("$size", "16"), |
| - 19: this._changes.iconPath.replace("$size", "19"), |
| - 20: this._changes.iconPath.replace("$size", "20"), |
| - 32: this._changes.iconPath.replace("$size", "32"), |
| - 38: this._changes.iconPath.replace("$size", "38"), |
| - 40: this._changes.iconPath.replace("$size", "40") |
| + 16: this._changes.iconPath.path.replace("$size", "16"), |
| + 19: this._changes.iconPath.path.replace("$size", "19"), |
| + 20: this._changes.iconPath.path.replace("$size", "20"), |
| + 32: this._changes.iconPath.path.replace("$size", "32"), |
| + 38: this._changes.iconPath.path.replace("$size", "38"), |
| + 40: this._changes.iconPath.path.replace("$size", "40") |
| }; |
| try |
| { |
| + // Try ImageData if available. |
| + if (this._changes.iconPath.imageData) |
| + { |
| + try |
| + { |
| + return browser.browserAction.setIcon({ |
| + tabId: this._tabId, |
| + imageData: this._changes.iconPath.imageData |
| + }); |
| + } |
| + catch (e) |
| + { |
| + } |
| + } |
| + |
| return browser.browserAction.setIcon({tabId: this._tabId, path}); |
| } |
| catch (e) |
| { |
| // Edge throws if passed icon sizes different than 19,20,38,40px. |
| delete path[16]; |
| delete path[32]; |
| return browser.browserAction.setIcon({tabId: this._tabId, path}); |
| @@ -394,19 +409,19 @@ |
| { |
| // If the tab is prerendered, browser.browserAction.set* fails |
| // and we have to delay our changes until the currently visible tab |
| // is replaced with the prerendered tab. |
| browser.tabs.onReplaced.addListener(onReplaced); |
| }); |
| } |
| }, |
| - setIcon(path) |
| + setIcon(path, imageData = null) |
| { |
| - this._addChange("iconPath", path); |
| + this._addChange("iconPath", {path, imageData}); |
| }, |
| setBadge(badge) |
| { |
| if (!badge) |
| { |
| this._addChange("badgeText", ""); |
| } |
| else |
