Pass the reason of a failure to SocketStreamError instance and notify it via SocketStreamHandleClient::didFailSocketStream().
Created attachment 196818 [details] Patch
Comment on attachment 196818 [details] Patch Thanks for your contribution! I think the change is good, but can you check if you missed any instance of didFailSocketStream being called in this file? Thanks!
(In reply to comment #2) > (From update of attachment 196818 [details]) > Thanks for your contribution! I think the change is good, but can you check if you missed any instance of didFailSocketStream being called in this file? Thanks! The only other call to didFailSocketStream is in notifyClose, which has no message param, so I think it's fine for the SocketError to use the default message-less constructor for that call.
Comment on attachment 196818 [details] Patch Ok, after Joe's explanation, this change is fine!
Comment on attachment 196818 [details] Patch Clearing flags on attachment: 196818 Committed r147915: <http://trac.webkit.org/changeset/147915>
All reviewed patches have been landed. Closing bug.