Bug 1374837 - Add some additional logging in case the Browser Toolbox fails to open;r=jryans draft
authorBrian Grinstead <bgrinstead@mozilla.com>
Wed, 21 Jun 2017 11:13:35 -0700
changeset 662482 f02a1818d98cd177c3ad18df8de05ca7a8fbaf3f
parent 662474 02c78441c03d5d004e57596a130856861a30188a
child 662483 b79ae85ac4651a5be3dd0d4506626c9c56cd8069
push id79095
push userbgrinstead@mozilla.com
push dateMon, 11 Sep 2017 18:34:30 +0000
reviewersjryans
bugs1374837
milestone57.0a1
Bug 1374837 - Add some additional logging in case the Browser Toolbox fails to open;r=jryans MozReview-Commit-ID: 8EYnjOcZFAU
devtools/client/framework/ToolboxProcess.jsm
--- a/devtools/client/framework/ToolboxProcess.jsm
+++ b/devtools/client/framework/ToolboxProcess.jsm
@@ -6,16 +6,17 @@
 
 "use strict";
 
 const { interfaces: Ci, utils: Cu, results: Cr } = Components;
 
 const DBG_XUL = "chrome://devtools/content/framework/toolbox-process-window.xul";
 const CHROME_DEBUGGER_PROFILE_NAME = "chrome_debugger_profile";
 
+const { console } = Cu.import("resource://gre/modules/Console.jsm", {});
 const { require, DevToolsLoader } = Cu.import("resource://devtools/shared/Loader.jsm", {});
 const { XPCOMUtils } = require("resource://gre/modules/XPCOMUtils.jsm");
 
 XPCOMUtils.defineLazyModuleGetter(this, "Subprocess", "resource://gre/modules/Subprocess.jsm");
 XPCOMUtils.defineLazyGetter(this, "Telemetry", function () {
   return require("devtools/client/shared/telemetry");
 });
 XPCOMUtils.defineLazyGetter(this, "EventEmitter", function () {
@@ -297,16 +298,18 @@ BrowserToolboxProcess.prototype = {
           data = await pipe.readString();
         }
       };
       dumpPipe(proc.stdout);
 
       proc.wait().then(() => this.close());
 
       return proc;
+    }, err => {
+      console.log(`Error loading Browser Toolbox: ${command} ${args.join(" ")}`, err);
     });
   },
 
   /**
    * Called upon receiving the connectionchange event from a debuggerServer.
    *
    * @param {String} what
    *        Type of connection change (can be either 'opened' or 'closed').