Created VideoFramePool to avoid unnecessary VideoFrame alloc/free.

media/base/video_frame.h

Index: media/base/video_frame.h
diff --git a/media/base/video_frame.h b/media/base/video_frame.h
index dc0932043b90a89e9e91ff801adb8db8c612732a..a986b0ac7a0d52668b6992dfcc58e243ea79f256 100644
--- a/media/base/video_frame.h
+++ b/media/base/video_frame.h
@@ -169,6 +169,15 @@ class MEDIA_EXPORT VideoFrame : public base::RefCountedThreadSafe<VideoFrame> {
       base::TimeDelta timestamp,
       const base::Closure& no_longer_needed_cb);
 
+  // Wraps |frame| and calls |wrapper_destroyed_cb| when the wrapper VideoFrame
+  // gets destroyed.
+  typedef base::Callback<void(const scoped_refptr<VideoFrame>&)>
+      WrapperDestroyedCB;
+
+  static scoped_refptr<VideoFrame> WrapVideoFrame(
+      const scoped_refptr<VideoFrame>& frame,
+      const WrapperDestroyedCB& wrapper_destroyed_cb);

[DaleCurtis, 2013/12/09 20:48:25]

Rename to no_longer_needed_cb for consistency with the above and change to
base::Closure()?  You can bind the frame in with the call to WrapVideoFrame and
avoid the extra bind within the function.

[acolwell GONE FROM CHROMIUM, 2013/12/09 21:42:11]

Done.

+
   // Creates a frame which indicates end-of-stream.
   static scoped_refptr<VideoFrame> CreateEOSFrame();