[WebSocket] Make subprotocol and extensions live in WebSocket

Source/web/WebSocketImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 67 matching lines @@
     return true;
 }
 
 void WebSocketImpl::connect(const WebURL& url, const WebString& protocol)
 {
     m_private->connect(url, protocol);
 }
 
 WebString WebSocketImpl::subprotocol()
 {
-    return m_private->subprotocol();
+    return m_subprotocol;
 }
 
 WebString WebSocketImpl::extensions()
 {
-    return m_private->extensions();
+    return m_extensions;
 }
 
 bool WebSocketImpl::sendText(const WebString& message)
 {
     return m_private->send(message) == WebSocketChannel::SendSuccess;
 }
 
 bool WebSocketImpl::sendArrayBuffer(const WebArrayBuffer& webArrayBuffer)
 {
     return m_private->send(*PassRefPtr<ArrayBuffer>(webArrayBuffer), 0, webArrayBuffer.byteLength()) == WebSocketChannel::SendSuccess;
@@ skipping 13 matching lines @@
 {
     m_private->fail(reason, ErrorMessageLevel, String(), 0);
 }
 
 void WebSocketImpl::disconnect()
 {
     m_private->disconnect();
     m_client = 0;
 }
 
-void WebSocketImpl::didConnect()
+void WebSocketImpl::didConnect(const String& subprotocol, const String& extensions)
 {
     m_client->didConnect();
+    m_subprotocol = subprotocol;
+    m_extensions = extensions;

[tyoshino (SeeGerritForStatus), 2014/06/13 09:00:12]

let's set variables first, and then didConnect()

[yhirano, 2014/06/13 09:04:03]

Done.

 }
 
 void WebSocketImpl::didReceiveMessage(const String& message)
 {
     m_client->didReceiveMessage(WebString(message));
 }
 
 void WebSocketImpl::didReceiveBinaryData(PassOwnPtr<Vector<char> > binaryData)
 {
     switch (m_binaryType) {
@@ skipping 20 matching lines @@
 {
     m_client->didStartClosingHandshake();
 }
 
 void WebSocketImpl::didClose(unsigned long bufferedAmount, ClosingHandshakeCompletionStatus status, unsigned short code, const String& reason)
 {
     m_client->didClose(bufferedAmount, static_cast<WebSocketClient::ClosingHandshakeCompletionStatus>(status), code, WebString(reason));
 }
 
 } // namespace blink