Make mojo renderer capable of supporting multiple streams/tracks

media/mojo/clients/mojo_renderer.h

Index: media/mojo/clients/mojo_renderer.h
diff --git a/media/mojo/clients/mojo_renderer.h b/media/mojo/clients/mojo_renderer.h
index 995a076368ae4106b4f791f8cc2bf81608edc996..221b9b1121441ed01a79f7ed8de6b5501d613a1b 100644
--- a/media/mojo/clients/mojo_renderer.h
+++ b/media/mojo/clients/mojo_renderer.h
@@ -96,8 +96,8 @@ class MojoRenderer : public Renderer, public mojom::RendererClient {
   // Callback for connection error on |remote_renderer_|.
   void OnConnectionError();
 
-  // Callback for connection error on |audio_stream_| and |video_stream_|.
-  void OnDemuxerStreamConnectionError(DemuxerStream::Type type);
+  // Callback for connection error on any of |streams_|.
+  void OnDemuxerStreamConnectionError(MojoDemuxerStreamImpl* stream);

[xhwang, 2017/01/31 00:15:21]

Add comment about what |stream| means in this API (e.g. the stream where the
error happened on..)

[servolk, 2017/01/31 01:10:40]

Done.

 
   // Callbacks for |remote_renderer_| methods.
   void OnInitialized(media::RendererClient* client, bool success);
@@ -131,8 +131,7 @@ class MojoRenderer : public Renderer, public mojom::RendererClient {
   // destroyed. The local demuxer streams returned by DemuxerStreamProvider
   // cannot be used after |this| is destroyed.
   // TODO(alokp): Add tests for MojoDemuxerStreamImpl.
-  std::unique_ptr<MojoDemuxerStreamImpl> audio_stream_;
-  std::unique_ptr<MojoDemuxerStreamImpl> video_stream_;
+  std::vector<std::unique_ptr<MojoDemuxerStreamImpl>> streams_;

[xhwang, 2017/01/31 00:15:21]

nit: include <vector> and <memory>

[servolk, 2017/01/31 01:10:40]

Done.

 
   // This class is constructed on one thread and used exclusively on another
   // thread. This member is used to safely pass the RendererPtr from one thread