From RFC 6455 http://tools.ietf.org/html/rfc6455#section-11.3.3 The |Sec-WebSocket-Accept| header MUST NOT appear more than once in an HTTP response.
Created attachment 132813 [details] Patch
Comment on attachment 132813 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=132813&action=review > Source/WebCore/Modules/websockets/WebSocketHandshake.cpp:578 > + bool hasAcceptValue = false; "Accept" is also an HTTP header, so this variable is named confusingly. I suggest something like "sawSecWebSocketAcceptHeaderField".
Created attachment 132940 [details] Patch
(In reply to comment #2) > (From update of attachment 132813 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=132813&action=review > > > Source/WebCore/Modules/websockets/WebSocketHandshake.cpp:578 > > + bool hasAcceptValue = false; > > "Accept" is also an HTTP header, so this variable is named confusingly. I suggest something like "sawSecWebSocketAcceptHeaderField". Thanks for your review, the patch has been updated already, please review again.
Comment on attachment 132940 [details] Patch Clearing flags on attachment: 132940 Committed r111554: <http://trac.webkit.org/changeset/111554>
All reviewed patches have been landed. Closing bug.