Provide an XMLHttpRequest constructor if used in JS modules

bootstrap.js.tmpl

 /*
  * This file is part of the Adblock Plus build tools,
  * Copyright (C) 2006-2013 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 const Cc = Components.classes;
 const Ci = Components.interfaces;
 const Cr = Components.results;
 const Cu = Components.utils;
 
 let {Services, atob, btoa, File} = Cu.import("resource://gre/modules/Services.jsm", null);
 
 {%- if hasXMLHttpRequest %}
-var XMLHttpRequest = Components.Constructor("@mozilla.org/xmlextras/xmlhttprequest;1", "nsIXMLHttpRequest");

[Felix Dahlke, 2013/07/25 13:24:11]

Why var and not let?

[Wladimir Palant, 2013/07/25 15:43:45]

Fixed

+let XMLHttpRequest = Components.Constructor("@mozilla.org/xmlextras/xmlhttprequest;1", "nsIXMLHttpRequest");
 {%- endif %}
 
 let addonData = null;
 
 function startup(params, reason)
 {
   addonData = params;
 
   {%- if hasChromeRequires %}
   Services.obs.addObserver(RequireObserver, "{{metadata.get('general', 'basename')}}-require", true);
@@ skipping 156 matching lines @@
   {
     if (topic == "{{metadata.get('general', 'basename')}}-require")
     {
       subject.wrappedJSObject.exports = require(data);
     }
   },
 
   QueryInterface: XPCOMUtils.generateQI([Ci.nsISupportsWeakReference, Ci.nsIObserver])
 };
 {%- endif %}