[StorageMonitor] Move StorageMonitor ownership to BrowserProcessImpl

chrome/browser/storage_monitor/storage_monitor_win.cc

Index: chrome/browser/storage_monitor/storage_monitor_win.cc
diff --git a/chrome/browser/storage_monitor/storage_monitor_win.cc b/chrome/browser/storage_monitor/storage_monitor_win.cc
index 4fcc8f6b2856f377ee57f7c3c533f1c08fb14c9e..beec36d7af1a6357c97446ac53978b7f3e46d28c 100644
--- a/chrome/browser/storage_monitor/storage_monitor_win.cc
+++ b/chrome/browser/storage_monitor/storage_monitor_win.cc
@@ -18,12 +18,6 @@ namespace chrome {
 
 // StorageMonitorWin -------------------------------------------------------
 
-// static
-StorageMonitorWin* StorageMonitorWin::Create() {
-  return new StorageMonitorWin(new VolumeMountWatcherWin(),
-                               new PortableDeviceWatcherWin());
-}
-
 StorageMonitorWin::StorageMonitorWin(
     VolumeMountWatcherWin* volume_mount_watcher,
     PortableDeviceWatcherWin* portable_device_watcher)
@@ -167,4 +161,9 @@ void StorageMonitorWin::OnDeviceChange(UINT event_type, LPARAM data) {
   portable_device_watcher_->OnWindowMessage(event_type, data);
 }
 
+StorageMonitor* StorageMonitor::Create() {
+  return new StorageMonitorWin(new VolumeMountWatcherWin(),

[vandebo (ex-Chrome), 2013/07/10 16:57:45]

Is this what you mean by the factory?  Would making the constructor protected
solve the problem?

[Greg Billock, 2013/07/10 21:11:16]

Yes, this is the windows factory impl. You mean make the StorageMonitor
constructor protected? It is now.

[vandebo (ex-Chrome), 2013/07/11 15:53:00]

The StorageMonitorWin constructor is currently public.

+                               new PortableDeviceWatcherWin());
+}
+
 }  // namespace chrome