Decouple DevTools from socket implementation.

net/server/http_server.h

Index: net/server/http_server.h
diff --git a/net/server/http_server.h b/net/server/http_server.h
index aaf5336c66de891539737309f58496994d79373b..1e090929df25c6a480fa1405049d77c92f8409e0 100644
--- a/net/server/http_server.h
+++ b/net/server/http_server.h
@@ -39,7 +39,8 @@ class HttpServer : public StreamListenSocket::Delegate,
     virtual ~Delegate() {}
   };
 
-  HttpServer(const std::string& host, int port, HttpServer::Delegate* del);
+  HttpServer(HttpServer::Delegate* delegate,
+             StreamListenSocketFactory* socket_factory);

[mmenke, 2012/05/14 20:45:19]

Think the factory could actually be:

const StreamListenSocketFactory& socket_factory.

Advantage of that is it makes clear it can't be NULL, and we don't take
ownership.  Should also probably be the first parameter, since we don't mutate
it.

[Philippe, 2012/05/15 14:39:09]

Indeed. For the other places where we take ownership, I made it a const pointer.

 
   void AcceptWebSocket(int connection_id,
                        const HttpServerRequestInfo& request);