WebSocket Pepper API: allow to close connection without code and reason

ppapi/api/ppb_websocket.idl

 /* Copyright (c) 2012 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.
  */
 
 /**
  * This file defines the <code>PPB_WebSocket</code> interface providing
  * bi-directional, full-duplex, communications over a single TCP socket.
  */
 label Chrome {
@@ skipping 40 matching lines @@
  * information.
  * <code>PP_WEBSOCKETSTATUSCODE_NORMAL_CLOSURE</code> and codes in the range
  * <code>PP_WEBSOCKETSTATUSCODE_USER_REGISTERED_MIN</code> to
  * <code>PP_WEBSOCKETSTATUSCODE_USER_REGISTERED_MAX</code>, and
  * <code>PP_WEBSOCKETSTATUSCODE_USER_PRIVATE_MIN</code> to
  * <code>PP_WEBSOCKETSTATUSCODE_USER_PRIVATE_MAX</code> are valid for Close().
  */
 [assert_size(4)]
 enum PP_WebSocketCloseCode {
   /**
+   * Indicates to request closing connection without status code. This code is
+   * internally assigned in WebKit, and not based on any WebSocket related
+   * specification.
+   */
+  PP_WEBSOCKETSTATUSCODE_NOT_SPECIFIED = 65535,

[dmichael (off chromium), 2012/05/14 16:10:49]

Maybe you should use -1? Since code is unsigned in the spec, that should never
conflict with a real code. What do you think?

[Takashi Toyoshima, 2012/05/15 10:55:28]

Yes, in the same reason, I defined the equivalent code as -1 in the WebKit API.
As you said in the following comments, -1 must be better if I can change
argument type from unsigned short.

+
+  /**
    * Status codes in the range 0-999 are not used.
    */
 
   /**
    * Indicates a normal closure.
    */
   PP_WEBSOCKETSTATUSCODE_NORMAL_CLOSURE = 1000,
 
   /**
    * Indicates that an endpoint is "going away", such as a server going down.
@@ skipping 355 matching lines @@
    * @param[in] web_socket A <code>PP_Resource</code> corresponding to a
    * WebSocket.
    *
    * @return Returns a <code>PP_VARTYPE_STRING</code> var. If called before the
    * connection is established, the var contains the empty string. Returns a
    * <code>PP_VARTYPE_UNDEFINED</code> if this function is called on an
    * invalid resource.
    */
   PP_Var GetURL([in] PP_Resource web_socket);
 };