[Refactor] to rename and re-layer the file_util stack layers.

webkit/fileapi/native_file_util.h

 // Copyright (c) 2011 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 WEBKIT_FILEAPI_LOCAL_FILE_SYSTEM_FILE_UTIL_H_
-#define WEBKIT_FILEAPI_LOCAL_FILE_SYSTEM_FILE_UTIL_H_
+#ifndef WEBKIT_FILEAPI_NATIVE_FILE_UTIL_H_
+#define WEBKIT_FILEAPI_NATIVE_FILE_UTIL_H_
 
-#include <vector>
-
-#include "base/callback.h"
 #include "base/file_path.h"
-#include "base/file_util.h"
 #include "base/file_util_proxy.h"
-#include "base/logging.h"
-#include "base/memory/scoped_ptr.h"
 #include "base/platform_file.h"
-#include "webkit/fileapi/file_system_file_util.h"
-#include "webkit/fileapi/file_system_types.h"
+#include "webkit/fileapi/fileapi_file_util.h"
 
 namespace base {
-struct PlatformFileInfo;
-class MessageLoopProxy;
 class Time;
 }
 
-class GURL;
-
 namespace fileapi {
 
 using base::PlatformFile;
 using base::PlatformFileError;
-
 class FileSystemOperationContext;
 
-// An instance of this class is created and owned by *MountPointProvider.
-class LocalFileSystemFileUtil : public FileSystemFileUtil {
+class NativeFileUtil : public FileUtil {

[ericu, 2011/08/03 22:59:26]

This needs a class comment that says what this particular subclass is used for.

[Dai Mikurube (NOT FULLTIME), 2011/08/04 03:54:48]

Done.

  public:
-  // |underlying_file_util| is owned by the instance.  It will be deleted by
-  // the owner instance.  For example, it can be instanciated as follows:
-  // FileSystemFileUtil* file_system_file_util =
-  //     new LocalFileSystemFileUtil(new FileSystemFileUtil());
-  explicit LocalFileSystemFileUtil(FileSystemFileUtil* underlying_file_util);
-  virtual ~LocalFileSystemFileUtil();
+  NativeFileUtil() {}
+  virtual ~NativeFileUtil() {}
 
   virtual PlatformFileError CreateOrOpen(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path,
       int file_flags,
       PlatformFile* file_handle,
       bool* created);
 
+  virtual PlatformFileError Close(
+      FileSystemOperationContext* unused,
+      PlatformFile);
+
   virtual PlatformFileError EnsureFileExists(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path, bool* created);

[ericu, 2011/08/03 22:59:26]

Please add OVERRIDE tags on all overridden methods in all files.

[Dai Mikurube (NOT FULLTIME), 2011/08/04 03:54:48]

Done.

 
   virtual PlatformFileError GetLocalFilePath(
-      FileSystemOperationContext* context,
-      const FilePath& virtual_file,
+      FileSystemOperationContext* unused,
+      const FilePath& virtual_path,
       FilePath* local_path);
 
   virtual PlatformFileError GetFileInfo(
-      FileSystemOperationContext* context,
-      const FilePath& file,
+      FileSystemOperationContext* unused,
+      const FilePath& file_,
       base::PlatformFileInfo* file_info,
-      FilePath* platform_file);
+      FilePath* platform_path);
 
   virtual PlatformFileError ReadDirectory(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path,
       std::vector<base::FileUtilProxy::Entry>* entries);
 
   virtual PlatformFileError CreateDirectory(
       FileSystemOperationContext* context,
       const FilePath& file_path,
       bool exclusive,
       bool recursive);
 
   virtual PlatformFileError CopyOrMoveFile(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& src_file_path,
       const FilePath& dest_file_path,
       bool copy);
 
   virtual PlatformFileError CopyInForeignFile(
-        FileSystemOperationContext* context,
+        FileSystemOperationContext* unused,
         const FilePath& src_file_path,
         const FilePath& dest_file_path);
 
   virtual PlatformFileError DeleteFile(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path);
 
   virtual PlatformFileError DeleteSingleDirectory(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path);
 
   virtual PlatformFileError Touch(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path,
       const base::Time& last_access_time,
       const base::Time& last_modified_time);
 
   virtual PlatformFileError Truncate(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& path,
       int64 length);
 
   virtual bool PathExists(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path);
 
   virtual bool DirectoryExists(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path);
 
   virtual bool IsDirectoryEmpty(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& file_path);
 
   virtual AbstractFileEnumerator* CreateFileEnumerator(
-      FileSystemOperationContext* context,
+      FileSystemOperationContext* unused,
       const FilePath& root_path);
 
  private:
-  // Given the filesystem's root URL and a virtual path, produces a real, full
-  // local path.
-  FilePath GetLocalPath(
-      FileSystemOperationContext* context,
-      const GURL& origin_url,
-      FileSystemType type,
-      const FilePath& virtual_path);
-
-  scoped_ptr<FileSystemFileUtil> underlying_file_util_;
-
-  DISALLOW_COPY_AND_ASSIGN(LocalFileSystemFileUtil);
+  DISALLOW_COPY_AND_ASSIGN(NativeFileUtil);
 };
 
 }  // namespace fileapi
 
-#endif  // WEBKIT_FILEAPI_LOCAL_FILE_SYSTEM_FILE_UTIL_H_
+#endif  // WEBKIT_FILEAPI_NATIVE_FILE_UTIL_H_