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

chrome/browser/system_monitor/image_capture_device_browser_mac_unittest.mm

Index: chrome/browser/system_monitor/image_capture_device_browser_mac_unittest.mm
diff --git a/chrome/browser/system_monitor/image_capture_device_browser_mac_unittest.mm b/chrome/browser/system_monitor/image_capture_device_browser_mac_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..c54b8b59b5eaca9600e58a5a6f24ebb1123b3239
--- /dev/null
+++ b/chrome/browser/system_monitor/image_capture_device_browser_mac_unittest.mm
@@ -0,0 +1,204 @@
+// 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/system_monitor/image_capture_device_browser_mac.h"
+
+#import <ImageCaptureCore/ImageCaptureCore.h>
+
+#include "base/file_path.h"
+#include "base/mac/foundation_util.h"
+#include "base/memory/weak_ptr.h"
+#include "base/message_loop.h"
+#include "base/system_monitor/system_monitor.h"
+#include "chrome/browser/system_monitor/image_capture_camera.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+const char kDeviceId[] = "id";
+
+}  // namespace
+
+@interface MockICCameraDevice : ICCameraDevice {
+}
+
+@end
+
+@implementation MockICCameraDevice
+
+- (NSString*)mountPoint {
+  return @"mountPoint";
+}
+
+- (NSString*)name {
+  return @"name";
+}
+
+- (NSString*)UUIDString {
+  return base::SysUTF8ToNSString(kDeviceId);
+}
+
+- (ICDeviceType)type {
+  return ICDeviceTypeCamera;
+}
+
+- (void)requestOpenSession {
+}
+
+- (void)requestCloseSession {
+}
+
+@end
+
+@interface MockICCameraFile : ICCameraFile {
+ @private
+  scoped_nsobject<NSString> name_;
+  scoped_nsobject<NSDate> date_;
+}
+
+- (id)init:(NSString*)name;
+
+@end
+
+@implementation MockICCameraFile
+
+- (id)init:(NSString*)name {
+  if ((self = [super init])) {
+    name_.reset(name);
+    date_.reset([[NSDate dateWithNaturalLanguageString:@"12/12/12"] retain]);
+  }
+  return self;
+}
+
+- (NSString*)name {
+  return name_.get();
+}
+
+- (NSString*)UTI {
+  return base::mac::CFToNSCast(kUTTypeImage);
+}
+
+- (NSDate*)modificationDate {
+  return date_.get();
+}
+
+- (NSDate*)creationDate {
+  return date_.get();
+}
+
+- (off_t)fileSize {
+  return 1000;
+}
+
+@end
+
+class TestCameraListener
+    : public ImageCaptureDeviceListener,
+      public base::SupportsWeakPtr<TestCameraListener> {
+ public:
+  TestCameraListener() : completed_(false), removed_(false) {}
+  virtual ~TestCameraListener() {}
+
+  virtual void ItemAdded(const std::string& name,
+                         const base::PlatformFileInfo& info) OVERRIDE {
+    items_.push_back(name);
+  }
+
+  virtual void NoMoreItems() OVERRIDE {
+    completed_ = true;
+  }
+
+  virtual void DownloadedFile(const std::string& name,
+                              base::PlatformFileError error) OVERRIDE {
+    downloads_.push_back(name);
+  }
+
+  virtual void DeviceRemoved() OVERRIDE {
+    removed_ = true;
+  }
+
+  std::vector<std::string> items_;
+  std::vector<std::string> downloads_;
+  bool completed_;
+  bool removed_;
+};
+
+class ImageCaptureDeviceBrowserTest : public testing::Test {
+ public:
+  virtual void SetUp() OVERRIDE {
+    base::SystemMonitor::AllocateSystemIOPorts();
+    system_monitor_.reset(new base::SystemMonitor());
+  }
+
+  ICCameraDevice* AttachDevice(
+      chrome::ImageCaptureDeviceBrowser* browser) {
+    // Ownership will be passed to the device browser delegate.
+    ICCameraDevice* device = [MockICCameraDevice alloc];
+    id<ICDeviceBrowserDelegate> delegate = browser->device_browser();
+    [delegate deviceBrowser:nil didAddDevice:device moreComing:NO];
+    return device;
+  }
+
+  void DetachDevice(chrome::ImageCaptureDeviceBrowser* browser,
+                    ICCameraDevice* device) {
+    id<ICDeviceBrowserDelegate> delegate = browser->device_browser();
+    [delegate deviceBrowser:nil didRemoveDevice:device moreGoing:NO];
+  }
+
+ protected:
+  MessageLoop message_loop_;
+  scoped_ptr<base::SystemMonitor> system_monitor_;
+  TestCameraListener listener_;
+};
+
+TEST_F(ImageCaptureDeviceBrowserTest, TestAttachDetach) {
+  chrome::ImageCaptureDeviceBrowser browser;
+  ICCameraDevice* device = AttachDevice(&browser);
+
+  std::vector<base::SystemMonitor::RemovableStorageInfo> devices =
+    system_monitor_->GetAttachedRemovableStorage();
+
+  ASSERT_EQ(1U, devices.size());
+  EXPECT_EQ(std::string("ic:") + kDeviceId, devices[0].device_id);
+
+  DetachDevice(&browser, device);
+  devices = system_monitor_->GetAttachedRemovableStorage();
+  ASSERT_EQ(0U, devices.size());
+};
+
+TEST_F(ImageCaptureDeviceBrowserTest, OpenCamera) {
+  chrome::ImageCaptureDeviceBrowser browser;
+  ICCameraDevice* device = AttachDevice(&browser);
+
+  EXPECT_FALSE(chrome::ImageCaptureDeviceBrowser::cameraInterfaceForUUID(
+      "nonexistent"));
+
+  scoped_nsobject<ImageCaptureCameraInterface> camera(
+      [chrome::ImageCaptureDeviceBrowser::cameraInterfaceForUUID(kDeviceId)
+       retain]);

[sail, 2012/12/17 23:55:03]

indent 3 more spaces

[Greg Billock, 2012/12/19 00:05:09]

Done.

+
+  [camera open];
+  [camera setListener:listener_.AsWeakPtr()];
+
+  scoped_nsobject<MockICCameraFile> picture1(
+      [[MockICCameraFile alloc] init:@"pic1"]);
+  [camera cameraDevice:nil didAddItem:picture1];
+  scoped_nsobject<MockICCameraFile> picture2(
+      [[MockICCameraFile alloc] init:@"pic2"]);
+  [camera cameraDevice:nil didAddItem:picture2];
+  ASSERT_EQ(2U, listener_.items_.size());
+  EXPECT_EQ("pic1", listener_.items_[0]);
+  EXPECT_EQ("pic2", listener_.items_[1]);
+  EXPECT_FALSE(listener_.completed_);
+
+  [camera deviceDidBecomeReadyWithCompleteContentCatalog:nil];
+  ASSERT_EQ(2U, listener_.items_.size());
+  EXPECT_TRUE(listener_.completed_);
+
+  [camera close];
+
+  DetachDevice(&browser, device);
+  EXPECT_FALSE(chrome::ImageCaptureDeviceBrowser::cameraInterfaceForUUID(
+      kDeviceId));
+}