Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(926)

Unified Diff: media/base/decoder_buffer.cc

Issue 760523008: Switch from a DataPipe per DecoderBuffer to a single one. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Fix. Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/decoder_buffer.cc
diff --git a/media/base/decoder_buffer.cc b/media/base/decoder_buffer.cc
index 0711268740ec7b12fe0cf9fe1a3d4577828f7340..2a80c623d6a964e5f3d01d735d79b029fdfb809f 100644
--- a/media/base/decoder_buffer.cc
+++ b/media/base/decoder_buffer.cc
@@ -104,4 +104,18 @@ void DecoderBuffer::set_timestamp(base::TimeDelta timestamp) {
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 AlignedAll
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

Powered by Google App Engine
This is Rietveld 408576698