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

Unified Diff: media/remoting/courier_renderer.cc

Issue 2491043003: MediaResource refactoring to support multiple streams (Closed)
Patch Set: Added a TODO about DemuxerStream enabled/set_enabled methods Created 3 years, 10 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
« no previous file with comments | « media/mojo/services/mojo_demuxer_stream_adapter.cc ('k') | media/remoting/fake_media_resource.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/remoting/courier_renderer.cc
diff --git a/media/remoting/courier_renderer.cc b/media/remoting/courier_renderer.cc
index 4722f4b9159bdec30c89e0b418b3516bd78a39fb..56e4a1841627dd5922083fa518d78698622d79ef 100644
--- a/media/remoting/courier_renderer.cc
+++ b/media/remoting/courier_renderer.cc
@@ -124,17 +124,20 @@ void CourierRenderer::Initialize(MediaResource* media_resource,
state_ = STATE_CREATE_PIPE;
+ // TODO(servolk): Add support for multiple streams. For now use the first
+ // enabled audio and video streams to preserve the existing behavior.
+ ::media::DemuxerStream* audio_demuxer_stream =
+ media_resource_->GetFirstStream(DemuxerStream::AUDIO);
+ ::media::DemuxerStream* video_demuxer_stream =
+ media_resource_->GetFirstStream(DemuxerStream::VIDEO);
+
// Create audio mojo data pipe handles if audio is available.
- DemuxerStream* audio_demuxer_stream =
- media_resource_->GetStream(DemuxerStream::AUDIO);
std::unique_ptr<mojo::DataPipe> audio_data_pipe;
if (audio_demuxer_stream) {
audio_data_pipe = base::WrapUnique(DemuxerStreamAdapter::CreateDataPipe());
}
// Create video mojo data pipe handles if video is available.
- DemuxerStream* video_demuxer_stream =
- media_resource_->GetStream(DemuxerStream::VIDEO);
std::unique_ptr<mojo::DataPipe> video_data_pipe;
if (video_demuxer_stream) {
video_data_pipe = base::WrapUnique(DemuxerStreamAdapter::CreateDataPipe());
@@ -324,9 +327,14 @@ void CourierRenderer::OnDataPipeCreated(
DCHECK_EQ(state_, STATE_CREATE_PIPE);
DCHECK(!init_workflow_done_callback_.is_null());
+ // TODO(servolk): Add support for multiple streams. For now use the first
+ // enabled audio and video streams to preserve the existing behavior.
+ ::media::DemuxerStream* audio_demuxer_stream =
+ media_resource_->GetFirstStream(DemuxerStream::AUDIO);
+ ::media::DemuxerStream* video_demuxer_stream =
+ media_resource_->GetFirstStream(DemuxerStream::VIDEO);
+
// Create audio demuxer stream adapter if audio is available.
- DemuxerStream* audio_demuxer_stream =
- media_resource_->GetStream(DemuxerStream::AUDIO);
if (audio_demuxer_stream && audio.is_valid() && audio_handle.is_valid() &&
audio_rpc_handle != RpcBroker::kInvalidHandle) {
VLOG(2) << "Initialize audio";
@@ -338,8 +346,6 @@ void CourierRenderer::OnDataPipeCreated(
}
// Create video demuxer stream adapter if video is available.
- DemuxerStream* video_demuxer_stream =
- media_resource_->GetStream(DemuxerStream::VIDEO);
if (video_demuxer_stream && video.is_valid() && video_handle.is_valid() &&
video_rpc_handle != RpcBroker::kInvalidHandle) {
VLOG(2) << "Initialize video";
« no previous file with comments | « media/mojo/services/mojo_demuxer_stream_adapter.cc ('k') | media/remoting/fake_media_resource.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698