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

Unified Diff: content/renderer/media/media_stream_renderer_factory_impl.cc

Issue 2472273002: Move passing of WebRTC rendering frames from main thread to compositor thread (Closed)
Patch Set: wez@ nits. Created 4 years, 1 month 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/media/media_stream_renderer_factory_impl.cc
diff --git a/content/renderer/media/media_stream_renderer_factory_impl.cc b/content/renderer/media/media_stream_renderer_factory_impl.cc
index 831057982a8219051d89695b884e173869440a62..5da4fcb3ad4f0d10a615adeeccd261f7afbb2945 100644
--- a/content/renderer/media/media_stream_renderer_factory_impl.cc
+++ b/content/renderer/media/media_stream_renderer_factory_impl.cc
@@ -61,6 +61,7 @@ MediaStreamRendererFactoryImpl::GetVideoRenderer(
const blink::WebMediaStream& web_stream,
const base::Closure& error_cb,
const MediaStreamVideoRenderer::RepaintCB& repaint_cb,
+ const scoped_refptr<base::SingleThreadTaskRunner>& compositor_task_runner,
const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
const scoped_refptr<base::TaskRunner>& worker_task_runner,
media::GpuVideoAcceleratorFactories* gpu_factories) {
@@ -76,9 +77,9 @@ MediaStreamRendererFactoryImpl::GetVideoRenderer(
return NULL;
}
- return new MediaStreamVideoRendererSink(video_tracks[0], error_cb, repaint_cb,
- media_task_runner, worker_task_runner,
- gpu_factories);
+ return new MediaStreamVideoRendererSink(
+ video_tracks[0], error_cb, repaint_cb, compositor_task_runner,
+ media_task_runner, worker_task_runner, gpu_factories);
}
scoped_refptr<MediaStreamAudioRenderer>

Powered by Google App Engine
This is Rietveld 408576698