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

chrome/browser/system_monitor/image_capture_camera.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.
+
+#import "chrome/browser/system_monitor/image_capture_camera.h"
+
+#include "base/file_util.h"
+#include "base/system_monitor/system_monitor.h"
+#include "chrome/browser/system_monitor/media_storage_util.h"
+#include "content/public/browser/browser_thread.h"
+
+namespace {
+
+// This continuation runs on the file thread to rename a saved file if needed.

[Lei Zhang, 2012/12/19 00:30:52]

Ading DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
makes for even better documentation.

[sail, 2012/12/19 01:18:54]

+1

[Greg Billock, 2012/12/19 17:20:08]

Done.

+void RenameFileAndReturn(const std::string& name,
+                         const FilePath& downloaded_filename,
+                         const FilePath& desired_filename,
+                         base::WeakPtr<ImageCaptureDeviceListener> listener) {

[sail, 2012/12/19 01:18:54]

As per my earlier comment this won't work.
See the earlier comment for a much better way to do this.

[Greg Billock, 2012/12/19 17:20:08]

How so? I agree bouncing between the threads will work too, but I like this
better -- requires no additional handoff between this continuation and the
caller. It relies on thread safety in the listener, but that's already pretty
much a certain requirement given other factors.

+  int64 edsize = 0;
+  file_util::GetFileSize(downloaded_filename, &edsize);

[Lei Zhang, 2012/12/19 00:30:52]

We are getting the file size but not using it.

[Greg Billock, 2012/12/19 17:20:08]

Thanks. This was leftover from logging trying to figure out what was going on
with the downloaded file when I was discovering it invented its own name.

+  bool error = file_util::ReplaceFile(downloaded_filename, desired_filename);

[Lei Zhang, 2012/12/19 00:30:52]

|error| should be renamed 'succeeded'.

[Greg Billock, 2012/12/19 17:20:08]

Done.

+
+  if (listener) {
+    listener->DownloadedFile(name,
+        error ? base::PLATFORM_FILE_OK : base::PLATFORM_FILE_ERROR_FAILED);
+  }
+}
+
+base::Time NSDateToBaseTime(NSDate* date) {
+  return base::Time::FromDoubleT([date timeIntervalSince1970]);
+}
+
+}  // namespace
+
+@implementation ImageCaptureCameraInterface
+
+- (id)initWithCameraDevice:(ICCameraDevice*)cameraDevice {
+  if ((self = [super init])) {
+    camera_.reset([cameraDevice retain]);
+    [camera_ setDelegate:self];
+  }
+  return self;
+}
+
+- (void)open {
+  [camera_ requestOpenSession];
+}
+
+- (void)dealloc {
+  // Make sure the session was closed before destruction.
+  DCHECK(![camera_ delegate]);
+  [super dealloc];
+}
+
+- (void)close {
+  [camera_ requestCloseSession];
+  [camera_ setDelegate:nil];
+}
+
+- (void)setListener:(base::WeakPtr<ImageCaptureDeviceListener>)listener {
+  listener_ = listener;
+}
+
+- (void)downloadFile:(const std::string&)name
+           localPath:(const FilePath&)localPath {
+  // Find the file with that name and start download.
+  for (ICCameraItem* item in [camera_ mediaFiles]) {
+    std::string itemName = base::SysNSStringToUTF8([item name]);
+    if (itemName == name) {
+      // To create save options for ImageCapture, we need to
+      // split the target filename into directory/name
+      // and encode the directory as a URL.
+      NSString* saveDirectory =
+          base::mac::FilePathToNSString(localPath.DirName());
+      NSString* saveFilename =
+          base::mac::FilePathToNSString(localPath.BaseName());
+
+      NSMutableDictionary* options =
+          [NSMutableDictionary dictionaryWithCapacity:3];
+      [options setObject:[NSURL fileURLWithPath:saveDirectory isDirectory:YES]
+                  forKey:ICDownloadsDirectoryURL];
+      [options setObject:saveFilename forKey:ICSaveAsFilename];
+      [options setObject:[NSNumber numberWithBool:YES] forKey:ICOverwrite];
+
+      [camera_ requestDownloadFile:base::mac::ObjCCastStrict<ICCameraFile>(item)
+                           options:options
+                  downloadDelegate:self
+               didDownloadSelector:
+                   @selector(didDownloadFile:error:options:contextInfo:)
+                       contextInfo:NULL];
+      return;
+    }
+  }
+
+  if (listener_)
+    listener_->DownloadedFile(name, base::PLATFORM_FILE_ERROR_NOT_FOUND);
+}
+
+- (void)cameraDevice:(ICCameraDevice*)camera didAddItem:(ICCameraItem*)item {
+  std::string name = base::SysNSStringToUTF8([item name]);
+  base::PlatformFileInfo info;
+  if ([[item UTI] isEqualToString:base::mac::CFToNSCast(kUTTypeFolder)])
+    info.is_directory = true;
+  else
+    info.size = [base::mac::ObjCCastStrict<ICCameraFile>(item) fileSize];
+  info.last_modified = NSDateToBaseTime([item modificationDate]);
+  info.creation_time = NSDateToBaseTime([item creationDate]);
+  info.last_accessed = info.last_modified;
+
+  if (listener_)
+    listener_->ItemAdded(name, info);
+}
+
+- (void)cameraDevice:(ICCameraDevice*)camera didAddItems:(NSArray*)items {
+  for (ICCameraItem* item in items)
+    [self cameraDevice:camera didAddItem:item];
+}
+
+- (void)didRemoveDevice:(ICDevice*)device {
+  device.delegate = NULL;
+  if (listener_) {
+    listener_->DeviceRemoved();
+    listener_.reset();
+  }
+}
+
+// Notifies that a session was opened with the given device; potentially
+// with an error.
+- (void)device:(ICDevice*)device didOpenSessionWithError:(NSError*)error {
+  if (error) {
+    [self didRemoveDevice:camera_];
+    if (listener_) {
+      listener_->DeviceRemoved();
+      listener_.reset();
+    }
+  }
+}
+
+- (void)device:(ICDevice*)device didEncounterError:(NSError*)error {
+  if (error) {
+    if (listener_) {
+      listener_->DeviceRemoved();
+      listener_.reset();
+    }
+  }
+}
+
+// When this message is received, all media metadata is now loaded.
+- (void)deviceDidBecomeReadyWithCompleteContentCatalog:(ICDevice*)device {
+  if (listener_)
+    listener_->NoMoreItems();
+}
+
+- (void)didDownloadFile:(ICCameraFile*)file
+                  error:(NSError*)error
+                options:(NSDictionary*)options
+            contextInfo:(void*)contextInfo {
+  std::string name = base::SysNSStringToUTF8([file name]);
+  if (error) {
+    if (listener_)
+      listener_->DownloadedFile(name, base::PLATFORM_FILE_ERROR_FAILED);
+    return;
+  }
+
+  std::string savedFilename =
+      base::SysNSStringToUTF8([options objectForKey:ICSavedFilename]);
+  std::string saveAsFilename =
+      base::SysNSStringToUTF8([options objectForKey:ICSaveAsFilename]);
+  if (savedFilename == saveAsFilename) {
+    if (listener_)
+      listener_->DownloadedFile(name, base::PLATFORM_FILE_OK);
+    return;
+  }
+
+  // ImageCapture did not save the file into the name we give it in the
+  // options. It picks a new name according to its best lights, so we need
+  // to rename the file.
+  FilePath save_dir(base::SysNSStringToUTF8(
+      [[options objectForKey:ICDownloadsDirectoryURL] path]));
+  FilePath saveAsPath = save_dir.Append(saveAsFilename);
+  FilePath savedPath = save_dir.Append(savedFilename);
+  content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
+      base::Bind(&RenameFileAndReturn,
+                 name, savedPath, saveAsPath, listener_));
+}
+
+@end  // ImageCaptureCameraInterface