Switch from a DataPipe per DecoderBuffer to a single one.

media/base/decoder_buffer.cc

 // Copyright (c) 2012 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.
 
 #include "media/base/decoder_buffer.h"
 
 #include "base/logging.h"
 #include "media/base/buffers.h"
 #include "media/base/decrypt_config.h"
 
@@ skipping 86 matching lines @@
     << " discard_padding (ms): (" << discard_padding_.first.InMilliseconds()
     << ", " << discard_padding_.second.InMilliseconds() << ")";
   return s.str();
 }
 
 void DecoderBuffer::set_timestamp(base::TimeDelta timestamp) {
   DCHECK(!end_of_stream());
   timestamp_ = timestamp;
 }
 
+void DecoderBuffer::CopySideDataFrom(const uint8* side_data,
+                                     int side_data_size) {
+  if (side_data_size > 0) {
+    side_data_.reset(reinterpret_cast<uint8*>(
+        base::AlignedAlloc(side_data_size_ + kPaddingSize, kAlignmentSize)));

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

Can we have a centralized place to call AlignedAlloc for side data? We are doing
the same on l.46.

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

Done.

+    side_data_size_ = side_data_size;
+    memcpy(side_data_.get(), side_data, side_data_size_);
+    memset(side_data_.get() + side_data_size_, 0, kPaddingSize);
+  } else {
+    side_data_.reset();
+    side_data_size_ = 0;
+  }
+}
+
 }  // namespace media