Unified Diff: chrome/ext/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
|
« background.js ('k')
|
chrome/icons/abp-38.png » ('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: chrome/ext/background.js |
| =================================================================== |
| --- a/chrome/ext/background.js |
| +++ b/chrome/ext/background.js |
| @@ -206,7 +206,14 @@ |
| BrowserAction.prototype = { |
| setIcon: function(path) |
| { |
| - chrome.browserAction.setIcon({tabId: this._tabId, path: path}); |
| + var paths = {}; |
| + for (var i = 1; i <= 2; i++) |
| + { |
| + var size = i * 19; |
| + paths[size] = path.replace("$size", size); |
| + } |
| + |
| + chrome.browserAction.setIcon({tabId: this._tabId, path: paths}); |
| }, |
| setBadge: function(badge) |
| { |
« no previous file with comments
|
« background.js ('k')
|
chrome/icons/abp-38.png » ('j')
|
no next file with comments »
