Decouple DevTools from socket implementation.

chrome/browser/debugger/remote_debugging_server.cc

Index: chrome/browser/debugger/remote_debugging_server.cc
diff --git a/chrome/browser/debugger/remote_debugging_server.cc b/chrome/browser/debugger/remote_debugging_server.cc
index 0094e7e38e10f5d9a66f3a20c36c84dd161cab5f..e8c6441ecabece8d0c0460f6c65a750c571f9b0e 100644
--- a/chrome/browser/debugger/remote_debugging_server.cc
+++ b/chrome/browser/debugger/remote_debugging_server.cc
@@ -9,6 +9,7 @@
 #include "chrome/browser/ui/webui/chrome_url_data_manager.h"
 #include "chrome/browser/ui/webui/devtools_ui.h"
 #include "content/public/browser/devtools_http_handler.h"
+#include "net/base/tcp_listen_socket.h"
 
 RemoteDebuggingServer::RemoteDebuggingServer(Profile* profile,
                                              const std::string& ip,
@@ -19,12 +20,12 @@ RemoteDebuggingServer::RemoteDebuggingServer(Profile* profile,
 
   net::URLRequestContextGetter* request_context_getter =
       profile->GetRequestContext();
-  devtools_http_handler_ =
-      content::DevToolsHttpHandler::Start(ip,
-                                          port,
-                                          frontend_url,
-                                          request_context_getter,
-                                          new BrowserListTabContentsProvider());
+
+  devtools_http_handler_ = content::DevToolsHttpHandler::Start(
+      frontend_url,
+      new net::TCPListenSocketFactory(ip, port),

[mmenke, 2012/05/15 17:40:40]

arguments now in the wrong order here.

[Philippe, 2012/05/16 16:56:24]

Indeed.

+      request_context_getter,
+      new BrowserListTabContentsProvider());
 }
 
 RemoteDebuggingServer::~RemoteDebuggingServer() {