Convert Media Galleries to use base::File

base/files/file_util_proxy.cc

 // 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 "base/files/file_util_proxy.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/file_util.h"
 #include "base/files/file.h"
 #include "base/location.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "base/task_runner.h"
 #include "base/task_runner_util.h"
 
 namespace base {
 
 namespace {
 
 void CallWithTranslatedParameter(const FileUtilProxy::StatusCallback& callback,
                                  bool value) {
   DCHECK(!callback.is_null());
-  callback.Run(value ? PLATFORM_FILE_OK : PLATFORM_FILE_ERROR_FAILED);
+  callback.Run(value ? File::FILE_OK : File::FILE_ERROR_FAILED);
 }
 
 // Helper classes or routines for individual methods.
 class CreateOrOpenHelper {
  public:
   CreateOrOpenHelper(TaskRunner* task_runner,
                      const FileUtilProxy::CloseTask& close_task)
       : task_runner_(task_runner),
         close_task_(close_task),
         file_handle_(kInvalidPlatformFileValue),
         created_(false),
-        error_(PLATFORM_FILE_OK) {}
+        error_(File::FILE_OK) {}
 
   ~CreateOrOpenHelper() {
     if (file_handle_ != kInvalidPlatformFileValue) {
       task_runner_->PostTask(
           FROM_HERE,
           base::Bind(base::IgnoreResult(close_task_), file_handle_));
     }
   }
 
   void RunWork(const FileUtilProxy::CreateOrOpenTask& task) {
     error_ = task.Run(&file_handle_, &created_);
   }
 
   void Reply(const FileUtilProxy::CreateOrOpenCallback& callback) {
     DCHECK(!callback.is_null());
     callback.Run(error_, PassPlatformFile(&file_handle_), created_);
   }
 
  private:
   scoped_refptr<TaskRunner> task_runner_;
   FileUtilProxy::CloseTask close_task_;
   PlatformFile file_handle_;
   bool created_;
-  PlatformFileError error_;
+  File::Error error_;
   DISALLOW_COPY_AND_ASSIGN(CreateOrOpenHelper);
 };
 
 class CreateTemporaryHelper {
  public:
   explicit CreateTemporaryHelper(TaskRunner* task_runner)
       : task_runner_(task_runner),
         file_handle_(kInvalidPlatformFileValue),
-        error_(PLATFORM_FILE_OK) {}
+        error_(File::FILE_OK) {}
 
   ~CreateTemporaryHelper() {
     if (file_handle_ != kInvalidPlatformFileValue) {
       FileUtilProxy::Close(
           task_runner_.get(), file_handle_, FileUtilProxy::StatusCallback());
     }
   }
 
   void RunWork(int additional_file_flags) {
     // TODO(darin): file_util should have a variant of CreateTemporaryFile
     // that returns a FilePath and a PlatformFile.
     base::CreateTemporaryFile(&file_path_);
 
     int file_flags =
         PLATFORM_FILE_WRITE |
         PLATFORM_FILE_TEMPORARY |
         PLATFORM_FILE_CREATE_ALWAYS |
         additional_file_flags;
 
-    error_ = PLATFORM_FILE_OK;
-    file_handle_ = CreatePlatformFile(file_path_, file_flags, NULL, &error_);
+    error_ = File::FILE_OK;
+    // TODO(rvargas): Convert this code to use File.
+    file_handle_ =
+        CreatePlatformFile(file_path_, file_flags, NULL,
+                           reinterpret_cast<PlatformFileError*>(&error_));
   }
 
   void Reply(const FileUtilProxy::CreateTemporaryCallback& callback) {
     DCHECK(!callback.is_null());
     callback.Run(error_, PassPlatformFile(&file_handle_), file_path_);
   }
 
  private:
   scoped_refptr<TaskRunner> task_runner_;
   PlatformFile file_handle_;
   FilePath file_path_;
-  PlatformFileError error_;
+  File::Error error_;
   DISALLOW_COPY_AND_ASSIGN(CreateTemporaryHelper);
 };
 
 class GetFileInfoHelper {
  public:
   GetFileInfoHelper()
-      : error_(PLATFORM_FILE_OK) {}
+      : error_(File::FILE_OK) {}
 
   void RunWorkForFilePath(const FilePath& file_path) {
     if (!PathExists(file_path)) {
-      error_ = PLATFORM_FILE_ERROR_NOT_FOUND;
+      error_ = File::FILE_ERROR_NOT_FOUND;
       return;
     }
     // TODO(rvargas): switch this file to base::File.
     if (!GetFileInfo(file_path, reinterpret_cast<File::Info*>(&file_info_)))
-      error_ = PLATFORM_FILE_ERROR_FAILED;
+      error_ = File::FILE_ERROR_FAILED;
   }
 
   void RunWorkForPlatformFile(PlatformFile file) {
-    if (!GetPlatformFileInfo(file, &file_info_))
-      error_ = PLATFORM_FILE_ERROR_FAILED;
+    if (!GetPlatformFileInfo(
+            file, reinterpret_cast<PlatformFileInfo*>(&file_info_))) {
+      error_ = File::FILE_ERROR_FAILED;
+    }
   }
 
   void Reply(const FileUtilProxy::GetFileInfoCallback& callback) {
     if (!callback.is_null()) {
       callback.Run(error_, file_info_);
     }
   }
 
  private:
-  PlatformFileError error_;
-  PlatformFileInfo file_info_;
+  File::Error error_;
+  File::Info file_info_;
   DISALLOW_COPY_AND_ASSIGN(GetFileInfoHelper);
 };
 
 class ReadHelper {
  public:
   explicit ReadHelper(int bytes_to_read)
       : buffer_(new char[bytes_to_read]),
         bytes_to_read_(bytes_to_read),
         bytes_read_(0) {}
 
   void RunWork(PlatformFile file, int64 offset) {
     bytes_read_ = ReadPlatformFile(file, offset, buffer_.get(), bytes_to_read_);
   }
 
   void Reply(const FileUtilProxy::ReadCallback& callback) {
     if (!callback.is_null()) {
-      PlatformFileError error =
-          (bytes_read_ < 0) ? PLATFORM_FILE_ERROR_FAILED : PLATFORM_FILE_OK;
+      File::Error error =
+          (bytes_read_ < 0) ? File::FILE_ERROR_FAILED : File::FILE_OK;
       callback.Run(error, buffer_.get(), bytes_read_);
     }
   }
 
  private:
   scoped_ptr<char[]> buffer_;
   int bytes_to_read_;
   int bytes_read_;
   DISALLOW_COPY_AND_ASSIGN(ReadHelper);
 };
 
 class WriteHelper {
  public:
   WriteHelper(const char* buffer, int bytes_to_write)
       : buffer_(new char[bytes_to_write]),
         bytes_to_write_(bytes_to_write),
         bytes_written_(0) {
     memcpy(buffer_.get(), buffer, bytes_to_write);
   }
 
   void RunWork(PlatformFile file, int64 offset) {
     bytes_written_ = WritePlatformFile(file, offset, buffer_.get(),
                                        bytes_to_write_);
   }
 
   void Reply(const FileUtilProxy::WriteCallback& callback) {
     if (!callback.is_null()) {
-      PlatformFileError error =
-          (bytes_written_ < 0) ? PLATFORM_FILE_ERROR_FAILED : PLATFORM_FILE_OK;
+      File::Error error =
+          (bytes_written_ < 0) ? File::FILE_ERROR_FAILED : File::FILE_OK;
       callback.Run(error, bytes_written_);
     }
   }
 
  private:
   scoped_ptr<char[]> buffer_;
   int bytes_to_write_;
   int bytes_written_;
   DISALLOW_COPY_AND_ASSIGN(WriteHelper);
 };
 
-PlatformFileError CreateOrOpenAdapter(
+File::Error CreateOrOpenAdapter(
     const FilePath& file_path, int file_flags,
     PlatformFile* file_handle, bool* created) {
   DCHECK(file_handle);
   DCHECK(created);
   if (!DirectoryExists(file_path.DirName())) {
     // If its parent does not exist, should return NOT_FOUND error.
-    return PLATFORM_FILE_ERROR_NOT_FOUND;
+    return File::FILE_ERROR_NOT_FOUND;
   }
-  PlatformFileError error = PLATFORM_FILE_OK;
-  *file_handle = CreatePlatformFile(file_path, file_flags, created, &error);
+  File::Error error = File::FILE_OK;
+  *file_handle =
+      CreatePlatformFile(file_path, file_flags, created,
+                         reinterpret_cast<PlatformFileError*>(&error));
   return error;
 }
 
-PlatformFileError CloseAdapter(PlatformFile file_handle) {
+File::Error CloseAdapter(PlatformFile file_handle) {
   if (!ClosePlatformFile(file_handle)) {
-    return PLATFORM_FILE_ERROR_FAILED;
+    return File::FILE_ERROR_FAILED;
   }
-  return PLATFORM_FILE_OK;
+  return File::FILE_OK;
 }
 
-PlatformFileError DeleteAdapter(const FilePath& file_path, bool recursive) {
+File::Error DeleteAdapter(const FilePath& file_path, bool recursive) {
   if (!PathExists(file_path)) {
-    return PLATFORM_FILE_ERROR_NOT_FOUND;
+    return File::FILE_ERROR_NOT_FOUND;
   }
   if (!base::DeleteFile(file_path, recursive)) {
     if (!recursive && !base::IsDirectoryEmpty(file_path)) {
-      return PLATFORM_FILE_ERROR_NOT_EMPTY;
+      return File::FILE_ERROR_NOT_EMPTY;
     }
-    return PLATFORM_FILE_ERROR_FAILED;
+    return File::FILE_ERROR_FAILED;
   }
-  return PLATFORM_FILE_OK;
+  return File::FILE_OK;
 }
 
 }  // namespace
 
 // static
 bool FileUtilProxy::CreateOrOpen(
     TaskRunner* task_runner,
     const FilePath& file_path, int file_flags,
     const CreateOrOpenCallback& callback) {
   return RelayCreateOrOpen(
@@ skipping 172 matching lines @@
 bool FileUtilProxy::RelayClose(
     TaskRunner* task_runner,
     const CloseTask& close_task,
     PlatformFile file_handle,
     const StatusCallback& callback) {
   return base::PostTaskAndReplyWithResult(
       task_runner, FROM_HERE, Bind(close_task, file_handle), callback);
 }
 
 }  // namespace base