[webkit-reviews] review granted: [Bug 136219] Websocket state should be set to closed in didReceiveMessage call back : [Attachment 237144] Patch-Updated

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Sep 15 17:39:45 PDT 2014


Alexey Proskuryakov <ap at webkit.org> has granted Shivakumar J M
<shiva.jm at samsung.com>'s request for review:
Bug 136219: Websocket state should be set to closed in didReceiveMessage call
back
https://bugs.webkit.org/show_bug.cgi?id=136219

Attachment 237144: Patch-Updated
https://bugs.webkit.org/attachment.cgi?id=237144&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
The existing code is somewhat confusing, but I think that the patch a step in
the right direction.

The key consideration is that there is only one place where an error event is
dispatched per the spec, and that's right before a close event.


More information about the webkit-reviews mailing list