Add a receiver interface to RemovableStorageNotifications.

chrome/browser/system_monitor/image_capture_device_manager.h

Index: chrome/browser/system_monitor/image_capture_device_manager.h
diff --git a/chrome/browser/system_monitor/image_capture_device_manager.h b/chrome/browser/system_monitor/image_capture_device_manager.h
index a10ae06da56f28321e5c21501c0973734adcf77b..721e3251666cfa109593ccf1e95978211f4f8128 100644
--- a/chrome/browser/system_monitor/image_capture_device_manager.h
+++ b/chrome/browser/system_monitor/image_capture_device_manager.h
@@ -33,6 +33,9 @@ class ImageCaptureDeviceManager {
   // Returns a weak pointer to the internal ImageCapture interface protocol.
   id<ICDeviceBrowserDelegate> device_browser();
 
+  // Sets the receiver for device attach/detach notifications.
+  void SetNotifications(RemovableStorageNotifications::Receiver* notifications);

[vandebo (ex-Chrome), 2013/01/31 23:51:29]

Why not pass this in to the constructor?

[Greg Billock, 2013/02/01 18:28:42]

They should be. I used these setters because of the ownership problems. What I
anticipate is that when all these are owned by the notifications class, they'll
be created that way. Currently, there's unevenness in where they're all made in
the ChromeBrowserPartsMain, and in tests, so that won't work uniformly. I hit
that early and then just copy-pasted to the rest, though. There could be a
couple where it would.

[vandebo (ex-Chrome), 2013/02/01 18:59:55]

Ok, can you add a TODO here then, so it's clear that's where we want to end up?

[Greg Billock, 2013/02/01 23:05:42]

Done.

+
  private:
   scoped_nsobject<ImageCaptureDeviceManagerImpl> device_browser_;
 };