Issue 5315 - Add support for Microsoft Edge

ext/popup.js

 "use strict";
 
 (function()
 {
   if (typeof chrome == "undefined" || typeof chrome.extension == "undefined")
     window.chrome = browser;
   const backgroundPage = chrome.extension.getBackgroundPage();
   window.ext = Object.create(backgroundPage.ext);
 
   window.ext.closePopup = () =>
   {
     window.close();
   };
 
-  // Edge needs the i18n to be local, otherwise it can't process substitutions

[kzar, 2017/07/17 14:08:19]

How about this? "Calling i18n from the background page causes Edge to throw an
exception. LINK"

[Oleksandr, 2017/07/17 23:52:09]

Rephrased to be more precise. Calling i18n alone does not throw... 

[kzar, 2017/07/18 09:30:49]

"Calling i18n.getMessage from the background page causes Edge to throw an
exception."?

[kzar, 2017/07/18 10:36:23]

It seems like your latest improvement to this comment isn't included in the
patch set?

+  // Calling i18n.getMessage from the background page causes Edge to throw an
+  // exception.
   // https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/12793975/
   window.ext.i18n = chrome.i18n;
 
   // We have to override ext.backgroundPage, because in order
   // to send messages the local "chrome" namespace must be used.
   window.ext.backgroundPage = {
     sendMessage: chrome.runtime.sendMessage,
 
     getWindow()
     {
       return backgroundPage;
     }
   };
 }());