Switched proxy for TCPServerSocketPrivate.

content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.h

Index: content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.h
diff --git a/content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.h b/content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.h
new file mode 100644
index 0000000000000000000000000000000000000000..5096cdf4b7c7b7e1343f236f71bb48c828ee71e1
--- /dev/null
+++ b/content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.h
@@ -0,0 +1,101 @@
+// Copyright 2013 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_RENDERER_HOST_PEPPER_PEPPER_TCP_SERVER_SOCKET_MESSAGE_FILTER_H_
+#define CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_TCP_SERVER_SOCKET_MESSAGE_FILTER_H_
+
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "content/browser/renderer_host/pepper/pepper_message_filter.h"
+#include "content/common/content_export.h"
+#include "ppapi/c/pp_instance.h"
+#include "ppapi/host/resource_message_filter.h"
+
+struct PP_NetAddress_Private;
+
+namespace net {
+class ServerSocket;
+class StreamSocket;
+}
+
+namespace content {
+
+class BrowserPpapiHostImpl;
+
+class CONTENT_EXPORT PepperTCPServerSocketMessageFilter
+    : public ppapi::host::ResourceMessageFilter {
+ public:
+  PepperTCPServerSocketMessageFilter(
+      BrowserPpapiHostImpl* host,
+      PP_Instance instance,
+      bool private_api,
+      const scoped_refptr<PepperMessageFilter>& pepper_message_filter);
+ protected:
+  virtual ~PepperTCPServerSocketMessageFilter();
+
+ private:
+  enum State {
+    STATE_BEFORE_LISTENING,
+    STATE_LISTEN_IN_PROGRESS,
+    STATE_LISTENING,
+    STATE_ACCEPT_IN_PROGRESS,
+    STATE_CLOSED
+  };
+
+  // ppapi::host::ResourceMessageFilter overrides.
+  virtual scoped_refptr<base::TaskRunner> OverrideTaskRunnerForMessage(
+      const IPC::Message& message) OVERRIDE;
+  virtual int32_t OnResourceMessageReceived(
+      const IPC::Message& msg,
+      ppapi::host::HostMessageContext* context) OVERRIDE;
+
+  int32_t OnMsgListen(const ppapi::host::HostMessageContext* context,
+                      const PP_NetAddress_Private& addr,
+                      int32_t backlog);
+  int32_t OnMsgAccept(const ppapi::host::HostMessageContext* context,
+                      uint32 plugin_dispatcher_id);
+  int32_t OnMsgStopListening(const ppapi::host::HostMessageContext* context);
+
+  void DoListen(const ppapi::host::ReplyMessageContext& context,
+                const PP_NetAddress_Private& addr,
+                int32_t backlog);
+
+  void OnListenCompleted(const ppapi::host::ReplyMessageContext& context,
+                         int result);

[yzshen1, 2013/07/19 23:51:58]

This is net::Error right?
Maybe it is better to use "int net_result". Because we also have "int32_t
result" which is PP_Error. This way, it is less likely that we pass "net_result"
mistakenly as a PP_Error.

[ygorshenin1, 2013/07/29 14:03:57]

Done.

+  void OnAcceptCompleted(const ppapi::host::ReplyMessageContext& context,
+                         uint32 plugin_dispatcher_id,
+                         int result);

[yzshen1, 2013/07/19 23:51:58]

ditto

[ygorshenin1, 2013/07/29 14:03:57]

Done.

+
+  void SendListenReply(const ppapi::host::ReplyMessageContext& context,
+                       int32_t result,
+                       const PP_NetAddress_Private& local_addr);
+  void SendListenError(const ppapi::host::ReplyMessageContext& context,
+                       int32_t result);
+  void SendAcceptReply(const ppapi::host::ReplyMessageContext& context,
+                       int32_t result,
+                       uint32 accepted_socket_id,
+                       const PP_NetAddress_Private& local_addr,
+                       const PP_NetAddress_Private& remote_addr);
+  void SendAcceptError(const ppapi::host::ReplyMessageContext& context,
+                       int32_t result);
+
+  State state_;
+  scoped_ptr<net::ServerSocket> socket_;
+  scoped_ptr<net::StreamSocket> socket_buffer_;
+
+  bool external_plugin_;
+  bool private_api_;
+  scoped_refptr<PepperMessageFilter> pepper_message_filter_;

[yzshen1, 2013/07/19 23:51:58]

nit: it is better to reorder the members according to whether they are used on
the IO or UI thread, and add comments

[ygorshenin1, 2013/07/29 14:03:57]

Done.

+
+  int render_process_id_;
+  int render_view_id_;
+
+  DISALLOW_COPY_AND_ASSIGN(PepperTCPServerSocketMessageFilter);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_TCP_SERVER_SOCKET_MESSAGE_FILTER_H_