Switch from a DataPipe per DecoderBuffer to a single one.

media/mojo/interfaces/media_types.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 "mojo/services/public/interfaces/geometry/geometry.mojom";
 
 // See media/base/buffering_state.h for descriptions.
 // Kept in sync with media::BufferingState via COMPILE_ASSERTs.
@@ skipping 172 matching lines @@
   string iv;
   array<SubsampleEntry> subsamples;
 };
 
 // This defines a mojo transport format for media::DecoderBuffer.
 struct MediaDecoderBuffer {
   // See media/base/buffers.h for details.
   int64 timestamp_usec;
   int64 duration_usec;
 
   // The number of bytes in |data|.

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

|data| doesn't exist now. Add a comment that the |data| is in a separate
DataPipe associated with this stream etc.

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

Done.

   uint32 data_size;
 
   // Indicates whether or not this buffer is a random access point.
   bool is_key_frame;
 
   // This is backed by an std::vector and results in a few copies.
   // Into the vector, onto and off the MessagePipe, back into a vector.
   array<uint8>? side_data;
   uint32 side_data_size;
 
   // DecryptConfig for a encrypted buffer. NULL if the buffer is not encrypted.
   DecryptConfig? decrypt_config;
 
   // These fields indicate the amount of data to discard after decoding.
   int64 front_discard_usec;
   int64 back_discard_usec;
 
   // Indicates this buffer is part of a splice around |splice_timestamp_usec|.
   int64 splice_timestamp_usec;
-
-  // The payload. Invalid handle indicates an end-of-stream (EOS) buffer.
-  // TODO(tim): This currently results in allocating a new, largeish DataPipe
-  // for each buffer. Remove this once framed data pipes exist, but using this
-  // for now for prototyping audio.
-  handle<data_pipe_consumer>? data;
 };