From 9d1738c70bc057c14f96b3731836c9601c78534e Mon Sep 17 00:00:00 2001 From: Nick Fitzgerald Date: Sun, 6 Oct 2013 11:45:07 -0700 Subject: [PATCH] Bug 923600 - don't pretty print RDP packets when devtools.debugger.log is false; r=robcee --- toolkit/devtools/server/transport.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/toolkit/devtools/server/transport.js b/toolkit/devtools/server/transport.js index 3eddfa2dd2cf..34d1736c3d7c 100644 --- a/toolkit/devtools/server/transport.js +++ b/toolkit/devtools/server/transport.js @@ -7,6 +7,8 @@ "use strict"; Components.utils.import("resource://gre/modules/NetUtil.jsm"); +let wantLogging = Services.prefs.getBoolPref("devtools.debugger.log"); + /** * An adapter that handles data transfers between the debugger client and * server. It can work with both nsIPipe and nsIServerSocket transports so @@ -66,8 +68,9 @@ DebuggerTransport.prototype = { * they are passed to this method. */ send: function DT_send(aPacket) { - // TODO (bug 709088): remove pretty printing when the protocol is done. - let data = JSON.stringify(aPacket, null, 2); + let data = wantLogging + ? JSON.stringify(aPacket, null, 2) + : JSON.stringify(aPacket); data = this._converter.ConvertFromUnicode(data); data = data.length + ':' + data; this._outgoing += data;