Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: ext/common.js

Issue 29458601: Issue 5315 - Add support for Microsoft Edge (Closed)
Patch Set: Created June 7, 2017, 12:41 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ext/common.js
===================================================================
--- a/ext/common.js
+++ b/ext/common.js
@@ -19,6 +19,11 @@
(function()
{
+ // Both Edge and Mozilla Web Extensions use
+ // the namespace 'browser' instead of 'chrome'
+ if (typeof browser != "undefined")
+ window.chrome = window.browser;
Sebastian Noack 2017/06/07 14:42:14 Firefox actually has the "chrome" namespace as wel
Oleksandr 2017/06/14 03:41:01 There is actually a `chrome` object defined in Edg
+
window.ext = {};
let EventTarget = ext._EventTarget = function()

Powered by Google App Engine
This is Rietveld