Unified Diff: options.js
Issue 29570575:
Issue 5587- Use dedicated mobile options page in Firefox Mobile (Closed)
Patch Set:
Created Oct. 9, 2017, 11:51 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: options.js |
| =================================================================== |
| --- a/options.js |
| +++ b/options.js |
| @@ -24,6 +24,11 @@ |
| document.title = iframe.contentDocument.title; |
| }; |
| +// In Edge the namespace for extension APIs is 'browser'. The 'chrome' object, |
| +// while available does not contain the runtime object needed below. |
| +if (typeof chrome == "undefined" || typeof chrome.runtime == "undefined") |
|
Manish Jethani
2017/10/09 12:03:52
LGTM
I would move it to the top of the file thoug
|
| + window.chrome = window.browser; |
| + |
| chrome.runtime.sendMessage({ |
| type: "app.get", |
| what: "application" |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
