Bug 1387128: Drop all promise = require("promise") and replace promise by Promise. r=tromey draft
authorAlexandre Poirot <poirot.alex@gmail.com>
Wed, 09 Aug 2017 12:11:48 +0200
changeset 643194 ed5bdf9a30d3b05ab608818c01ce4e411bb94240
parent 643193 b660322b1bb24deff01f27e5ae0e24381befe1f5
child 643195 7f928a8411327cc2f95592438ab0ec7f327f504b
push id73009
push userbmo:poirot.alex@gmail.com
push dateWed, 09 Aug 2017 10:25:45 +0000
reviewerstromey
bugs1387128
milestone57.0a1
Bug 1387128: Drop all promise = require("promise") and replace promise by Promise. r=tromey Part4: also remove lazy require for "promise" $ sed -i '/lazyRequireGetter(this, "promise")/d' $(egrep -rl 'lazyRequireGetter\(this, "promise"\)' devtools) MozReview-Commit-ID: 2ikcfUW9lIu
devtools/client/animationinspector/animation-controller.js
devtools/server/performance/recorder.js
devtools/shared/system.js
--- a/devtools/client/animationinspector/animation-controller.js
+++ b/devtools/client/animationinspector/animation-controller.js
@@ -12,17 +12,16 @@
 
 "use strict";
 
 var { classes: Cc, interfaces: Ci, utils: Cu, results: Cr } = Components;
 
 var { loader, require } = Cu.import("resource://devtools/shared/Loader.jsm", {});
 var { Task } = require("devtools/shared/task");
 
-loader.lazyRequireGetter(this, "promise");
 loader.lazyRequireGetter(this, "EventEmitter", "devtools/shared/event-emitter");
 loader.lazyRequireGetter(this, "AnimationsFront", "devtools/shared/fronts/animation", true);
 
 const { LocalizationHelper } = require("devtools/shared/l10n");
 const L10N =
       new LocalizationHelper("devtools/client/locales/animationinspector.properties");
 
 // Global toolbox/inspector, set when startup is called.
--- a/devtools/server/performance/recorder.js
+++ b/devtools/server/performance/recorder.js
@@ -2,17 +2,16 @@
  * 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/. */
 "use strict";
 
 const { Cu } = require("chrome");
 const { Task } = require("devtools/shared/task");
 
 loader.lazyRequireGetter(this, "Services");
-loader.lazyRequireGetter(this, "promise");
 loader.lazyRequireGetter(this, "Class",
   "sdk/core/heritage", true);
 loader.lazyRequireGetter(this, "EventTarget",
   "sdk/event/target", true);
 loader.lazyRequireGetter(this, "events",
   "sdk/event/core");
 
 loader.lazyRequireGetter(this, "Memory",
--- a/devtools/shared/system.js
+++ b/devtools/shared/system.js
@@ -2,17 +2,16 @@
  * 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/. */
 "use strict";
 
 const { Cc, Ci } = require("chrome");
 const { Task } = require("devtools/shared/task");
 
 loader.lazyRequireGetter(this, "Services");
-loader.lazyRequireGetter(this, "promise");
 loader.lazyRequireGetter(this, "defer", "devtools/shared/defer");
 loader.lazyRequireGetter(this, "OS", "resource://gre/modules/commonjs/node/os.js");
 loader.lazyRequireGetter(this, "DebuggerServer", "devtools/server/main", true);
 loader.lazyRequireGetter(this, "AppConstants",
   "resource://gre/modules/AppConstants.jsm", true);
 loader.lazyGetter(this, "screenManager", () => {
   return Cc["@mozilla.org/gfx/screenmanager;1"].getService(Ci.nsIScreenManager);
 });