Remove timing limitation of SetOption invocation for PPAPI sockets.

content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h

Index: content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h
diff --git a/content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h b/content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h
index 7c9250f48995bc4e6fd35d377c06dc9da08e5294..897a92b43c3db4f375a78468ca3bd9fec2904ce6 100644
--- a/content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h
+++ b/content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h
@@ -63,6 +63,12 @@ class CONTENT_EXPORT PepperTCPSocketMessageFilter
   static size_t GetNumInstances();
 
  private:
+  enum SocketOption {
+    SOCKET_OPTION_NODELAY = 1 << 0,
+    SOCKET_OPTION_RCVBUF_SIZE = 1 << 1,
+    SOCKET_OPTION_SNDBUF_SIZE = 1 << 2
+  };
+
   ~PepperTCPSocketMessageFilter() override;
 
   // ppapi::host::ResourceMessageFilter overrides.
@@ -185,6 +191,14 @@ class CONTENT_EXPORT PepperTCPSocketMessageFilter
 
   scoped_ptr<net::SingleRequestHostResolver> resolver_;
 
+  // Bitwise-or of SocketOption flags. This stores the state about whether
+  // each option is set.

[bbudge, 2014/11/07 17:50:39]

To make the comment a little clearer, how about this?

// Bitwise-or of SocketOption flags. This stores the state about whether
// each option is set before Connect() is called.

[hidehiko, 2014/11/07 21:13:55]

Done.

+  int socket_options_;
+
+  // Locally cached value of buffer size.
+  int32_t rcvbuf_size_;
+  int32_t sndbuf_size_;
+
   // |address_list_| may store multiple addresses when
   // PPB_TCPSocket_Private.Connect() is used, which involves name resolution.
   // In that case, we will try each address in the list until a connection is