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: Rebase to current tip Created July 27, 2017, 9:45 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
@@ -22,7 +22,7 @@
// Both Edge and Mozilla Web Extensions use the namespace
// 'browser' instead of 'chrome'. Edge has chrome namespace defined,
// in some cases, but only with one property: 'app'.
- if (typeof chrome == "undefined" || !chrome.extension)
+ if (typeof chrome == "undefined" || typeof chrome.extension == "undefined")
window.chrome = window.browser;
window.ext = {};

Powered by Google App Engine
This is Rietveld