Use GpuMemoryBufferVideoFramePool for WebMediaPlayerMS and MediaStreamVideoRendererSink

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 ce8141951a33d01c60ae6ff4b320cd437e6a992f..c375b0c912e211ce7d8c425be19e2b6d405b41c1 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -4735,14 +4735,18 @@ WebMediaPlayer* RenderFrameImpl::CreateWebMediaPlayerForMediaStream(
       (android_getCpuFeatures() & ANDROID_CPU_ARM_FEATURE_NEON) != 0;
   UMA_HISTOGRAM_BOOLEAN("Platform.WebRtcNEONFound", found_neon);
 #endif  // defined(OS_ANDROID) && defined(ARCH_CPU_ARMEL)
-  scoped_refptr<base::SingleThreadTaskRunner>  compositor_task_runner =
-      RenderThreadImpl::current()->compositor_task_runner();
+  RenderThreadImpl* render_thread = RenderThreadImpl::current();

[mcasas, 2015/10/02 19:30:46]

Gosh more static methods! Is this OK with unit tests...? Because there
are unit tests for it, right?
At least DCHECK(render_thread), TODO() etc :)

[emircan, 2015/10/03 00:11:26]

Well the earlier usage on l.4739 worked, so I am assuming it works Ok now. I
will work on additional unit tests once dcastgan@ gives approval on the overall
design and signatures.

+
+  scoped_refptr<base::SingleThreadTaskRunner> compositor_task_runner =
+      render_thread->compositor_task_runner();
   if (!compositor_task_runner.get())
     compositor_task_runner = base::MessageLoop::current()->task_runner();
+
   return new WebMediaPlayerMS(
       frame_, client, weak_factory_.GetWeakPtr(), new RenderMediaLog(),
-      CreateRendererFactory(),
-      compositor_task_runner);
+      CreateRendererFactory(), compositor_task_runner,
+      render_thread->GetMediaThreadTaskRunner(),
+      render_thread->GetWorkerTaskRunner(), render_thread->GetGpuFactories());
 #else
   return NULL;
 #endif  // defined(ENABLE_WEBRTC)