From 376339dd820bcafb5ab20628f61b70b1d1da335b Mon Sep 17 00:00:00 2001
From: Panagiotis Astithas <pastith@gmail.com>
Date: Fri, 11 May 2012 19:04:43 +0300
Subject: [PATCH 1/3] Use the new extension API from bug 753401:

https://bugzilla.mozilla.org/show_bug.cgi?id=753401
---
 content/httpmonitor/remote/remoteProxy.js     |  1 +
 content/httpmonitor/server/netMonitorActor.js |  2 +-
 content/httpmonitor/server/traceActor.js      | 12 +-----------
 3 files changed, 3 insertions(+), 12 deletions(-)

diff --git a/content/httpmonitor/remote/remoteProxy.js b/content/httpmonitor/remote/remoteProxy.js
index cfd2378..7e150e4 100644
--- a/content/httpmonitor/remote/remoteProxy.js
+++ b/content/httpmonitor/remote/remoteProxy.js
@@ -28,6 +28,7 @@ RemoteProxy.prototype = Obj.extend(Proxy,
         this.protocol.getTabList(function(packet)
         {
             var result = [];
+            var networkMonitorActor = packet.networkMonitorActor;
             var tabs = packet.tabs;
             for (var i=0; i<tabs.length; ++i)
             {
diff --git a/content/httpmonitor/server/netMonitorActor.js b/content/httpmonitor/server/netMonitorActor.js
index d283db6..3feb549 100644
--- a/content/httpmonitor/server/netMonitorActor.js
+++ b/content/httpmonitor/server/netMonitorActor.js
@@ -217,7 +217,7 @@ function networkMonitorActorHandler(tab, request)
     return actor.grip();
 }
 
-DebuggerServer.addTabRequest("networkMonitorActor", networkMonitorActorHandler);
+DebuggerServer.addTabActor("networkMonitorActor", NetworkMonitorActor);
 
 // ********************************************************************************************* //
 });
diff --git a/content/httpmonitor/server/traceActor.js b/content/httpmonitor/server/traceActor.js
index 0ed46ec..f90acf2 100644
--- a/content/httpmonitor/server/traceActor.js
+++ b/content/httpmonitor/server/traceActor.js
@@ -140,17 +140,7 @@ function traceActorHandler(rootActor, request)
     return actor.grip();
 }
 
-// xxxHonza: I believe this should be part of dbg-browser-actor.js
-if (typeof(DebuggerServer.addRequest) == "undefined")
-{
-    DebuggerServer.addRequest = function DS_addRequest(aName, aFunction) {
-      DebuggerServer.BrowserRootActor.prototype.requestTypes[aName] = function(aRequest) {
-        return aFunction(this, aRequest);
-      }
-    };
-}
-
-DebuggerServer.addRequest("traceActor", traceActorHandler);
+DebuggerServer.addGlobalActor("traceActor", TraceActor);
 
 // ********************************************************************************************* //
 });

From 7be67d9b70d15b1865edd5df0b047f13e6f6b033 Mon Sep 17 00:00:00 2001
From: Panagiotis Astithas <pastith@gmail.com>
Date: Mon, 6 Aug 2012 21:45:44 +0300
Subject: [PATCH 2/3] Update to the latest debugger server API changes (bug
 753401).

---
 content/httpmonitor/remote/connection.js      |  2 +-
 content/httpmonitor/remote/protocol.js        | 14 +----
 content/httpmonitor/remote/remoteProxy.js     |  5 +-
 content/httpmonitor/server/httpServer.js      | 55 ++++++++++++-------
 content/httpmonitor/server/netMonitorActor.js | 25 +--------
 content/httpmonitor/server/traceActor.js      | 18 +-----
 6 files changed, 45 insertions(+), 74 deletions(-)

diff --git a/content/httpmonitor/remote/connection.js b/content/httpmonitor/remote/connection.js
index f856c47..eb7ccbd 100644
--- a/content/httpmonitor/remote/connection.js
+++ b/content/httpmonitor/remote/connection.js
@@ -13,7 +13,7 @@ var Cu = Components.utils;
 
 try
 {
-    Cu["import"]("resource:///modules/devtools/dbg-client.jsm");
+    Cu["import"]("resource://gre/modules/devtools/dbg-client.jsm");
 }
 catch (err)
 {
diff --git a/content/httpmonitor/remote/protocol.js b/content/httpmonitor/remote/protocol.js
index d02a499..2be4f39 100644
--- a/content/httpmonitor/remote/protocol.js
+++ b/content/httpmonitor/remote/protocol.js
@@ -95,11 +95,6 @@ Protocol.prototype =
         return this.currentTab ? this.currentTab.actor : null;
     },
 
