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/cpp | |
| 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/cpp')
| -rw-r--r-- | ppapi/cpp/websocket.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ppapi/cpp/websocket.h b/ppapi/cpp/websocket.h index 8a4d6f4..a34fdb7 100644 --- a/ppapi/cpp/websocket.h +++ b/ppapi/cpp/websocket.h @@ -94,7 +94,7 @@ class WebSocket : public Resource { /// 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(). + /// if a previous call to Close() is not finished. int32_t Close(uint16_t code, const Var& reason, const CompletionCallback& callback); |