Mojo: WebSocket interface now reuses the DataPipe for subsequent sends or

mojo/services/public/interfaces/network/web_socket.mojom

Index: mojo/services/public/interfaces/network/web_socket.mojom
diff --git a/mojo/services/public/interfaces/network/web_socket.mojom b/mojo/services/public/interfaces/network/web_socket.mojom
index 87f0d6c055b7b3d642167fa0dd766e3dd7081f5f..af32ac3edc75eee1481e74ae3131799ec0d77896 100644
--- a/mojo/services/public/interfaces/network/web_socket.mojom
+++ b/mojo/services/public/interfaces/network/web_socket.mojom
@@ -14,17 +14,22 @@ interface WebSocket {
   };
   const uint16 kAbnormalCloseCode = 1006;  // stolen from websocket_bridge
 
-  Connect(
-      string url, string[] protocols, string origin, WebSocketClient client);
-  Send(bool fin, MessageType type, handle<data_pipe_consumer> data);
+  Connect(string url,

[darin (slow to review), 2014/09/19 05:40:08]

nit: please document the protocol here. it is not immediately obvious how
send_stream and the Send method are related. the other methods really could use
some documentation as well.

[Matt Perry, 2014/09/19 19:50:45]

I documented Send/Receive/Connect and explained the data pipes. I didn't
document the other methods because I don't actually know what they do (beyond
the obvious) and couldn't find any documentation.

+          string[] protocols,
+          string origin,
+          handle<data_pipe_consumer> send_stream,
+          WebSocketClient client);
+  Send(bool fin, MessageType type, uint32 num_bytes);
   FlowControl(int64 quota);
   Close(uint16 code, string reason);
 };
 
 interface WebSocketClient {
-  DidConnect(bool fail, string selected_subprotocol, string extensions);
-  DidReceiveData(
-      bool fin, WebSocket.MessageType type, handle<data_pipe_consumer> data);
+  DidConnect(bool fail,
+             string selected_subprotocol,
+             string extensions,
+             handle<data_pipe_consumer> receive_stream);
+  DidReceiveData(bool fin, WebSocket.MessageType type, uint32 num_bytes);
   DidReceiveFlowControl(int64 quota);
   DidFail(string message);
   DidClose(bool was_clean, uint16 code, string reason);