-    getNetActor: function(tabActor, callback)
-    {
-        this.connection.sendPacket(tabActor, "networkMonitorActor", true, callback);
-    },
-
     subscribe: function(netActor, callback)
     {
         if (this.currentNetActor)
@@ -139,13 +134,10 @@ Protocol.prototype =
 
         // A tab has been selected so, subscribe to the Net monitor actor. The callback
         // will receive events about any HTTP traffic within the target tab.
-        this.getNetActor(tabActor.id, function(packet)
-        {
-            self.subscribe(packet.actor, netEvent);
+        self.subscribe(tabActor.networkMonitorActor, netEvent);
 
-            // Also register callback for 'tabNavigated' events.
-            self.connection.addCallback(tabActor.id, tabNavigated, false);
-        });
+        // Also register callback for 'tabNavigated' events.
+        self.connection.addCallback(tabActor.id, tabNavigated, false);
     },
 
     onTabUnselected: function(tabActor)
diff --git a/content/httpmonitor/remote/remoteProxy.js b/content/httpmonitor/remote/remoteProxy.js
index d4a302e..5968c7a 100644
--- a/content/httpmonitor/remote/remoteProxy.js
+++ b/content/httpmonitor/remote/remoteProxy.js
@@ -25,10 +25,11 @@ RemoteProxy.prototype = Obj.extend(Proxy,
         if (FBTrace.DBG_REMOTENETMONITOR)
             FBTrace.sysout("remotenet; RemoteProxy.getTabs()");
 
+        var self = this;
         this.protocol.getTabList(function(packet)
         {
             var result = [];
-            var networkMonitorActor = packet.networkMonitorActor;
+            self.protocol.traceActor = packet.traceActor;
             var tabs = packet.tabs;
             for (var i=0; i<tabs.length; ++i)
             {
@@ -36,9 +37,9 @@ RemoteProxy.prototype = Obj.extend(Proxy,
                 result.push({
                     id: tab.actor,
                     label: tab.title ? tab.title : tab.url,
+                    networkMonitorActor: tab.networkMonitorActor
                 })
             }
-
             callback(result);
         });
     },
diff --git a/content/httpmonitor/server/httpServer.js b/content/httpmonitor/server/httpServer.js
index a8c6bd8..c5e4765 100644
--- a/content/httpmonitor/server/httpServer.js
+++ b/content/httpmonitor/server/httpServer.js
@@ -37,39 +37,30 @@ var HttpServer =
         TabCacheModel.initialize();
         TabCacheModel.initializeUI();
 
+        // TODO: this initialization is happening way too soon. The server
+        // should be started when the user launches the tool from the UI.
         try
         {
-            Components.utils.import("resource:///modules/devtools/dbg-server.jsm");
+            Components.utils.import("resource://gre/modules/devtools/dbg-server.jsm");
 
             // RemoteDebugger object should exist on Fennec so, use it if the server
             // is running on Fennec
             // https://bugzilla.mozilla.org/show_bug.cgi?id=739966
             if (typeof(RemoteDebugger) != "undefined")
             {
-                RemoteDebugger._start();
+                RemoteDebugger.init();
             }
             // Otherwise initialize the browser debugger.
-            else if (!DebuggerServer.initialized)
+            else
             {
-                // Initialize the server (e.g. appends script debugger actors)
-                DebuggerServer.init();
-
-                try
-                {
-                    // Only available on Fennec
-                    DebuggerServer.addActors("chrome://browser/content/dbg-browser-actors.js");
-                }
-                catch (err)
+                if (!DebuggerServer.initialized)
                 {
-                    // This should happen for Firefox
+                    // Initialize the server (e.g. appends script debugger actors)
+                    DebuggerServer.init(HttpServer._allowConnection);
                     DebuggerServer.addBrowserActors();
                 }
-
                 // Open a TCP listener
-                // xxxHonza: what about a pref for the port number and true/false for
-                // loopback device? What if the script debugger already opened that
-                // listener?
-                DebuggerServer.openListener(2929, false);
+                DebuggerServer.openListener(Services.prefs.getIntPref("devtools.debugger.remote-port"));
             }
         }
         catch (ex)
@@ -82,9 +73,35 @@ var HttpServer =
     {
         FBTrace.sysout("HttpServer; shutdown");
 
-        // xxxHonza: what if there are other tools sharing the connection?
         DebuggerServer.closeListener();
     },
+
+    /**
+     * Prompt the user to accept or decline the incoming connection.
+     *
+     * @return true if the connection should be permitted, false otherwise
+     */
+    _allowConnection: function()
+    {
+        let title = "Incoming Connection";
+        let msg = "An incoming request to permit remote debugging connection was detected. A remote client can take complete control over your browser! Allow connection?";
+        let disableButton = "Disable";
+        let prompt = Services.prompt;
+        let flags = prompt.BUTTON_POS_0 * prompt.BUTTON_TITLE_OK +
+                    prompt.BUTTON_POS_1 * prompt.BUTTON_TITLE_CANCEL +
+                    prompt.BUTTON_POS_2 * prompt.BUTTON_TITLE_IS_STRING +
+                    prompt.BUTTON_POS_1_DEFAULT;
+        let result = prompt.confirmEx(null, title, msg, flags, null, null,
+                                      disableButton, null, { value: false });
+        if (result == 0) {
+            return true;
+        }
+        if (result == 2) {
+            DebuggerServer.closeListener(true);
+            Services.prefs.setBoolPref("devtools.debugger.remote-enabled", false);
+        }
+        return false;
+    },
 }
 
 // ********************************************************************************************* //
