Unified Diff: background.js
Issue 5838171610808320:
Added high resolution icons for Retina displays (Closed)
Patch Set: Addressed comment
Created April 3, 2014, 11:40 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
chrome/ext/background.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: background.js |
| =================================================================== |
| --- a/background.js |
| +++ b/background.js |
| @@ -117,13 +117,13 @@ |
| var whitelisted = isWhitelisted(tab.url); |
| var iconFilename; |
| - if (require("info").platform == "safari") |
| + if (whitelisted && require("info").platform != "safari") |
| // There is no grayscale version of the icon for whitelisted tabs |
| // when using Safari, because icons are grayscale already and icons |
| // aren't per tab in Safari. |
| - iconFilename = "icons/abp-16.png" |
| + iconFilename = "icons/abp-$size-whitelisted.png"; |
| else |
| - iconFilename = whitelisted ? "icons/abp-19-whitelisted.png" : "icons/abp-19.png"; |
| + iconFilename = "icons/abp-$size.png"; |
| tab.browserAction.setIcon(iconFilename); |
| iconAnimation.registerTab(tab, iconFilename); |
« no previous file with comments
|
« no previous file
|
chrome/ext/background.js » ('j')
|
no next file with comments »
