Media Remoting: Data/Control plumbing between renderer and Media Router.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index cefdf2ba5537e1728dde9898f84d1cdb75309bea..f0895410216cb22f3e92fd127aa05fb9aef5cf58 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -2633,6 +2633,10 @@ blink::WebMediaPlayer* RenderFrameImpl::createMediaPlayer(
                      base::Unretained(render_thread)),
           GetMediaInterfaceProvider()));
 #else
+  // TODO(miu): In future change, call GetRemoterFactory()->Create() and pass
+  // the media::mojom::RemoterPtr to the media::RendererFactory here so that it
+  // can decide whether to create local media::Renderers or "remoted"
+  // media::Renderers.

[xjz, 2016/09/09 17:38:48]

This seems different from our current design. But we'll find it out later.  :)

[miu, 2016/09/11 00:25:22]

Updated and moved this TODO comment.

   std::unique_ptr<media::RendererFactory> media_renderer_factory =
       GetContentClient()->renderer()->CreateMediaRendererFactory(
           this, render_thread->GetGpuFactories(), media_log);
@@ -6139,6 +6143,12 @@ bool RenderFrameImpl::AreSecureCodecsSupported() {
 #endif  // defined(OS_ANDROID)
 }
 
+media::mojom::RemoterFactory* RenderFrameImpl::GetRemoterFactory() {
+  if (!remoter_factory_)
+    GetRemoteInterfaces()->GetInterface(&remoter_factory_);
+  return remoter_factory_.get();
+}
+
 media::CdmFactory* RenderFrameImpl::GetCdmFactory() {
   if (cdm_factory_)
     return cdm_factory_.get();