diff --git a/content/httpmonitor/server/netMonitorActor.js b/content/httpmonitor/server/netMonitorActor.js
index 8db8503..a087e6e 100644
--- a/content/httpmonitor/server/netMonitorActor.js
+++ b/content/httpmonitor/server/netMonitorActor.js
@@ -211,30 +211,7 @@ NetworkMonitorActor.prototype.requestTypes =
     "sendRequest": NetworkMonitorActor.prototype.onSendRequest,
 };
 
-// ********************************************************************************************* //
-// Network Monitor Actor Handler
-
-function networkMonitorActorHandler(tab, request)
-{
-    // Reuse a previously-created actor, if any.
-    if (tab.networkMonitorActor)
-        return tab.networkMonitorActor.grip();
-
-    var actor = new NetworkMonitorActor(tab);
-    tab.networkMonitorActor = actor;
-
-    // xxxHonza: this needs to be reviewed by Panos
-    // If it's added into the tabActorPool, disconnect is not executed for the actor.
-    //tab.tabActorPool.addActor(actor);
-    tab.conn.addActor(actor);
-
-    if (FBTrace.DBG_NETACTOR)
-        FBTrace.sysout("networkMonitorActorHandler ", {tab: tab, request: request});
-
-    return actor.grip();
-}
-
-DebuggerServer.addTabActor("networkMonitorActor", NetworkMonitorActor);
+DebuggerServer.addTabActor(NetworkMonitorActor, "networkMonitorActor");
 
 // ********************************************************************************************* //
 });
diff --git a/content/httpmonitor/server/traceActor.js b/content/httpmonitor/server/traceActor.js
index f90acf2..fec8007 100644
--- a/content/httpmonitor/server/traceActor.js
+++ b/content/httpmonitor/server/traceActor.js
@@ -124,23 +124,7 @@ TraceActor.prototype.requestTypes =
     "detach": TraceActor.prototype.onDetach,
 };
 
-// ********************************************************************************************* //
-// Trace Actor Handler
-
-function traceActorHandler(rootActor, request)
-{
-    // Reuse a previously-created actor, if any.
-    if (rootActor.traceActor)
-        return rootActor.traceActor.grip();
-
-    var actor = new TraceActor(rootActor);
-    rootActor.traceActor = actor;
-    rootActor.conn.addActor(actor);
-
-    return actor.grip();
-}
-
-DebuggerServer.addGlobalActor("traceActor", TraceActor);
+DebuggerServer.addGlobalActor(TraceActor, "traceActor");
 
 // ********************************************************************************************* //
 });

From 91c1632403de042009f175b629120011435deca7 Mon Sep 17 00:00:00 2001
From: Panagiotis Astithas <pastith@gmail.com>
Date: Thu, 9 Aug 2012 18:28:10 +0300
Subject: [PATCH 3/3] Update the debugger server initialization code to account
 for bug 781515

---
 content/httpmonitor/server/httpServer.js | 29 +-----------------------
 1 file changed, 1 insertion(+), 28 deletions(-)

diff --git a/content/httpmonitor/server/httpServer.js b/content/httpmonitor/server/httpServer.js
index c5e4765..774e21b 100644
--- a/content/httpmonitor/server/httpServer.js
+++ b/content/httpmonitor/server/httpServer.js
@@ -56,7 +56,7 @@ var HttpServer =
                 if (!DebuggerServer.initialized)
                 {
                     // Initialize the server (e.g. appends script debugger actors)
-                    DebuggerServer.init(HttpServer._allowConnection);
+                    DebuggerServer.init();
                     DebuggerServer.addBrowserActors();
                 }
                 // Open a TCP listener
@@ -75,33 +75,6 @@ var HttpServer =
 
         DebuggerServer.closeListener();
     },
-
-    /**
-     * Prompt the user to accept or decline the incoming connection.
-     *
-     * @return true if the connection should be permitted, false otherwise
-     */
-    _allowConnection: function()
-    {
-        let title = "Incoming Connection";
-        let msg = "An incoming request to permit remote debugging connection was detected. A remote client can take complete control over your browser! Allow connection?";
-        let disableButton = "Disable";
-        let prompt = Services.prompt;
-        let flags = prompt.BUTTON_POS_0 * prompt.BUTTON_TITLE_OK +
-                    prompt.BUTTON_POS_1 * prompt.BUTTON_TITLE_CANCEL +
-                    prompt.BUTTON_POS_2 * prompt.BUTTON_TITLE_IS_STRING +
-                    prompt.BUTTON_POS_1_DEFAULT;
-        let result = prompt.confirmEx(null, title, msg, flags, null, null,
-                                      disableButton, null, { value: false });
-        if (result == 0) {
-            return true;
-        }
-        if (result == 2) {
-            DebuggerServer.closeListener(true);
-            Services.prefs.setBoolPref("devtools.debugger.remote-enabled", false);
-        }
-        return false;
-    },
 }
 
 // ********************************************************************************************* //