Remove message loop parameter from RenderThreadImpl::GetGpuFactories()

RenderThreadImpl::GetGpuFactories() makes the assumption that all the
GpuVideoAcceleratorFactories retrieved from it can use the same 3D context.
This is only true if all the factories use the same message loop for the
3D context.  Ensure this by unconditionally using
RenderThreadImpl::GetMediaThreadMessageLoopProxy() for that message loop.

This change should be a functional no-op according to the current
architecture.

BUG=306333
TEST=local build, run on CrOS snow, build on desktop Linux

Review URL: https://codereview.chromium.org/27019011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229600 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 1 insertions, 5 deletions

diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 9cf66c3..be6f157 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -3153,17 +3153,13 @@ WebMediaPlayer* RenderViewImpl::createMediaPlayer(
     DVLOG(1) << "Using AudioRendererMixerManager-provided sink: " << sink.get();
   }
 
-  scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories =
-      RenderThreadImpl::current()->GetGpuFactories(
-          RenderThreadImpl::current()->GetMediaThreadMessageLoopProxy());
-
   WebMediaPlayerParams params(
       RenderThreadImpl::current()->GetMediaThreadMessageLoopProxy(),
       base::Bind(&ContentRendererClient::DeferMediaLoad,
                  base::Unretained(GetContentClient()->renderer()),
                  static_cast<RenderView*>(this)),
       sink,
-      gpu_factories,
+      RenderThreadImpl::current()->GetGpuFactories(),
       new RenderMediaLog());
   return new WebMediaPlayerImpl(frame, client, AsWeakPtr(), params);
 }