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

Unified Diff: chromecast/browser/media/cast_renderer.cc

Issue 1873513003: Add video-rendering to mojo media pipeline. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed comments Created 4 years, 8 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/browser/media/cast_renderer.cc
diff --git a/chromecast/browser/media/cast_renderer.cc b/chromecast/browser/media/cast_renderer.cc
index a6d3c3a293deee95d3dcc2396c4d095c577a35f6..f267242f3e2dbc8f4bda1972f1053e07962a12ac 100644
--- a/chromecast/browser/media/cast_renderer.cc
+++ b/chromecast/browser/media/cast_renderer.cc
@@ -47,6 +47,7 @@ void CastRenderer::Initialize(
const ::media::PipelineStatusCB& init_cb,
const ::media::StatisticsCB& statistics_cb,
const ::media::BufferingStateCB& buffering_state_cb,
+ const ::media::NaturalSizeChangedCB& natural_size_changed_cb,
const base::Closure& ended_cb,
const ::media::PipelineStatusCB& error_cb,
const base::Closure& waiting_for_decryption_key_cb) {
@@ -100,13 +101,13 @@ void CastRenderer::Initialize(
demuxer_stream_provider->GetStream(::media::DemuxerStream::VIDEO);
if (video_stream) {
VideoPipelineClient video_client;
- // TODO(alokp): Set VideoPipelineClient::natural_size_changed_cb.
video_client.av_pipeline_client.wait_for_key_cb =
waiting_for_decryption_key_cb;
video_client.av_pipeline_client.eos_cb =
base::Bind(&CastRenderer::OnEos, base::Unretained(this), STREAM_VIDEO);
video_client.av_pipeline_client.playback_error_cb = error_cb;
video_client.av_pipeline_client.statistics_cb = statistics_cb;
+ video_client.natural_size_changed_cb = natural_size_changed_cb;
// TODO(alokp): Change MediaPipelineImpl API to accept a single config
// after CmaRenderer is deprecated.
std::vector<::media::VideoDecoderConfig> video_configs;

Powered by Google App Engine
This is Rietveld 408576698