PepperWebSocketHost should not call blink::WebSocket APIs after closed.

PepperWebSocketHost should not call blink::WebSocket APIs after closed.

blink::WebSocket assumes that any APIs won't be called once disconnect is
called, but PepperWebSocketHost didn't keep the contract. Note that
PepperWebSocketHost can receive SEND* / CLOSE messages even if it sends
a CLOSED message to the ppapi process, because these IPCs are asynchronous.

BUG=407885
R=ricea@chromium.org

Committed: https://crrev.com/c411803acfaca9e2628f4ab937b738395ad8030f
Cr-Commit-Position: refs/heads/master@{#292167}

[Adam Rice, 2014-08-27 05:58:57]

lgtm

https://codereview.chromium.org/510663002/diff/1/ppapi/tests/test_websocket.cc
File ppapi/tests/test_websocket.cc (right):

https://codereview.chromium.org/510663002/diff/1/ppapi/tests/test_websocket.c...
ppapi/tests/test_websocket.cc:1181: for (size_t i = 0; i < 10000; ++i) {
Nit: this test could be flaky. However, I can't see a good way to de-flake it.

[yhirano, 2014-08-27 07:06:33]

yhirano@chromium.org changed reviewers:
+ dmichael@chromium.org

[yhirano, 2014-08-27 07:06:33]

+dmichael for OWNER review.

https://codereview.chromium.org/510663002/diff/1/ppapi/tests/test_websocket.cc
File ppapi/tests/test_websocket.cc (right):

https://codereview.chromium.org/510663002/diff/1/ppapi/tests/test_websocket.c...
ppapi/tests/test_websocket.cc:1181: for (size_t i = 0; i < 10000; ++i) {
On 2014/08/27 05:58:57, Adam Rice wrote:
> Nit: this test could be flaky. However, I can't see a good way to de-flake it.

me neither.

[dmichael (off chromium), 2014-08-27 14:51:31]

lgtm

[yhirano, 2014-08-27 14:57:06]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2014-08-27 14:58:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yhirano@chromium.org/510663002/1

[commit-bot: I haz the power, 2014-08-27 15:51:34]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)

[commit-bot: I haz the power, 2014-08-27 16:47:36]

Committed patchset #1 (id:1) as 0917fe5b17ac904f14ce8c3ccd844fed3c733199

[commit-bot: I haz the power, 2014-09-10 02:52:11]

Patchset 1 (id:??) landed as
https://crrev.com/c411803acfaca9e2628f4ab937b738395ad8030f
Cr-Commit-Position: refs/heads/master@{#292167}