Bind RenderViewImpl routing_id to SocketStreamHost

webkit/glue/websocketstreamhandle_impl.h

Index: webkit/glue/websocketstreamhandle_impl.h
diff --git a/webkit/glue/websocketstreamhandle_impl.h b/webkit/glue/websocketstreamhandle_impl.h
index 575cc6cbbf316bc9cef0b7b92fd1dff2050e95e5..26bdaf7a00ccdd6b69971ab0976f0aa486f72d99 100644
--- a/webkit/glue/websocketstreamhandle_impl.h
+++ b/webkit/glue/websocketstreamhandle_impl.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
@@ -6,6 +6,7 @@
 #define WEBKIT_GLUE_WEBSOCKETSTREAMHANDLE_IMPL_H_
 
 #include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebSocketStreamHandle.h"
 
 namespace webkit_glue {
@@ -20,14 +21,19 @@ class WebSocketStreamHandleImpl : public WebKit::WebSocketStreamHandle {
   // WebSocketStreamHandle methods:
   virtual void connect(
       const WebKit::WebURL& url,
-      WebKit::WebSocketStreamHandleClient* client);
-  virtual bool send(const WebKit::WebData& data);
-  virtual void close();
+      WebKit::WebSocketStreamHandleClient* client) OVERRIDE;
+  virtual bool send(const WebKit::WebData& data) OVERRIDE;
+  virtual void close() OVERRIDE;
+
+  // Handle RenderView routing id associated with the WebSocket object.

[jam, 2012/03/15 02:01:19]

nit: don't mention RenderView or routing_id's at the webkit layer, even in
comments

[Takashi Toyoshima, 2012/03/15 22:49:03]

Should I abstract them?
In the case of WebKit side change, Darin suggests ExtraData approach. I try that
approach here, too.

+  int RequestorId() const { return requestor_id_; };
+  void SetRequestorId(int id) { requestor_id_ = id; };
 
  private:
   class Context;
   scoped_refptr<Context> context_;
   WebKitPlatformSupportImpl* platform_;
+  int requestor_id_;
 
   DISALLOW_COPY_AND_ASSIGN(WebSocketStreamHandleImpl);
 };