Notify WebSocket connection failure, chromium side

net/websockets/websocket_event_interface.h

Index: net/websockets/websocket_event_interface.h
diff --git a/net/websockets/websocket_event_interface.h b/net/websockets/websocket_event_interface.h
index 7eb54bf8b15a09534405cbf690150adf2ce64f0d..36f7530eb52c2327adf7e7a96d2e5947d91dc24d 100644
--- a/net/websockets/websocket_event_interface.h
+++ b/net/websockets/websocket_event_interface.h
@@ -51,6 +51,7 @@ class NET_EXPORT WebSocketEventInterface {
   // Called when the channel has been dropped, either due to a network close, a
   // network error, or a protocol error. This may or may not be preceeded by a
   // call to OnClosingHandshake().
+  // |fail| is true if the user agent is required to fail the connection.

[Adam Rice, 2013/10/23 03:18:06]

I have just realised we may need more information than just a boolean parameter.
From
http://www.whatwg.org/specs/web-apps/current-work/multipage/network.html#conc...
:

"""
2. If the user agent was required to fail the WebSocket connection or the
WebSocket connection is closed with prejudice, fire a simple event named error
at the WebSocket object. [WSP]

3. Create a trusted event that uses the CloseEvent interface, with the event
type close, which does not bubble, is not cancelable, has no default action,
whose wasClean attribute is initialized to true if the connection closed cleanly
and false otherwise
"""

I see three questions:

1. Was the user agent required to fail the WebSocket connection?
2. Was the WebSocket connection closed with prejudice?
3. Did the connection close cleanly?

I am not sure whether case 1. only applies to requirements from the WHATWG spec
or also includes requirements from RFC6455.

I have not been able to find a definition for "with prejudice" as mentioned in
case 2.

If case 1 or 2 are true, then case 3 is definitely true. However, I think case 3
can be true without cases 1 or 2 being true.

[tyoshino (SeeGerritForStatus), 2013/10/23 05:21:34]

I think yes. But basically we should focus on keeping the new stack behave
consistently to the old stack.
An error event should be fired where "must close the ... with prejudice" is
used.

In other words, those sentences are defining what "with prejudice" means.

[yhirano, 2013/10/23 05:42:23]

You are right.
IIUC "close with prejudice" occurs in the renderer and we don't have to care
about it here.

I added "NotifyFailure" message to websocket_message.h.

   //
   // Warning: Both the |code| and |reason| are passed through to Javascript, so
   // callers must take care not to provide details that could be useful to
@@ -62,7 +63,9 @@ class NET_EXPORT WebSocketEventInterface {
   // It is not safe to delete the WebSocketChannel from within this
   // callback. It is recommended to delete the channel after returning to the
   // event loop.
-  virtual void OnDropChannel(uint16 code, const std::string& reason) = 0;
+  virtual void OnDropChannel(bool fail,
+                             uint16 code,
+                             const std::string& reason) = 0;
 
  protected:
   WebSocketEventInterface() {}