https://bugs.webkit.org/show_bug.cgi?id=80103#c16 Introduce ThreadableWebsocketChannel::SendResult so that WebSocketChannel can return the result of message validation.
Created attachment 130243 [details] Patch
Comment on attachment 130243 [details] Patch ok
Comment on attachment 130243 [details] Patch Thank you!
Comment on attachment 130243 [details] Patch Clearing flags on attachment: 130243 Committed r109832: <http://trac.webkit.org/changeset/109832>
All reviewed patches have been landed. Closing bug.
Sorry, I had to revert this due to WebSocket test failures on Mac and Linux: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=ui_tests&tests=PPAPITest.WebSocket_UtilityTextSendReceive%2CPPAPINaClTest.WebSocket_BinarySendReceive%2CPPAPINaClTest.WebSocket_UtilityBinarySendReceive%2CPPAPITest.WebSocket_BinarySendReceive%2CPPAPITest.WebSocket_TextSendReceive%2CPPAPINaClTest.WebSocket_UtilityBufferedAmount%2CPPAPINaClTest.WebSocket_CcInterfaces%2CPPAPINaClTest.WebSocket_TextSendReceive%2CPPAPITest.WebSocket_BufferedAmount%2CPPAPITest.WebSocket_CcInterfaces%2CPPAPINaClTest.WebSocket_BufferedAmount%2CPPAPITest.WebSocket_UtilityBufferedAmount%2CPPAPINaClTest.WebSocket_UtilityTextSendReceive%2CPPAPITest.WebSocket_UtilityBinarySendReceive Rolled out in r109942. (The rollout script failed to re-open the bug, so I'm doing it here).
Created attachment 130484 [details] Patch
(In reply to comment #6) > Rolled out in r109942. (The rollout script failed to re-open the bug, so I'm doing it here). Sorry for breaking tests. I didn't notice that WebKit/chromium/src/WebSocketImpl.cpp uses WebSocketChannel directly. Revised the patch to check the return value of WebSocketChannel::send() in WebSocketImpl.
Comment on attachment 130484 [details] Patch Clearing flags on attachment: 130484 Committed r110012: <http://trac.webkit.org/changeset/110012>