Bug 1277674 - Explicitly require Services in source actor. r=tromey
authorJ. Ryan Stinnett <jryans@gmail.com>
Fri, 03 Jun 2016 13:38:23 -0500
changeset 339379 4900b0a4b6981d9cf0260974421847ecccf303c6
parent 339378 9e2fcb41e05a213689ed4447f705cd068af86005
child 339380 4168becc7a7adf0771df9fc0959efefe78cd6f67
push id6249
push userjlund@mozilla.com
push dateMon, 01 Aug 2016 13:59:36 +0000
treeherdermozilla-beta@bad9d4f5bf7e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstromey
bugs1277674, 1193390
milestone49.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1277674 - Explicitly require Services in source actor. r=tromey It was being implicitly loaded into modules by a single arg Cu.import elsewhere in the code base, but only if JSON Viewer is enabled (Dev. Ed and Nightly channels). I plan to address the Cu.import issue more thoroughly over in bug 1193390. MozReview-Commit-ID: 6AGJGqYvw3c
devtools/server/actors/source.js
--- a/devtools/server/actors/source.js
+++ b/devtools/server/actors/source.js
@@ -2,16 +2,17 @@
 /* vim: set ft=javascript ts=2 et sw=2 tw=80: */
 /* 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/. */
 
 "use strict";
 
 const { Cc, Ci } = require("chrome");
+const Services = require("Services");
 const { BreakpointActor, setBreakpointAtEntryPoints } = require("devtools/server/actors/breakpoint");
 const { OriginalLocation, GeneratedLocation } = require("devtools/server/actors/common");
 const { createValueGrip } = require("devtools/server/actors/object");
 const { ActorClass, Arg, RetVal, method } = require("devtools/shared/protocol");
 const DevToolsUtils = require("devtools/shared/DevToolsUtils");
 const { assert, fetch } = DevToolsUtils;
 const { joinURI } = require("devtools/shared/path");
 const promise = require("promise");