diff options
| author | toyoshim@chromium.org <toyoshim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-25 11:39:07 +0000 |
|---|---|---|
| committer | toyoshim@chromium.org <toyoshim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-25 11:39:07 +0000 |
| commit | 9c8b2fd3e1451243ef8e1b6fbe8f7325b7566a3c (patch) | |
| tree | a2c843b405f0bb52b859f71dbaf2f88c8d5cf43a /ppapi/api | |
| parent | 6a6d0edd5ffcd8eb6b16be2376e2aba00cb961aa (diff) | |
| download | chromium_src-9c8b2fd3e1451243ef8e1b6fbe8f7325b7566a3c.zip chromium_src-9c8b2fd3e1451243ef8e1b6fbe8f7325b7566a3c.tar.gz chromium_src-9c8b2fd3e1451243ef8e1b6fbe8f7325b7566a3c.tar.bz2 | |
WebSocket Pepper API: the second Close() should not return error code.
The WebSocket API defines that call to Close() on CLOSED state must do nothing
and cause no error. Currently Pepper API returns PP_ERROR_INPROGRESS. This
error code is confusing. We should return PP_OK here.
BUG=124866
TEST=browser_tests --gtest_filter='*WebSocket*'
Review URL: http://codereview.chromium.org/10169036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133894 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/api')
| -rw-r--r-- | ppapi/api/ppb_websocket.idl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ppapi/api/ppb_websocket.idl b/ppapi/api/ppb_websocket.idl index 7ec3e1f..1e138bb 100644 --- a/ppapi/api/ppb_websocket.idl +++ b/ppapi/api/ppb_websocket.idl @@ -269,8 +269,8 @@ interface PPB_WebSocket { * 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 an InvalidAccessError in the WebSocket API specification. - * Returns <code>PP_ERROR_INPROGRESS</code> if this is not the first call to - * Close(). + * Returns <code>PP_ERROR_INPROGRESS</code> if a previous call to Close() is + * not finished. */ int32_t Close([in] PP_Resource web_socket, [in] uint16_t code, |