RefCounted types should not have public destructors, content/browser part 2

content/renderer/media/video_capture_impl_manager.h

Index: content/renderer/media/video_capture_impl_manager.h
diff --git a/content/renderer/media/video_capture_impl_manager.h b/content/renderer/media/video_capture_impl_manager.h
index 8427957fbc43527208464cf120c82f7e75d10aee..ca0d7cbc511926872888f6d176df9d5e1bd30d46 100644
--- a/content/renderer/media/video_capture_impl_manager.h
+++ b/content/renderer/media/video_capture_impl_manager.h
@@ -26,7 +26,6 @@ class CONTENT_EXPORT VideoCaptureImplManager
     : public base::RefCountedThreadSafe<VideoCaptureImplManager> {
  public:
   VideoCaptureImplManager();
-  virtual ~VideoCaptureImplManager();
 
   // Called by video capture client |handler| to add device referenced
   // by |id| to VideoCaptureImplManager's list of opened device list.
@@ -46,6 +45,9 @@ class CONTENT_EXPORT VideoCaptureImplManager
   }
 
  private:
+  friend class base::RefCountedThreadSafe<VideoCaptureImplManager>;
+  friend class MockVideoCaptureImplManager;

[scherkus (not reviewing), 2012/04/27 20:59:48]

is this needed because the dtor isn't protected?

+
   struct Device {
     Device(VideoCaptureImpl* device,
            media::VideoCapture::EventHandler* handler);
@@ -55,6 +57,8 @@ class CONTENT_EXPORT VideoCaptureImplManager
     std::list<media::VideoCapture::EventHandler*> clients;
   };
 
+  virtual ~VideoCaptureImplManager();
+
   void FreeDevice(VideoCaptureImpl* vc);
 
   typedef std::map<media::VideoCaptureSessionId, Device*> Devices;