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

chrome/browser/system_monitor/image_capture_device_browser_mac.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_MAC_H_
+#define CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MAC_H_
+
+#import <Foundation/Foundation.h>
+
+#include "base/file_path.h"
+#include "base/memory/scoped_nsobject.h"
+#include "base/platform_file.h"
+#include "base/string_util.h"
+#include "base/sys_string_conversions.h"
+
+@class ImageCaptureCameraInterface;
+@class ImageCaptureDeviceBrowserMac;
+class ImageCaptureDeviceBrowserTest;
+
+namespace chrome {
+
+// This wrapper basically allows the owner to have a scoped_ptr rather than a
+// scoped_nsobject, isolating the Objective-C parts of the implementation away.
+class ImageCaptureDeviceBrowser {
+ public:
+  ImageCaptureDeviceBrowser();
+  ~ImageCaptureDeviceBrowser();
+
+  // 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 ImageCaptureCameraInterface* cameraInterfaceForUUID(
+      const std::string& uuid);
+
+ private:
+  friend class ::ImageCaptureDeviceBrowserTest;

[sail, 2012/12/17 03:16:03]

It would be better to add a public accessor that returns a
ICDeviceBrowserDelegate. That way you don't need this friend statement and the
cast in the test.

[Greg Billock, 2012/12/17 23:20:41]

Sounds good.

+
+  scoped_nsobject<ImageCaptureDeviceBrowserMac> device_browser_;
+};
+
+}  // namespace chrome
+
+
+
+#endif   // CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MAC_H_