Bind RenderViewImpl routing_id to SocketStreamHost

content/renderer/websocketstreamhandle_impl_extradata_impl.h

+// 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.
+
+#ifndef CONTENT_RENDERER_WEBSOCKETSTREAMHANDLE_IMPL_EXTRADATA_IMPL_H_
+#define CONTENT_RENDERER_WEBSOCKETSTREAMHANDLE_IMPL_EXTRADATA_IMPL_H_
+#pragma once
+
+#include "webkit/glue/websocketstreamhandle_impl.h"
+
+// Chrome's implementation of the "extra data" stored in each
+// WebSocketStreamHandleImpl.
+class WebSocketStreamHandleImplExtraDataImpl

[darin (slow to review), 2012/03/19 20:58:28]

nit: use a shorter name here?

How about SocketHandleData?

[Takashi Toyoshima, 2012/03/19 22:58:44]

OK.
I know that *SocketHandle exist in net/socket. These are not WebSocket specific
classes but more generic one. So SocketHandleData could be confusing. How about
SocketStreamHandleData. It's longer than your proposal. But, SocketStream is a
good keyword for WebSocket.

BTW, should I move this file to content/common/?
content/common/socket_stream_dispatcher has a dependency on this file, and other
files in content/common looks having no dependency on content/renderer.

+    : public webkit_glue::WebSocketStreamHandleImpl::ExtraData {
+ public:
+  explicit WebSocketStreamHandleImplExtraDataImpl(int render_view_id)
+      : render_view_id_(render_view_id) {
+  }
+  virtual ~WebSocketStreamHandleImplExtraDataImpl() {}
+
+  int render_view_id() const { return render_view_id_; }
+
+ private:
+  int render_view_id_;
+
+  DISALLOW_COPY_AND_ASSIGN(WebSocketStreamHandleImplExtraDataImpl);
+};
+
+#endif  // CONTENT_RENDERER_WEBSOCKETSTREAMHANDLE_IMPL_EXTRADATA_IMPL_H_