[Media Galleries] Add an ImageCaptureCore listener for Mac. (part 2)

chrome/browser/system_monitor/image_capture_device_manager.h

+// 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.
+
+#ifndef CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_
+#define CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_
+
+#import <Foundation/Foundation.h>
+#include <string>
+
+#include "base/memory/scoped_nsobject.h"
+
+@protocol ICDeviceBrowserDelegate;
+@class ImageCaptureDevice;
+@class ImageCaptureDeviceManagerImpl;
+
+namespace chrome {
+
+// This wrapper basically allows the owner to have a scoped_ptr rather than a

[sail, 2013/01/04 18:58:56]

This comment explaining how a C++ wrapper is not that useful. Instead could you
explain what this class does. For example, if I create an instance of this class
what happens?

[Greg Billock, 2013/01/04 21:05:50]

Yeah, this was bad. Fixed up.

On 2013/01/04 18:58:56, sail wrote:

+// scoped_nsobject, isolating the Objective-C parts of the implementation away.
+class ImageCaptureDeviceManager {
+ public:
+  ImageCaptureDeviceManager();
+  ~ImageCaptureDeviceManager();
+
+  // The UUIDs passed here are available in the device attach notifications
+  // given through SystemMonitor. They're gotten by cracking the device ID
+  // and taking the unique ID output.
+  static ImageCaptureDevice* deviceForUUID(const std::string& uuid);
+
+  // Returns a weak pointer to the internal ImageCapture interface protocol.
+  id<ICDeviceBrowserDelegate> device_browser();
+
+ private:
+  scoped_nsobject<ImageCaptureDeviceManagerImpl> device_browser_;
+};
+
+}  // namespace chrome
+
+#endif  // CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_