Update comments of the Pepper networking APIs.

ppapi/cpp/dev/tcp_socket_dev.h

Index: ppapi/cpp/dev/tcp_socket_dev.h
diff --git a/ppapi/cpp/dev/tcp_socket_dev.h b/ppapi/cpp/dev/tcp_socket_dev.h
index 16d7deedba577dac95720136734f472ffb371c09..88296629e85bfd942c92613e5f72354cf3e52925 100644
--- a/ppapi/cpp/dev/tcp_socket_dev.h
+++ b/ppapi/cpp/dev/tcp_socket_dev.h
@@ -15,34 +15,147 @@ namespace pp {
 class CompletionCallback;
 class InstanceHandle;
 
+/// The <code>TCPSocket_Dev</code> class provides TCP socket operations.
+///
+/// Permissions: Apps permission <code>socket</code> with subrule
+/// <code>tcp-connect</code> is required for <code>Connect()</code>.
+/// For more details about network communication permissions, please see:
+/// http://developer.chrome.com/apps/app_network.html
 class TCPSocket_Dev: public Resource {
  public:
+  /// Default constructor for creating an is_null() <code>TCPSocket_Dev</code>
+  /// object.
   TCPSocket_Dev();
 
+  /// A contructor used to create a <code>TCPSocket_Dev</code> object.

[bbudge, 2013/06/20 21:02:32]

s/contructor/constructor

[yzshen1, 2013/06/20 22:20:48]

Done.

+  ///
+  /// @param[in] instance The instance with which this resource will be
+  /// associated.
   explicit TCPSocket_Dev(const InstanceHandle& instance);
 
+  /// A constructor used when you have received a <code>PP_Resource</code> as a
+  /// return value that has already been added 1 ref for you.

[bbudge, 2013/06/20 21:02:32]

s/been added 1 ref/has had 1 ref added

[yzshen1, 2013/06/20 22:20:48]

Done.

+  ///
+  /// @param[in] resource A <code>PPB_TCPSocket_Dev</code> resource.
   TCPSocket_Dev(PassRef, PP_Resource resource);
 
+  /// The copy constructor for <code>TCPSocket_Dev</code>.
+  ///
+  /// @param[in] other A reference to another <code>TCPSocket_Dev</code>.
   TCPSocket_Dev(const TCPSocket_Dev& other);
 
+  /// The destructor.
   virtual ~TCPSocket_Dev();
 
+  /// The assign operator for <code>TCPSocket_Dev</code>.

[bbudge, 2013/06/20 21:02:32]

s/assign/assignment

[yzshen1, 2013/06/20 22:20:48]

Done.

+  ///
+  /// @param[in] other A reference to another <code>TCPSocket_Dev</code>.
+  ///
+  /// @return A reference to this <code>TCPSocket_Dev</code> object.
   TCPSocket_Dev& operator=(const TCPSocket_Dev& other);
 
-  // Returns true if the required interface is available.
+  /// Static function for determining whether the browser supports the required

[bbudge, 2013/06/20 21:02:32]

s/required//

[yzshen1, 2013/06/20 22:20:48]

Done.

+  /// <code>PPB_TCPSocket_Dev</code> interface.
+  ///
+  /// @return true if the interface is available, false otherwise.
   static bool IsAvailable();
 
+  /// Connects to the given address.

[bbudge, 2013/06/20 21:02:32]

s/Connects/Connects the socket

[yzshen1, 2013/06/20 22:20:48]

Done.

+  ///
+  /// @param[in] addr A <code>NetAddress_Dev</code> object.
+  /// @param[in] callback A <code>CompletionCallback</code> to be called upon
+  /// completion.
+  ///
+  /// @return An int32_t containing an error code from <code>pp_errors.h</code>,
+  /// including (but not limited to):
+  /// - <code>PP_ERROR_NOACCESS</code>: the caller doesn't have required
+  ///   permissions.
+  /// - <code>PP_ERROR_ADDRESS_UNREACHABLE</code>: <code>addr</code> is
+  ///   unreachable.
+  /// - <code>PP_ERROR_CONNECTION_REFUSED</code>: the connection attempt was
+  ///   refused.
+  /// - <code>PP_ERROR_CONNECTION_FAILED</code>: the connection attempt failed.
+  /// - <code>PP_ERROR_CONNECTION_TIMEDOUT</code>: the connection attempt timed
+  ///   out.
   int32_t Connect(const NetAddress_Dev& addr,
                   const CompletionCallback& callback);
+
+  /// Gets the local address of the socket, if it is connected.
+  ///
+  /// @return A <code>NetAddress_Dev</code> object. The object will be null
+  /// (i.e., is_null() returns true) on failure.
   NetAddress_Dev GetLocalAddress() const;
+
+  /// Gets the remote address of the socket, if it is connected.
+  ///
+  /// @return A <code>NetAddress_Dev</code> object. The object will be null
+  /// (i.e., is_null() returns true) on failure.
   NetAddress_Dev GetRemoteAddress() const;
+
+  /// Reads data from the socket. The socket must be connected. It may perform a
+  /// partial read.
+  ///
+  /// <strong>Caveat:</strong> You should be careful about the lifetime of
+  /// <code>buffer</code>. Typically you will use a
+  /// <code>CompletionCallbackFactory</code> to scope callbacks to the lifetime
+  /// of your class. When your class goes out of scope, the callback factory
+  /// will not actually cancel the operation, but will rather just skip issuing
+  /// the callback on your class. This means that if the underlying
+  /// <code>PPB_TCPSocket_Dev</code> resource outlives your class, the browser
+  /// will still try to write into your buffer when the operation completes.
+  /// The buffer must be kept valid until then to avoid memory corruption.
+  /// If you want to release the buffer while the <code>Read()</code> call is
+  /// still pending, you should call <code>Close()</code> to ensure that the
+  /// buffer won't be accessed in the future.
+  ///
+  /// @param[out] buffer The buffer to store the received data on success. It
+  /// must be at least as large as <code>bytes_to_read</code>.
+  /// @param[in] bytes_to_read The number of bytes to read.
+  /// @param[in] callback A <code>CompletionCallback</code> to be called upon
+  /// completion.
+  ///
+  /// @return A non-negative number on success to indicate how many bytes have
+  /// been read, 0 means that end-of-file was reached; otherwise, an error code
+  /// from <code>pp_errors.h</code>.
   int32_t Read(char* buffer,
                int32_t bytes_to_read,
                const CompletionCallback& callback);
+
+  /// Writes data to the socket. The socket must be connected. It may perform a
+  /// partial write.
+  ///
+  /// @param[in] buffer The buffer containing the data to write.
+  /// @param[in] bytes_to_write The number of bytes to write.
+  /// @param[in] callback A <code>CompletionCallback</code> to be called upon
+  /// completion.
+  ///
+  /// @return A non-negative number on success to indicate how many bytes have
+  /// been written; otherwise, an error code from <code>pp_errors.h</code>.
   int32_t Write(const char* buffer,
                 int32_t bytes_to_write,
                 const CompletionCallback& callback);
+
+  /// Cancels all pending reads and writes, disconnects the socket. Any pending

[bbudge, 2013/06/20 21:02:32]

s/,/ and

[yzshen1, 2013/06/20 22:20:48]

Done.

+  /// callbacks will still run, reporting <code>PP_ERROR_ABORTED</code> if
+  /// pending IO was interrupted. After a call to this method, no output
+  /// buffer pointers passed into previous <code>Read()</code> calls will be
+  /// accessed. It is not valid to call <code>Connect()</code> again.
+  ///
+  /// The socket is implicitly closed if it is destroyed, so you are not
+  /// required to call this method.
   void Close();
+
+  /// Sets a socket option on the TCP socket.
+  /// Please see the <code>PP_TCPSocket_Option_Dev</code> description for option
+  /// names, value types and allowed values.
+  ///
+  /// @param[in] name The option type to set.

[bbudge, 2013/06/20 21:02:32]

nit s/option type/option

[yzshen1, 2013/06/20 22:20:48]

Done.

+  /// @param[in] value The option value to set.
+  /// @param[in] callback A <code>CompletionCallback</code> to be called upon
+  /// completion.
+  ///
+  /// @return An int32_t containing an error code from <code>pp_errors.h</code>.
+  ////
   int32_t SetOption(PP_TCPSocket_Option_Dev name,
                     const Var& value,
                     const CompletionCallback& callback);