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

Unified Diff: media/base/decoder_buffer.h

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/data_buffer.cc ('k') | media/base/decoder_buffer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/decoder_buffer.h
diff --git a/media/base/decoder_buffer.h b/media/base/decoder_buffer.h
new file mode 100644
index 0000000000000000000000000000000000000000..e822de555c623227da160885a94d531bdb561428
--- /dev/null
+++ b/media/base/decoder_buffer.h
@@ -0,0 +1,66 @@
+// 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.
+
+// A specialized buffer for interfacing with audio / video decoders.
+//
+// Specifically ensures that data is aligned and padded as necessary by the
+// underlying decoding framework. On desktop platforms this means memory is
+// allocated using FFmpeg with particular alignment and padding requirements.
+//
+// Also includes decoder specific functionality for decryption.
+
+#ifndef MEDIA_BASE_DECODER_BUFFER_H_
+#define MEDIA_BASE_DECODER_BUFFER_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "media/base/buffers.h"
+#include "media/base/decrypt_config.h"
+
+namespace media {
+
+class MEDIA_EXPORT DecoderBuffer : public Buffer {
+ public:
+ // Allocates buffer of size |buffer_size| >= 0. Buffer will be padded and
+ // aligned as necessary.
+ explicit DecoderBuffer(int buffer_size);
+
+ // Create a DecoderBuffer whose |data_| is copied from |data|. Buffer will be
+ // padded and aligned as necessary. |data| must not be NULL and |size| >= 0.
+ static scoped_refptr<DecoderBuffer> CopyFrom(const uint8* data, int size);
+
+ // Create a DecoderBuffer indicating we've reached end of stream. GetData()
+ // and GetWritableData() will return NULL and GetDataSize() will return 0.
+ static scoped_refptr<DecoderBuffer> CreateEOSBuffer();
+
+ // Buffer implementation.
+ virtual const uint8* GetData() const OVERRIDE;
+ virtual int GetDataSize() const OVERRIDE;
+
+ // Returns a read-write pointer to the buffer data.
+ virtual uint8* GetWritableData();
+
+ virtual const DecryptConfig* GetDecryptConfig() const;
+ virtual void SetDecryptConfig(scoped_ptr<DecryptConfig> decrypt_config);
+
+ protected:
+ // Allocates a buffer of size |size| >= 0 and copies |data| into it. Buffer
+ // will be padded and aligned as necessary. If |data| is NULL then |data_| is
+ // set to NULL and |buffer_size_| to 0.
+ DecoderBuffer(const uint8* data, int size);
+ virtual ~DecoderBuffer();
+
+ private:
+ int buffer_size_;
+ uint8* data_;
+ scoped_ptr<DecryptConfig> decrypt_config_;
+
+ // Constructor helper method for memory allocations.
+ void Initialize();
+
+ DISALLOW_COPY_AND_ASSIGN(DecoderBuffer);
+};
+
+} // namespace media
+
+#endif // MEDIA_BASE_DECODER_BUFFER_H_
« no previous file with comments | « media/base/data_buffer.cc ('k') | media/base/decoder_buffer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698