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

Unified Diff: chromecast/renderer/media/chromecast_media_renderer_factory.cc

Issue 1372393007: [Chromecast] Upgrade to new CMA backend API (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Remove unnecessary abstract base classes. Created 5 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: chromecast/renderer/media/chromecast_media_renderer_factory.cc
diff --git a/chromecast/renderer/media/chromecast_media_renderer_factory.cc b/chromecast/renderer/media/chromecast_media_renderer_factory.cc
index 3a43e917b14b47212684cd80ae93efb8793cdd87..5acf69fa2e4169a3026b761d01f65f7fa9b87ea0 100644
--- a/chromecast/renderer/media/chromecast_media_renderer_factory.cc
+++ b/chromecast/renderer/media/chromecast_media_renderer_factory.cc
@@ -59,11 +59,10 @@ scoped_ptr<::media::Renderer> ChromecastMediaRendererFactory::CreateRenderer(
DCHECK(default_render_factory_);
// TODO(erickung): crbug.com/443956. Need to provide right LoadType.
LoadType cma_load_type = kLoadTypeMediaSource;
- scoped_ptr<MediaPipeline> cma_media_pipeline(
- new MediaPipelineProxy(
- render_frame_id_,
- content::RenderThread::Get()->GetIOMessageLoopProxy(),
- cma_load_type));
+ scoped_ptr<MediaPipelineProxy> cma_media_pipeline(new MediaPipelineProxy(
+ render_frame_id_,
+ content::RenderThread::Get()->GetIOMessageLoopProxy(),
+ cma_load_type));
scoped_ptr<CmaRenderer> cma_renderer(new CmaRenderer(
cma_media_pipeline.Pass(), video_renderer_sink, gpu_factories_));
scoped_ptr<::media::Renderer> default_media_render(
slan 2015/10/07 20:56:16 nit: default_media_renderer
kmackay 2015/10/09 20:35:40 Done.

Powered by Google App Engine
This is Rietveld 408576698