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

Unified Diff: options.js

Issue 29570575: Issue 5587- Use dedicated mobile options page in Firefox Mobile (Closed)
Patch Set: Move to the top of file. Change the commit message Created Oct. 9, 2017, 3:12 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: options.js
===================================================================
--- a/options.js
+++ b/options.js
@@ -17,6 +17,11 @@
"use strict";
+// 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")
+ window.chrome = window.browser;
Sebastian Noack 2017/10/09 15:29:27 This fallback is duplicated in ext/common.js. Howe
+
let iframe = document.getElementById("content");
iframe.onload = () =>
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld