Use scoped_ptr<> and deleter for media::DecoderBuffer::data_.

media/base/decoder_buffer.h

Index: media/base/decoder_buffer.h
diff --git a/media/base/decoder_buffer.h b/media/base/decoder_buffer.h
index d0c57ce07f53afdc4a97435cce0e38b9d358d749..7e388a93c4ae8be00440b96c2f760a12191288ab 100644
--- a/media/base/decoder_buffer.h
+++ b/media/base/decoder_buffer.h
@@ -13,6 +13,7 @@
 #ifndef MEDIA_BASE_DECODER_BUFFER_H_
 #define MEDIA_BASE_DECODER_BUFFER_H_
 
+#include "base/memory/aligned_memory.h"
 #include "base/memory/scoped_ptr.h"
 #include "build/build_config.h"
 #include "media/base/buffers.h"
@@ -62,7 +63,7 @@ class MEDIA_EXPORT DecoderBuffer : public Buffer {
 
  private:
   int buffer_size_;
-  uint8* data_;
+  scoped_ptr<uint8[], base::ScopedPtrAlignedFree> data_;

[awong, 2012/12/28 21:17:26]

Is uint8* or uint8[] more appropriate here?  Do you want the array-access API,
or the pointer API?

[xhwang, 2012/12/29 22:37:26]

Right. I don't use array-access API. Changed to scoped_ptr<uint8, ...>.

   scoped_ptr<DecryptConfig> decrypt_config_;
 
   // Constructor helper method for memory allocations.