Unified Diff: background.js
Issue 6000668259123200:
Remove ext.browserAction.show/hide (Closed)
Patch Set:
Created Dec. 17, 2013, 3:31 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
chrome/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 |
| @@ -82,7 +82,6 @@ |
| localStorage[opt] = val; |
| } |
| - defaultOptionValue("shouldShowIcon", "true"); |
| defaultOptionValue("shouldShowBlockElementMenu", "true"); |
| removeDeprecatedOptions(); |
| @@ -134,11 +133,6 @@ |
| iconAnimation.registerTab(tab, iconFilename); |
| - if (localStorage.shouldShowIcon == "false") |
| - tab.browserAction.hide(); |
| - else |
| - tab.browserAction.show(); |
| - |
| if (require("info").platform == "chromium") // TODO: Implement context menus for Safari |
| // Set context menu status according to whether current tab has whitelisted domain |
| if (excluded) |
« no previous file with comments
|
« no previous file
|
chrome/background.js » ('j')
|
no next file with comments »
