Adding a skeleton MediaStreamCenter.

content/renderer/renderer_webkitplatformsupport_impl.cc

 // 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.
 
 #include "content/renderer/renderer_webkitplatformsupport_impl.h"
 
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/platform_file.h"
@@ skipping 654 matching lines @@
 void RendererWebKitPlatformSupportImpl::GetPlugins(
     bool refresh, std::vector<webkit::WebPluginInfo>* plugins) {
   if (!RenderThreadImpl::current()->plugin_refresh_allowed())
     refresh = false;
   RenderThreadImpl::current()->Send(
       new ViewHostMsg_GetPlugins(refresh, plugins));
 }
 
 //------------------------------------------------------------------------------
 
+namespace {
+
+RenderViewImpl* findRenderView() {
+  WebFrame* web_frame = WebFrame::frameForCurrentContext();
+  if (!web_frame)
+    return NULL;
+  return RenderViewImpl::FromWebView(web_frame->view());
+}
+
+}
+
 WebKit::WebPeerConnectionHandler*
 RendererWebKitPlatformSupportImpl::createPeerConnectionHandler(
     WebKit::WebPeerConnectionHandlerClient* client) {
-  WebFrame* web_frame = WebFrame::frameForCurrentContext();
-  if (!web_frame)
-    return NULL;
-  RenderViewImpl* render_view = RenderViewImpl::FromWebView(web_frame->view());
+  RenderViewImpl* render_view = findRenderView();
   if (!render_view)
     return NULL;
   return render_view->CreatePeerConnectionHandler(client);
 }
+
+WebKit::WebMediaStreamCenter*
+RendererWebKitPlatformSupportImpl::createMediaStreamCenter(
+    WebKit::WebMediaStreamCenterClient* client) {
+  RenderViewImpl* render_view = findRenderView();

[piman, 2012/02/08 01:52:24]

Is there any way we can avoid this? This is taking the render view from the
javascript context, which overall seems iffy. Is it possible to create the
WebMediaStreamCenter through the WebViewClient on the webkit side, that goes
directly to RenderViewImpl.

[tommyw, 2012/02/08 13:14:18]

Short answer: No

Longer answer: the MediaStreamCenter is not a page client, it lives in the
WebCore platform layer. Therefore we can't do what you suggest.

On 2012/02/08 01:52:24, piman wrote:

+  if (!render_view)
+    return NULL;
+  return render_view->CreateMediaStreamCenter(client);
+}