Websocket documentation change

ppapi/api/ppb_websocket.idl

Index: ppapi/api/ppb_websocket.idl
===================================================================
--- ppapi/api/ppb_websocket.idl	(revision 128234)
+++ ppapi/api/ppb_websocket.idl	(working copy)
@@ -223,7 +223,7 @@
    * WebSocket API specification.
    * Returns <code>PP_ERROR_NOACCESS</code> if the protocol specified in the
    * <code>url</code> is not a secure protocol, but the origin of the caller
-   * has a secure scheme. Also returns <code>PP_ERROR_NOACCESS</CODE> if the
+   * has a secure scheme. Also returns <code>PP_ERROR_NOACCESS</code> if the
    * port specified in the <code>url</code> is a port that the user agent
    * is configured to block access to because it is a well-known port like
    * SMTP. <code>PP_ERROR_NOACCESS</code> corresponds to SecurityError of the
@@ -244,7 +244,7 @@
    * @param[in] web_socket A <code>PP_Resource</code> corresponding to a
    * WebSocket.
    *
-   * @param[in] code The WebSocket close code. Ignored if it is 0.
+   * @param[in] code The WebSocket close code. This is ignored if it is 0.
    * <code>PP_WEBSOCKETSTATUSCODE_NORMAL_CLOSURE</code> must be used for the
    * usual case. To indicate some specific error cases, codes in the range
    * <code>PP_WEBSOCKETSTATUSCODE_USER_REGISTERED_MIN</code> to
@@ -268,8 +268,8 @@
    * in the WebSocket API specification.
    * Returns <code>PP_ERROR_NOACCESS</code> if the code is not an integer
    * equal to 1000 or in the range 3000 to 4999. <code>PP_ERROR_NOACCESS</code>
-   * corresponds to InvalidAccessError of the specification. Returns
-   * <code>PP_ERROR_INPROGRESS</code> if this is not the first call to
+   * corresponds to InvalidAccessError in the WebSocket API specification.

[dmichael (off chromium), 2012/03/23 20:48:15]

If you do add the indefinite article before the errors, shouldn't you also do it
in the C interface? I mostly care that we're consistent. (Though I have a slight
preference for just leaving articles out in front of the error codes).

[jond, 2012/03/23 21:42:07]

Yeah, I actually like leave it off. I'm going to do that.

On 2012/03/23 20:48:15, dmichael wrote:

+   * Returns <code>PP_ERROR_INPROGRESS</code> if this is not the first call to
    * Close().
    */
   int32_t Close([in] PP_Resource web_socket,