[webkit-reviews] review granted: [Bug 82100] [WebSocket]Reserved bits test case should cover both extension and no-extension scenarios : [Attachment 133791] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 28 01:57:39 PDT 2012


Kent Tamura <tkent at chromium.org> has granted joey <li.yin at intel.com>'s request
for review:
Bug 82100: [WebSocket]Reserved bits test case should cover both extension and
no-extension scenarios
https://bugs.webkit.org/show_bug.cgi?id=82100

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

------- Additional Comments from Kent Tamura <tkent at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=133791&action=review


> LayoutTests/http/tests/websocket/tests/hybi/reserved-bits.html:17
> +var compress_bitNumber = ["compressed=false&bit_number=1",

nit: We usually use camelCase for variable names. 
compressedAndBitNumberParameters?


More information about the webkit-reviews mailing list