Make mojo renderer capable of supporting multiple streams/tracks

media/mojo/services/demuxer_stream_provider_shim.cc

Index: media/mojo/services/demuxer_stream_provider_shim.cc
diff --git a/media/mojo/services/demuxer_stream_provider_shim.cc b/media/mojo/services/demuxer_stream_provider_shim.cc
index 0c9d8fea64f70708b9d9e88ded2be08ea32527d4..6000b60131f8bd5d62dfacc56a85f1abb7fc4568 100644
--- a/media/mojo/services/demuxer_stream_provider_shim.cc
+++ b/media/mojo/services/demuxer_stream_provider_shim.cc
@@ -13,36 +13,31 @@
 namespace media {
 
 DemuxerStreamProviderShim::DemuxerStreamProviderShim(
-    mojom::DemuxerStreamPtr audio,
-    mojom::DemuxerStreamPtr video,
+    std::vector<mojom::DemuxerStreamPtr> streams,
     const base::Closure& demuxer_ready_cb)
     : demuxer_ready_cb_(demuxer_ready_cb),
       streams_ready_(0),
       weak_factory_(this) {
-  DCHECK(audio || video);
+  DCHECK(!streams.empty());
   DCHECK(!demuxer_ready_cb_.is_null());
 
-  if (audio) {
-    streams_.push_back(new MojoDemuxerStreamAdapter(
-        std::move(audio), base::Bind(&DemuxerStreamProviderShim::OnStreamReady,
-                                     weak_factory_.GetWeakPtr())));
-  }
-
-  if (video) {
-    streams_.push_back(new MojoDemuxerStreamAdapter(
-        std::move(video), base::Bind(&DemuxerStreamProviderShim::OnStreamReady,
-                                     weak_factory_.GetWeakPtr())));
+  for (auto& s : streams) {
+    streams_.emplace_back(new MojoDemuxerStreamAdapter(
+        std::move(s), base::Bind(&DemuxerStreamProviderShim::OnStreamReady,
+                                 weak_factory_.GetWeakPtr())));
   }
 }
 
 DemuxerStreamProviderShim::~DemuxerStreamProviderShim() {
 }
 
+// This function returns only the first stream of the given |type| for now.
+// TODO: Make this work with multiple streams.

[xhwang, 2017/01/31 07:49:07]

nit: TODO(ldap)

[servolk, 2017/01/31 17:38:05]

Done.

 DemuxerStream* DemuxerStreamProviderShim::GetStream(DemuxerStream::Type type) {
   DCHECK(demuxer_ready_cb_.is_null());
-  for (auto* stream : streams_) {
+  for (auto& stream : streams_) {
     if (stream->type() == type)
-      return stream;
+      return stream.get();
   }
 
   return nullptr;