Update the Chromium version of webrtc::VideoCaptureModule to be reference counted.

content/renderer/media/video_capture_module_impl.h

Index: content/renderer/media/video_capture_module_impl.h
===================================================================
--- content/renderer/media/video_capture_module_impl.h	(revision 100680)
+++ content/renderer/media/video_capture_module_impl.h	(working copy)
@@ -22,7 +22,10 @@
  public:
   VideoCaptureModuleImpl(const media::VideoCaptureSessionId id,
                          VideoCaptureImplManager* vc_manager);
-  virtual ~VideoCaptureModuleImpl();
+  // Implement reference counting. This make it possible to reference the
+  // object from both Chromium and WebRtc.
+  virtual int32_t AddRef();

[scherkus (not reviewing), 2011/09/13 15:22:58]

is this actually an interface that we're implementing? why does it need to be
virtual?

traditionally (since implementing refcounting is tricky), you extend
base::RefCountedThreadSafe<MyClassName>

[perkj_chrome, 2011/09/19 11:14:21]

Yes- it is part of the interface we are implementing. 
Same as webrtc_audio_device_impl.h.
We have not used base::RefCountedThreadSafe since the return type is not the
same.

On 2011/09/13 15:22:58, scherkus wrote:

+  virtual int32_t Release();
 
   // Override webrtc::videocapturemodule::VideoCaptureImpl implementation.
   virtual WebRtc_Word32 StartCapture(
@@ -45,6 +48,7 @@
       const media::VideoCaptureParams& device_info);
 
  private:
+  virtual ~VideoCaptureModuleImpl();
   void Init();
 
   void StartCaptureOnCaptureThread(
@@ -76,6 +80,7 @@
   media::VideoCapture* capture_engine_;
   bool pending_start_;
   webrtc::VideoCaptureCapability pending_cap_;
+  int ref_count_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoCaptureModuleImpl);
 };