Replace StreamListenSocket with StreamSocket in HttpServer.

net/socket/tcp_server_socket.h

Index: net/socket/tcp_server_socket.h
diff --git a/net/socket/tcp_server_socket.h b/net/socket/tcp_server_socket.h
index faff9ad826af3115ac25bb31e64706c9a290f926..ce6f45758de916a9e65c3090e2c2ec7e3a179ea7 100644
--- a/net/socket/tcp_server_socket.h
+++ b/net/socket/tcp_server_socket.h
@@ -11,6 +11,7 @@
 #include "net/base/ip_endpoint.h"
 #include "net/base/net_export.h"
 #include "net/base/net_log.h"
+#include "net/base/net_util.h"

[mmenke, 2014/05/23 19:20:58]

What is this needed for?

[byungchul, 2014/05/30 00:19:02]

Done.

 #include "net/socket/server_socket.h"
 #include "net/socket/tcp_socket.h"
 
@@ -27,6 +28,9 @@ class NET_EXPORT_PRIVATE TCPServerSocket : public ServerSocket {
   virtual int Accept(scoped_ptr<StreamSocket>* socket,
                      const CompletionCallback& callback) OVERRIDE;
 
+  // Factory with IP address string and port.
+  static scoped_ptr<TCPServerSocket> Create(const std::string& ip, int port);

[mmenke, 2014/05/23 19:20:58]

I think it's kinda of weird to have TCPServerSocket() and Listen() combined, but
only when using this special alternative function, which takes in address in a
different format.

If it's worth adding a convenience function that takes the ip as a string, I
think it should be added as an alternative Listen function, to ServerSocket. 
The implementation should also be in ServerSocket, since it just calls into
Listen.

You should also modify existing callers that could benefit from the function to
use it (See extensions/browser/api/socket/tcp_socket.cc).

[byungchul, 2014/05/30 00:19:02]

Defined ServerSocketFactory which is necessary to store ip/port info and
create/listen later for devtools.

Added TODO for tcp_socket.cc.

+
  private:
   // Converts |accepted_socket_| and stores the result in
   // |output_accepted_socket|.