[StorageMonitor] Handle EjectDevice call for MTP devices

chrome/browser/storage_monitor/image_capture_device_manager.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/storage_monitor/image_capture_device_manager.h"
 
 #import <ImageCaptureCore/ImageCaptureCore.h>
 
 #import "chrome/browser/storage_monitor/image_capture_device.h"
 #include "chrome/browser/storage_monitor/storage_info.h"
@@ skipping 128 matching lines @@
 ImageCaptureDeviceManager::~ImageCaptureDeviceManager() {
   g_image_capture_device_manager = NULL;
   [device_browser_ close];
 }
 
 void ImageCaptureDeviceManager::SetNotifications(
     StorageMonitor::Receiver* notifications) {
   [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) {
   ImageCaptureDeviceManagerImpl* manager =
       g_image_capture_device_manager->device_browser_;
   return [manager deviceForUUID:uuid];
 }
 
 id<ICDeviceBrowserDelegate> ImageCaptureDeviceManager::device_browser() {
   return device_browser_.get();
 }
 
 }  // namespace chrome