Issue 4544 - Consider platformHTMLBindings.xml when detecting available keyboard shortcuts

templates/bootstrap.js.tmpl

 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 const Cc = Components.classes;
 const Ci = Components.interfaces;
 const Cr = Components.results;
 const Cu = Components.utils;
 
 let {Services} = Cu.import("resource://gre/modules/Services.jsm", {});
 
 Cu.importGlobalProperties(["atob", "btoa", "File", "URL",
-    "TextDecoder", "TextEncoder"]);
-
-{%- if hasXMLHttpRequest %}
-Cu.importGlobalProperties(["XMLHttpRequest"]);
-{%- endif %}
+    "TextDecoder", "TextEncoder", "XMLHttpRequest"]);

[Wladimir Palant, 2016/10/31 10:06:42]

For reference: originally there was no Cu.importGlobalProperties() and creating
the XMLHttpRequest global was non-trivial. Now the special handling for
XMLHttpRequest is completely pointless of course.

 
 let addonData = null;
 
 function startup(params, reason)
 {
   addonData = params;
 
   {%- if hasChromeRequires %}
   Services.obs.addObserver(RequireObserver, "{{metadata.get('general', 'basename')}}-require", true);
   onShutdown.add(function()
@@ skipping 110 matching lines @@
         applicationVersion: appInfo.version,
         platform: "gecko",
         platformVersion: appInfo.platformVersion
       };
     }
     else
     {
     {%- endif %}
       let url = addonData.resourceURI.spec + "lib/" + module + ".js";
       scopes[module] = {
-        Cc, Ci, Cr, Cu, atob, btoa, File, URL, require,
+        Cc, Ci, Cr, Cu, atob, btoa, File, URL, TextDecoder, TextEncoder,

[Wladimir Palant, 2016/10/31 10:06:42]

TextDecoder and TextEncoder globals weren't declared here properly - things were
still working because they "leaked" into the scope but that leakage is something
we want to fix soon.

+        XMLHttpRequest, require,
         {% if hasShutdownHandlers %}
         onShutdown,
         {% endif %}
-        {%- if hasXMLHttpRequest %}
-        XMLHttpRequest,
-        {% endif %}
         exports: {}};
       {%- if multicompartment %}
       let principal = Cc["@mozilla.org/systemprincipal;1"].getService(Ci.nsIPrincipal);
       scopes[module] = new Cu.Sandbox(principal, {
         sandboxName: url,
         sandboxPrototype: scopes[module],
         wantXrays: false
       });
       {%- endif %}
       Services.scriptloader.loadSubScript(url, scopes[module]);
@@ skipping 17 matching lines @@
   {
     if (topic == "{{metadata.get('general', 'basename')}}-require")
     {
       subject.wrappedJSObject.exports = require(data);
     }
   },
 
   QueryInterface: XPCOMUtils.generateQI([Ci.nsISupportsWeakReference, Ci.nsIObserver])
 };
 {%- endif %}