PPB_TCPSocket: add support for TCP server socket operations.

content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h

Index: content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h
diff --git a/content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h b/content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h
index 64c8b4210d6da134686aed494dfdfb88fd519fff..9fd451ce7f6099410e823b5f4ca62839c137e645 100644
--- a/content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h
+++ b/content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h
@@ -8,11 +8,9 @@
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "net/socket/tcp_socket.h"

[bbudge, 2013/09/17 19:42:18]

Could you forward declare net::TCPSocket?

[yzshen1, 2013/09/18 16:56:26]

I tried but it wasn't allowed.
net::TCPSocket is a typedef, I guess that is a problem?

On 2013/09/17 19:42:18, bbudge1 wrote:

 #include "ppapi/host/host_factory.h"
-
-namespace net {
-class StreamSocket;
-}
+#include "ppapi/shared_impl/ppb_tcp_socket_shared.h"
 
 namespace ppapi {
 class PpapiPermissions;
@@ -35,13 +33,12 @@ class ContentBrowserPepperHostFactory : public ppapi::host::HostFactory {
       PP_Instance instance,
       const IPC::Message& message) OVERRIDE;
 
-  // Creates ResourceHost for already accepted TCP |socket|.  Takes
-  // ownership of the |socket|. In the case of failure returns wrapped
-  // NULL.
+  // Creates ResourceHost for already accepted TCP |socket|. In the case of
+  // failure returns wrapped NULL.
   scoped_ptr<ppapi::host::ResourceHost> CreateAcceptedTCPSocket(
       PP_Instance instance,
-      bool private_api,
-      net::StreamSocket* socket);
+      ppapi::TCPSocketVersion version,
+      scoped_ptr<net::TCPSocket> socket);
 
  private:
   const ppapi::PpapiPermissions& GetPermissions() const;