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

Unified Diff: media/base/decoder_buffer.cc

Issue 10447035: Introducing DecoderBuffer and general Buffer cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: s/2011/2012/ Created 8 years, 7 months 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
« no previous file with comments | « media/base/decoder_buffer.h ('k') | media/base/decoder_buffer_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/decoder_buffer.cc
diff --git a/media/base/decoder_buffer.cc b/media/base/decoder_buffer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..527fb65ec945c550fc673aca5501aed16ec79fe3
--- /dev/null
+++ b/media/base/decoder_buffer.cc
@@ -0,0 +1,87 @@
+// 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/decrypt_config.h"
+#if !defined(OS_ANDROID)
+#include "media/ffmpeg/ffmpeg_common.h"
+#endif
+
+namespace media {
+
+DecoderBuffer::DecoderBuffer(int buffer_size)
+ : Buffer(base::TimeDelta(), base::TimeDelta()),
+ buffer_size_(buffer_size) {
+ Initialize();
+}
+
+DecoderBuffer::DecoderBuffer(const uint8* data, int buffer_size)
+ : Buffer(base::TimeDelta(), base::TimeDelta()),
+ buffer_size_(buffer_size) {
+ // Prevent invalid allocations. Also used to create end of stream buffers.
+ if (!data) {
+ buffer_size_ = 0;
+ data_ = NULL;
+ return;
+ }
+
+ Initialize();
+ memcpy(data_, data, buffer_size_);
+}
+
+DecoderBuffer::~DecoderBuffer() {
+#if !defined(OS_ANDROID)
+ av_free(data_);
+#else
+ delete[] data_;
+#endif
+}
+
+void DecoderBuffer::Initialize() {
+ DCHECK_GE(buffer_size_, 0);
+#if !defined(OS_ANDROID)
+ // Why FF_INPUT_BUFFER_PADDING_SIZE? FFmpeg assumes all input buffers are
+ // padded. Using av_malloc with padding ensures FFmpeg only recieves data
+ // padded and aligned to its specifications.
+ data_ = reinterpret_cast<uint8*>(
+ av_malloc(buffer_size_ + FF_INPUT_BUFFER_PADDING_SIZE));
+ memset(data_ + buffer_size_, 0, FF_INPUT_BUFFER_PADDING_SIZE);
+#else
+ data_ = new uint8[buffer_size_];
+#endif
+}
+
+scoped_refptr<DecoderBuffer> DecoderBuffer::CopyFrom(const uint8* data,
+ int data_size) {
+ DCHECK(data);
+ return make_scoped_refptr(new DecoderBuffer(data, data_size));
+}
+
+scoped_refptr<DecoderBuffer> DecoderBuffer::CreateEOSBuffer() {
+ return make_scoped_refptr(new DecoderBuffer(NULL, 0));
+}
+
+const uint8* DecoderBuffer::GetData() const {
+ return data_;
+}
+
+int DecoderBuffer::GetDataSize() const {
+ return buffer_size_;
+}
+
+uint8* DecoderBuffer::GetWritableData() {
+ return data_;
+}
+
+const DecryptConfig* DecoderBuffer::GetDecryptConfig() const {
+ return decrypt_config_.get();
+}
+
+void DecoderBuffer::SetDecryptConfig(scoped_ptr<DecryptConfig> decrypt_config) {
+ decrypt_config_ = decrypt_config.Pass();
+}
+
+} // namespace media
« no previous file with comments | « media/base/decoder_buffer.h ('k') | media/base/decoder_buffer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698