Bug 1374735 - stop exposing devtools require to extensions ext-* files;r=kmag draft
authorJulian Descottes <jdescottes@mozilla.com>
Thu, 22 Jun 2017 21:04:02 +0200
changeset 648853 7caebb839b6b454d5edafb2194e309587ccc019b
parent 648852 97a12fa049acf954cb5b28f7f7d91501efa53b26
child 726965 8857f664bffafb40b2d57a1406f6fb96173b218c
push id74909
push userjdescottes@mozilla.com
push dateFri, 18 Aug 2017 12:10:48 +0000
reviewerskmag
bugs1374735
milestone57.0a1
Bug 1374735 - stop exposing devtools require to extensions ext-* files;r=kmag There is no longer any file in components/extensions/ext-* that use require(). Therefore it should be ok to stop exposing it. MozReview-Commit-ID: EgZYBludlcy
toolkit/components/extensions/ExtensionCommon.jsm
--- a/toolkit/components/extensions/ExtensionCommon.jsm
+++ b/toolkit/components/extensions/ExtensionCommon.jsm
@@ -1255,17 +1255,16 @@ class SchemaAPIManager extends EventEmit
 
     Cu.import("resource://gre/modules/AppConstants.jsm", global);
 
     XPCOMUtils.defineLazyGetter(global, "console", getConsole);
 
     XPCOMUtils.defineLazyModuleGetters(global, {
       ExtensionUtils: "resource://gre/modules/ExtensionUtils.jsm",
       XPCOMUtils: "resource://gre/modules/XPCOMUtils.jsm",
-      require: "resource://devtools/shared/Loader.jsm",
     });
 
     return global;
   }
 
   /**
    * Load an ext-*.js script. The script runs in its own scope, if it wishes to
    * share state with another script it can assign to the `global` variable. If