Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(669)

Unified Diff: content/renderer/render_thread_impl.cc

Issue 27420004: Remove threading from RendererGpuVideoAcceleratorFactories (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@dthread
Patch Set: f77fc38a Unittest fixes. Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 775b6bf5c861110e0bf9995a8f83be2c2ea382f1..1ceb29b991a688706b2b08190c3fdad91888192d 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -874,6 +874,8 @@ RenderThreadImpl::GetGpuFactories() {
scoped_refptr<GpuChannelHost> gpu_channel_host = GetGpuChannel();
const CommandLine* cmd_line = CommandLine::ForCurrentProcess();
scoped_refptr<RendererGpuVideoAcceleratorFactories> gpu_factories;
+ scoped_refptr<base::MessageLoopProxy> media_loop_proxy =
+ GetMediaThreadMessageLoopProxy();
if (!cmd_line->HasSwitch(switches::kDisableAcceleratedVideoDecode)) {
if (!gpu_va_context_provider_ ||
gpu_va_context_provider_->DestroyedOnMainThread()) {
@@ -888,11 +890,14 @@ RenderThreadImpl::GetGpuFactories() {
WebKit::WebGraphicsContext3D::Attributes(),
GURL("chrome://gpu/RenderThreadImpl::GetGpuVDAContext3D"))),
"GPU-VideoAccelerator-Offscreen");
+ media_loop_proxy->PostTask(FROM_HERE, base::Bind(
+ base::IgnoreResult(&cc::ContextProvider::BindToCurrentThread),
Ami GONE FROM CHROMIUM 2013/10/26 19:30:26 I wonder if that IgnoreResult should be a CheckOnF
+ gpu_va_context_provider_));
}
}
if (gpu_channel_host) {
gpu_factories = new RendererGpuVideoAcceleratorFactories(
- gpu_channel_host.get(), gpu_va_context_provider_);
+ gpu_channel_host, media_loop_proxy, gpu_va_context_provider_);
}
return gpu_factories;
}

Powered by Google App Engine
This is Rietveld 408576698