Remove PlatformFile from fileapi::FileSystemFileUtil

webkit/browser/fileapi/local_file_util.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 "webkit/browser/fileapi/local_file_util.h"
 
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/file_util_proxy.h"
 #include "url/gurl.h"
@@ skipping 58 matching lines @@
 }
 
 bool LocalFileEnumerator::IsDirectory() {
   return file_util_info_.IsDirectory();
 }
 
 LocalFileUtil::LocalFileUtil() {}
 
 LocalFileUtil::~LocalFileUtil() {}
 
-base::File::Error LocalFileUtil::CreateOrOpen(
+base::File LocalFileUtil::CreateOrOpen(
     FileSystemOperationContext* context,
-    const FileSystemURL& url, int file_flags,
-    base::PlatformFile* file_handle, bool* created) {
-  *created = false;
+    const FileSystemURL& url, int file_flags) {
   base::FilePath file_path;
   base::File::Error error = GetLocalFilePath(context, url, &file_path);
   if (error != base::File::FILE_OK)
-    return error;
+    return base::File(error);
   // Disallow opening files in symlinked paths.
   if (base::IsLink(file_path))
-    return base::File::FILE_ERROR_NOT_FOUND;
+    return base::File(base::File::FILE_ERROR_NOT_FOUND);
 
-  // TODO(rvargas): make FileSystemFileUtil use base::File.
-  base::File file = NativeFileUtil::CreateOrOpen(file_path, file_flags);
-  if (!file.IsValid())
-    return file.error_details();
-
-  *created = file.created();
-  *file_handle = file.TakePlatformFile();
-  return base::File::FILE_OK;
-}
-
-base::File::Error LocalFileUtil::Close(FileSystemOperationContext* context,
-                                       base::PlatformFile file) {
-  base::File auto_closed(file);
-  return base::File::FILE_OK;
+  return NativeFileUtil::CreateOrOpen(file_path, file_flags);
 }
 
 base::File::Error LocalFileUtil::EnsureFileExists(
     FileSystemOperationContext* context,
     const FileSystemURL& url,
     bool* created) {
   base::FilePath file_path;
   base::File::Error error = GetLocalFilePath(context, url, &file_path);
   if (error != base::File::FILE_OK)
     return error;
@@ skipping 151 matching lines @@
     base::FilePath* platform_path) {
   DCHECK(file_info);
   // We're just returning the local file information.
   *error = GetFileInfo(context, url, file_info, platform_path);
   if (*error == base::File::FILE_OK && file_info->is_directory)
     *error = base::File::FILE_ERROR_NOT_A_FILE;
   return webkit_blob::ScopedFile();
 }
 
 }  // namespace fileapi