Created VideoFramePool to avoid unnecessary VideoFrame alloc/free.

media/base/video_frame.h

 // 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.
 
 #ifndef MEDIA_BASE_VIDEO_FRAME_H_
 #define MEDIA_BASE_VIDEO_FRAME_H_
 
 #include "base/callback.h"
 #include "base/md5.h"
 #include "base/memory/shared_memory.h"
@@ skipping 151 matching lines @@
       const gfx::Size& natural_size,
       int32 y_stride,
       int32 u_stride,
       int32 v_stride,
       uint8* y_data,
       uint8* u_data,
       uint8* v_data,
       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();
 
   // Allocates YV12 frame based on |size|, and sets its data to the YUV(y,u,v).
   static scoped_refptr<VideoFrame> CreateColorFrame(
       const gfx::Size& size,
       uint8 y, uint8 u, uint8 v,
       base::TimeDelta timestamp);
 
   // Allocates YV12 frame based on |size|, and sets its data to the YUV
@@ skipping 111 matching lines @@
   base::TimeDelta timestamp_;
 
   const bool end_of_stream_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(VideoFrame);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_FRAME_H_