Use FFmpegVideoDecoder from a UtilityProcess

media/base/video_frame_pool.h

 // Copyright 2013 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_POOL_H_
 #define MEDIA_BASE_VIDEO_FRAME_POOL_H_
 
 #include <stddef.h>
 
 #include "base/macros.h"
 #include "media/base/media_export.h"
 #include "media/base/video_frame.h"
 
 namespace media {
 
+class VideoFramePoolDelegate {

[sandersd (OOO until July 31), 2016/12/15 00:38:01]

I am opposed to the name Delegate on the grounds that media/ has a few of them
and it never means the same thing. This particular usage might be better named
VideoFrameProvider? (Or go all the way and call it
VideoFrame::Provider/VideoFrame::Factory or something like that.)

+ public:
+  virtual ~VideoFramePoolDelegate() {}
+  virtual scoped_refptr<VideoFrame> CreateZeroInitializedFrame(
+      VideoPixelFormat format,
+      const gfx::Size& coded_size,
+      const gfx::Rect& visible_rect,
+      const gfx::Size& natural_size,
+      base::TimeDelta timestamp) = 0;
+
+  virtual scoped_refptr<VideoFrame> WrapVideoFrame(
+      const scoped_refptr<VideoFrame>& frame) = 0;
+};
+
 // Simple VideoFrame pool used to avoid unnecessarily allocating and destroying
 // VideoFrame objects. The pool manages the memory for the VideoFrame
 // returned by CreateFrame(). When one of these VideoFrames is destroyed,
 // the memory is returned to the pool for use by a subsequent CreateFrame()
 // call. The memory in the pool is retained for the life of the
 // VideoFramePool object. If the parameters passed to CreateFrame() change
 // during the life of this object, then the memory used by frames with the old
 // parameter values will be purged from the pool.
 class MEDIA_EXPORT VideoFramePool {
  public:
   VideoFramePool();
   ~VideoFramePool();
 
   // Returns a frame from the pool that matches the specified
   // parameters or creates a new frame if no suitable frame exists in
   // the pool. The pool is drained if no matching frame is found.
   // The buffer for the new frame will be zero initialized.  Reused frames will
   // not be zero initialized.
   scoped_refptr<VideoFrame> CreateFrame(VideoPixelFormat format,
                                         const gfx::Size& coded_size,
                                         const gfx::Rect& visible_rect,
                                         const gfx::Size& natural_size,
                                         base::TimeDelta timestamp);
 
-protected:
+  void SetDelegate(std::unique_ptr<VideoFramePoolDelegate> delegate);

[sandersd (OOO until July 31), 2016/12/15 00:38:01]

It seems like the pool doesn't work without calling this. I recommend making it
a constructor argument with a default value.

+
+ protected:
   friend class VideoFramePoolTest;
 
   // Returns the number of frames in the pool for testing purposes.
   size_t GetPoolSizeForTesting() const;
 
  private:
   class PoolImpl;
   scoped_refptr<PoolImpl> pool_;
+  std::unique_ptr<VideoFramePoolDelegate> delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoFramePool);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_FRAME_POOL_H_