Provide a Chrome-owned buffer to FFmpeg for video decoding, instead of

media/base/video_frame.h

Index: media/base/video_frame.h
===================================================================
--- media/base/video_frame.h	(revision 139175)
+++ media/base/video_frame.h	(working copy)
@@ -119,14 +119,15 @@
   // context.  Calls MD5Update with the context and the contents of the frame.
   void HashFrameForTesting(base::MD5Context* context);
 
- private:
-  friend class base::RefCountedThreadSafe<VideoFrame>;
-  // Clients must use the static CreateFrame() method to create a new frame.
   VideoFrame(Format format,
              size_t video_width,
              size_t video_height,
              base::TimeDelta timestamp,
              base::TimeDelta duration);
+  int AllocateAlignedWithStrides(int strides[], int aligned_h);

[Ami GONE FROM CHROMIUM, 2012/05/26 22:52:34]

This change (ctor public, add public allocate method) seems strange.
Why wouldn't you bury the alignment details inside the CreateFrame public entry
point?
For that matter, what code path wants to allocate *unaligned* memory?  I suspect
you can drop all changes to this file from this CL (restricting your change to
the .cc file).

+
+private:
+  friend class base::RefCountedThreadSafe<VideoFrame>;
   virtual ~VideoFrame();
 
   // Used internally by CreateFrame().
@@ -150,6 +151,7 @@
 
   // Array of data pointers to each plane.
   uint8* data_[kMaxPlanes];
+  int is_aligned_alloc_;
 
   // Native texture ID, if this is a NATIVE_TEXTURE frame.
   uint32 texture_id_;