Unified Diff: ext/common.js
Issue 29458601:
Issue 5315 - Add support for Microsoft Edge (Closed)
Patch Set: Handle webRequest.ResourceType (#5321) and RTCPeerConnection.generateCertificate, address comments
Created June 15, 2017, 11:18 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« ext/background.js ('k')
|
ext/popup.js » ('j')
|
inject.preload.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: ext/common.js |
| =================================================================== |
| --- a/ext/common.js |
| +++ b/ext/common.js |
| @@ -19,6 +19,13 @@ |
| (function() |
| { |
| + // Both Edge and Mozilla Web Extensions use the namespace |
| + // 'browser' instead of 'chrome'. Edge has chrome namespace defined, |
| + // but only with one property: 'app'. |
| + if ((typeof chrome == "undefined") || |
|
Oleksandr
2017/06/15 11:21:32
'chrome' is undefined in content script, but in ba
Sebastian Noack
2017/06/19 11:07:02
Nit: The inner parenthesis are redundant.
Oleksandr
2017/06/21 23:38:35
Done.
|
| + (typeof chrome.extension == "undefined")) |
| + window.chrome = window.browser; |
| + |
| window.ext = {}; |
| let EventTarget = ext._EventTarget = function() |
« no previous file with comments
|
« ext/background.js ('k')
|
ext/popup.js » ('j')
|
inject.preload.js » ('J')
