Remove timing limitation of SetOption invocation for PPAPI sockets.

content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h

 // 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_SOCKET_MESSAGE_FILTER_H_
 #define CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_TCP_SOCKET_MESSAGE_FILTER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 45 matching lines @@
 
   // Used for creating already connected sockets.
   PepperTCPSocketMessageFilter(BrowserPpapiHostImpl* host,
                                PP_Instance instance,
                                ppapi::TCPSocketVersion version,
                                scoped_ptr<net::TCPSocket> socket);
 
   static size_t GetNumInstances();
 
  private:
+  enum SocketOptions {
+    SOCKET_OPTION_NODELAY = 1 << 0,
+    SOCKET_OPTION_RCVBUF_SIZE = 1 << 1,
+    SOCKET_OPTION_SNDBUF_SIZE = 1 << 2
+  };
+
   ~PepperTCPSocketMessageFilter() override;
 
   // ppapi::host::ResourceMessageFilter overrides.
   scoped_refptr<base::TaskRunner> OverrideTaskRunnerForMessage(
       const IPC::Message& message) override;
   int32_t OnResourceMessageReceived(
       const IPC::Message& msg,
       ppapi::host::HostMessageContext* context) override;
 
   int32_t OnMsgBind(const ppapi::host::HostMessageContext* context,
@@ skipping 102 matching lines @@
   ppapi::TCPSocketState state_;
   bool end_of_file_reached_;
 
   // This is the address requested to bind. Please note that this is not the
   // bound address. For example, |bind_input_addr_| may have port set to 0.
   // It is used to check permission for listening.
   PP_NetAddress_Private bind_input_addr_;
 
   scoped_ptr<net::SingleRequestHostResolver> resolver_;
 
+  // Bitwise-or of SocketOption flags. This stores the state about whether
+  // each option is set.
+  int socket_options_;
+
+  // Receive/Send buffer sizes which is set before connect().

[jar (doing other things), 2014/11/04 22:51:09]

This code actually sets this value whether called before, or after connect().

I think the comment should be something like:

"Locally cached value of buffer sizes, if set to a non-default value."

[hidehiko, 2014/11/05 12:48:29]

As we do not take care about "default" value here, I just said:
"Locally cached value of buffer sizes."
Please note that, this is actually used only "before" connect() in *this layer*,
unlike net/... changes I made in this CL.

+  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
   // successfully established.
   net::AddressList address_list_;
   // Where we are in the above list.
   size_t address_index_;
 
   // Non-null unless an SSL connection is requested.
   scoped_ptr<net::TCPSocket> socket_;
@@ skipping 13 matching lines @@
   bool pending_accept_;
   scoped_ptr<net::TCPSocket> accepted_socket_;
   net::IPEndPoint accepted_address_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperTCPSocketMessageFilter);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_TCP_SOCKET_MESSAGE_FILTER_H_