Switch from a DataPipe per DecoderBuffer to a single one.

media/mojo/interfaces/demuxer_stream.mojom

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 module mojo;
 
 import "media/mojo/interfaces/media_types.mojom";
 
 // DemuxerStream is modeled after media::DemuxerStream using mojo in order to
 // enable proxying between a media::Pipeline and media::Renderer living in two
@@ skipping 12 matching lines @@
   enum Status {
     OK = 0,
     ABORTED,
     CONFIG_CHANGED,
   };
 
   // Request a MediaDecoderBuffer from this stream for decoding and rendering.
   // When available, the callback will be invoked with a Status and |response|
   // buffer.  See media::DemuxerStream::ReadCB for explanation of fields.
   //
-  // TODO(tim): Remove this method in favor of initializing the
-  // DemuxerStreamClient with a DataPipeConsumerHandle once we have a framed
-  // DataPipe that we can serialize [|status| | response|]* over directly.
+  // TODO(dalecurtis): Remove this method in favor of serializing everything
+  // into the DataPipe given to OnStreamReady() once DataPipe supports framed
+  // data in a nicer fashion.
   Read() => (Status status, MediaDecoderBuffer response);
 };
 
 interface DemuxerStreamClient {
   // Informs the client that the stream is ready for reading.  If |pipe| is

[xhwang, 2014/12/06 00:03:44]

pipe muse be present now

[DaleCurtis, 2014/12/06 00:42:59]

Done.

   // present, it means the client should read
   //
-  //     [ |DemuxerStream::Status| |MediaDecoderBuffer| ]
+  //     [ |media::DecoderBuffer::data()| ]
   //
   // payloads from the DataPipe directly.  If |pipe| is NULL, it means the

[xhwang, 2014/12/06 00:03:44]

pipe cannot be null now.

[DaleCurtis, 2014/12/06 00:42:59]

Done.

   // client needs to use DemuxerStream::Read() directly to obtain buffers.
-  OnStreamReady(handle<data_pipe_consumer>? pipe);
+  OnStreamReady(handle<data_pipe_consumer> pipe);
 
   // A new AudioDecoderConfig is available. Will be sent by the DemuxerStream
   // whenever a DemuxerStream::STATUS_CONFIG_CHANGED is observed (either
   // in a Read() callback or over the DataPipe).
   OnAudioDecoderConfigChanged(AudioDecoderConfig config);
 
   // A new VideoDecoderConfig is available. Will be sent by the DemuxerStream
   // whenever a DemuxerStream::STATUS_CONFIG_CHANGED is observed (either
   // in a Read() callback or over the DataPipe).
   OnVideoDecoderConfigChanged(VideoDecoderConfig config);
 };