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

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

Issue 1873513003: Add video-rendering to mojo media pipeline. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: export VideoOverlayFactory Created 4 years, 7 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/cma_renderer.cc
diff --git a/chromecast/renderer/media/cma_renderer.cc b/chromecast/renderer/media/cma_renderer.cc
index 636b2f783bbafd9989a9f043ff63853c99e068c5..2db1906e6ee76932645ed0d687921f0f13db1136 100644
--- a/chromecast/renderer/media/cma_renderer.cc
+++ b/chromecast/renderer/media/cma_renderer.cc
@@ -20,7 +20,6 @@
#include "chromecast/media/cma/pipeline/media_pipeline_client.h"
#include "chromecast/media/cma/pipeline/video_pipeline_client.h"
#include "chromecast/renderer/media/audio_pipeline_proxy.h"
-#include "chromecast/renderer/media/hole_frame_factory.h"
#include "chromecast/renderer/media/media_pipeline_proxy.h"
#include "chromecast/renderer/media/video_pipeline_proxy.h"
#include "media/base/bind_to_current_loop.h"
@@ -29,7 +28,7 @@
#include "media/base/renderer_client.h"
#include "media/base/time_delta_interpolator.h"
#include "media/base/video_renderer_sink.h"
-#include "media/renderers/gpu_video_accelerator_factories.h"
+#include "media/renderers/video_overlay_factory.h"
#include "ui/gfx/geometry/size.h"
namespace chromecast {
@@ -95,7 +94,8 @@ void CmaRenderer::Initialize(
demuxer_stream_provider->GetStream(::media::DemuxerStream::VIDEO));
// Deferred from ctor so as to initialise on correct thread.
- hole_frame_factory_.reset(new HoleFrameFactory(gpu_factories_));
+ video_overlay_factory_.reset(
+ new ::media::VideoOverlayFactory(gpu_factories_));
BeginStateTransition();
@@ -379,7 +379,7 @@ void CmaRenderer::OnStatisticsUpdated(
void CmaRenderer::OnNaturalSizeChanged(const gfx::Size& size) {
DCHECK(thread_checker_.CalledOnValidThread());
video_renderer_sink_->PaintFrameUsingOldRenderingPath(
- hole_frame_factory_->CreateHoleFrame(size));
+ video_overlay_factory_->CreateFrame(size));
client_->OnVideoNaturalSizeChange(size);
}

Powered by Google App Engine
This is Rietveld 408576698