Use GpuMemoryBufferVideoFramePool for WebMediaPlayerMS and MediaStreamVideoRendererSink

content/renderer/render_frame_impl.cc

 // Copyright 2013 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/render_frame_impl.h"
 
 #include <map>
 #include <string>
 
 #include "base/auto_reset.h"
@@ skipping 4713 matching lines @@
       this,
       RenderThreadImpl::current()->GetPeerConnectionDependencyFactory(),
       make_scoped_ptr(new MediaStreamDispatcher(this)).Pass());
 #endif
 }
 
 WebMediaPlayer* RenderFrameImpl::CreateWebMediaPlayerForMediaStream(
     WebMediaPlayerClient* client) {
 #if defined(ENABLE_WEBRTC)
 #if defined(OS_ANDROID) && defined(ARCH_CPU_ARMEL)
   bool found_neon =

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

const :) :)

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

Done.

       (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)
 }
 
 scoped_ptr<MediaStreamRendererFactory>
 RenderFrameImpl::CreateRendererFactory() {
   scoped_ptr<MediaStreamRendererFactory> factory =
       GetContentClient()->renderer()->CreateMediaStreamRendererFactory();
   if (factory.get())
@@ skipping 349 matching lines @@
   mojo::ServiceProviderPtr service_provider;
   mojo::URLRequestPtr request(mojo::URLRequest::New());
   request->url = mojo::String::From(url);
   mojo_shell_->ConnectToApplication(request.Pass(), GetProxy(&service_provider),
                                     nullptr, nullptr,
                                     base::Bind(&OnGotContentHandlerID));
   return service_provider.Pass();
 }
 
 }  // namespace content