TCPSockets are switched to the new Pepper proxy.

content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_PEPPER_CONTENT_BROWSER_PEPPER_HOST_FACTORY_H_
 #define CONTENT_BROWSER_PEPPER_CONTENT_BROWSER_PEPPER_HOST_FACTORY_H_
 
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
-#include "content/browser/renderer_host/pepper/pepper_message_filter.h"
+#include "content/browser/renderer_host/pepper/ssl_context_helper.h"
 #include "ppapi/host/host_factory.h"
 
+namespace net {
+class StreamSocket;
+}
+
 namespace ppapi {
 class PpapiPermissions;
 }
 
 namespace content {
 
 class BrowserPpapiHostImpl;
 
 class ContentBrowserPepperHostFactory : public ppapi::host::HostFactory {
  public:
   // Non-owning pointer to the filter must outlive this class.
-  ContentBrowserPepperHostFactory(
-      BrowserPpapiHostImpl* host,
-      // TODO (ygorshenin@): remove this once TCP sockets are
-      // converted to the new design.
-      const scoped_refptr<PepperMessageFilter>& pepper_message_filter);
+  ContentBrowserPepperHostFactory(BrowserPpapiHostImpl* host);

[yzshen1, 2013/08/16 20:40:42]

explicit?

[ygorshenin1, 2013/08/19 14:33:35]

Done.

+
   virtual ~ContentBrowserPepperHostFactory();
 
   virtual scoped_ptr<ppapi::host::ResourceHost> CreateResourceHost(
       ppapi::host::PpapiHost* host,
       const ppapi::proxy::ResourceMessageCallParams& params,
       PP_Instance instance,
       const IPC::Message& message) OVERRIDE;
 
+  // Creates PepperTCPSocketMessageFilter from |socket| in already
+  // connected state and puts it into a list of pending resource
+  // hosts. In the case of success returns id of the corresponding
+  // pending resource host, zero otherwise.

[yzshen1, 2013/08/16 20:40:42]

- Please take about the ownership of |socket|.
- It is better to let this factory return scoped_ptr<ppapi::host::ResourceHost>,
the work of adding to pending resource host list should be handled by
TCPServerSocket itself.

[ygorshenin1, 2013/08/19 14:33:35]

Done.

+  int AddAcceptedTCPSocket(PP_Instance instance,
+                           bool private_api,
+                           net::StreamSocket* socket);
+
  private:
   const ppapi::PpapiPermissions& GetPermissions() const;
 
   // Non-owning pointer.
   BrowserPpapiHostImpl* host_;
 
-  scoped_refptr<PepperMessageFilter> pepper_message_filter_;
+  scoped_refptr<SSLContextHelper> ssl_context_helper_;

[yzshen1, 2013/08/16 20:40:42]

Maybe we could make it part of BrowserPpapiHostImpl?
And then you don't need to pass it into TCPSocket, because it has access to
BrowserPpapiHostImpl.

[ygorshenin1, 2013/08/19 14:33:35]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(ContentBrowserPepperHostFactory);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_PEPPER_CONTENT_BROWSER_PEPPER_HOST_FACTORY_H_