Refactor video capturing code in the render process

content/renderer/pepper/pepper_video_capture_host.h

Index: content/renderer/pepper/pepper_video_capture_host.h
diff --git a/content/renderer/pepper/pepper_video_capture_host.h b/content/renderer/pepper/pepper_video_capture_host.h
index 8f5b77ad3b1778894631d1f45ea7544eb20243eb..1cea4c46ef6f7ef8532d80f7561c54bd55207e4c 100644
--- a/content/renderer/pepper/pepper_video_capture_host.h
+++ b/content/renderer/pepper/pepper_video_capture_host.h
@@ -7,21 +7,25 @@
 
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "content/common/media/video_capture.h"
 #include "content/public/renderer/renderer_ppapi_host.h"
 #include "content/renderer/pepper/pepper_device_enumeration_host_helper.h"
 #include "content/renderer/pepper/ppb_buffer_impl.h"
-#include "media/video/capture/video_capture.h"
 #include "media/video/capture/video_capture_types.h"
 #include "ppapi/c/dev/ppp_video_capture_dev.h"
 #include "ppapi/host/host_message_context.h"
 #include "ppapi/host/resource_host.h"
 
+namespace media {
+class VideoFrame;
+}  // namespace media
+
 namespace content {
 class PepperPlatformVideoCapture;
 class RendererPpapiHostImpl;
 
-class PepperVideoCaptureHost : public ppapi::host::ResourceHost,
-                               public media::VideoCapture::EventHandler {
+class PepperVideoCaptureHost : public ppapi::host::ResourceHost {
  public:
   PepperVideoCaptureHost(RendererPpapiHostImpl* host,
                          PP_Instance instance,
@@ -35,17 +39,14 @@ class PepperVideoCaptureHost : public ppapi::host::ResourceHost,
       const IPC::Message& msg,
       ppapi::host::HostMessageContext* context) OVERRIDE;
 
-  void OnInitialized(media::VideoCapture* capture, bool succeeded);
+  void OnInitialized(bool succeeded);
 
-  // media::VideoCapture::EventHandler
-  virtual void OnStarted(media::VideoCapture* capture) OVERRIDE;
-  virtual void OnStopped(media::VideoCapture* capture) OVERRIDE;
-  virtual void OnPaused(media::VideoCapture* capture) OVERRIDE;
-  virtual void OnError(media::VideoCapture* capture, int error_code) OVERRIDE;
-  virtual void OnRemoved(media::VideoCapture* capture) OVERRIDE;
-  virtual void OnFrameReady(media::VideoCapture* capture,
-                            const scoped_refptr<media::VideoFrame>& frame)
-      OVERRIDE;
+  void OnStarted();

[Ami GONE FROM CHROMIUM, 2014/04/21 23:42:53]

It's strange to see methods like this not be OVERRIDEs or =0 or documented.  
I wonder whether you could get away with exposing only getters for the CB
(bound) versions of these methods?

[Alpha Left Google, 2014/04/23 18:48:33]

I added documents for these methods. PepperVidoeCaptureHost is accessed by
PapperPlatformVideoCapture directly without callbacks, in other ways than video
frame delivery and state changes. I would like to keep it this way but get rid
of the inheritance. 

+  void OnStopped();
+  void OnPaused();
+  void OnError();
+  void OnFrameReady(const scoped_refptr<media::VideoFrame>& frame,
+                    media::VideoCaptureFormat format);
 
  private:
   int32_t OnOpen(ppapi::host::HostMessageContext* context,
@@ -72,7 +73,7 @@ class PepperVideoCaptureHost : public ppapi::host::ResourceHost,
 
   bool SetStatus(PP_VideoCaptureStatus_Dev status, bool forced);
 
-  scoped_refptr<PepperPlatformVideoCapture> platform_video_capture_;
+  scoped_ptr<PepperPlatformVideoCapture> platform_video_capture_;
 
   // Buffers of video frame.
   struct BufferInfo {