[StorageMonitor] Handle EjectDevice call for MTP devices

chrome/browser/storage_monitor/image_capture_device_manager.mm

Index: chrome/browser/storage_monitor/image_capture_device_manager.mm
diff --git a/chrome/browser/storage_monitor/image_capture_device_manager.mm b/chrome/browser/storage_monitor/image_capture_device_manager.mm
index b51d32c38a3d09924ee18a69addbf880338e1d11..28e251884884b642d5020ced8f4a89f7c10af187 100644
--- a/chrome/browser/storage_monitor/image_capture_device_manager.mm
+++ b/chrome/browser/storage_monitor/image_capture_device_manager.mm
@@ -146,6 +146,16 @@ void ImageCaptureDeviceManager::SetNotifications(
   [device_browser_ setNotifications:notifications];
 }
 
+void ImageCaptureDeviceManager::EjectDevice(
+    const std::string& uuid,
+    base::Callback<void(StorageMonitor::EjectStatus)> callback) {
+  base::scoped_nsobject<ImageCaptureDevice> camera_device(

[sail, 2013/09/09 21:26:13]

Is there a reason you need to keep a strong reference to this here? If there's
not then you could just do:
  ImageCaptureDevice* camera_device = [device_browser_ deviceForUUID:uuid];

[Greg Billock, 2013/09/12 17:43:59]

No, not really. This is how we did it elsewhere, so I was just following that
basically.

+      [[device_browser_ deviceForUUID:uuid] retain]);
+  [camera_device eject];
+  [camera_device close];
+  callback.Run(StorageMonitor::EJECT_OK);
+}
+
 // static
 ImageCaptureDevice* ImageCaptureDeviceManager::deviceForUUID(
     const std::string& uuid) {