WebSocket Pepper API documents minor revision

WebSocket Pepper API documents minor revision

BUG=87310
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110260

[Takashi Toyoshima, 2011-11-15 14:45:13]

I'd like to make minor revision on WebSocket Pepper API documents.
Could you review on this minor changes?

1. GetCloseReason() was inconsistent with other interfaces which return PP_Var.
 - must return empty string if called before the value is set.

2. Returns PP_VARTYPE_UNDEFINED if called on an invalid resource on interfaces
which return PP_Var.
 - I'm preparing a thunk implementation ( http://codereview.chromium.org/8571002
), then I feel allocating empty string via ppb_var interface looks a little
expensive for those cases.

3. Return PP_WEBSOCKETREADYSTATE_CLOSED_DEV if called before connect() is
called.
 - JS API defines it return CONNECTING before connection is established because
connection started at object creation time. But in our case, it's not true. I
think returning CLOSED is better.

[dmichael (off chromium), 2011-11-15 17:33:11]

one suggestion, otherwise LGTM

http://codereview.chromium.org/8572009/diff/1/ppapi/api/dev/ppb_websocket_dev...
File ppapi/api/dev/ppb_websocket_dev.idl (right):

http://codereview.chromium.org/8572009/diff/1/ppapi/api/dev/ppb_websocket_dev...
ppapi/api/dev/ppb_websocket_dev.idl:290: * before connect() is called, or called
on an invalid resource.
It seems to me it would make sense to define a new value; say,
'PP_WEBSOCKETREADYSTATE_INVALID_DEV' or similar.

[Takashi Toyoshima, 2011-11-15 17:50:05]

Thanks.
I fixed it.

[commit-bot: I haz the power, 2011-11-16 02:31:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/toyoshim@chromium.org/8572009/1004

[commit-bot: I haz the power, 2011-11-16 05:25:02]

Change committed as 110260