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

webkit/fileapi/local_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_LOCAL_FILE_UTIL_H_
+#define WEBKIT_FILEAPI_LOCAL_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"
 
 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 LocalFileUtil : public FileSystemFileUtil {
  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();
+  // FileSystemFileUtil* file_util = new LocalFileUtil(new NativeFileUtil());
+  explicit LocalFileUtil(FileSystemFileUtil* underlying_file_util);
+  virtual ~LocalFileUtil();
 
   virtual PlatformFileError CreateOrOpen(
       FileSystemOperationContext* context,
       const FilePath& file_path,
       int file_flags,
       PlatformFile* file_handle,
-      bool* created);
+      bool* created) OVERRIDE;
 

[kinuko, 2011/08/29 07:20:00]

while you're there could you remove all the empty lines between virtual methods?

http://www.chromium.org/developers/coding-style
"When you implement an interface, group those functions in your header file in
one section, clearly labeled as implementing that interface. Be sure to preserve
the order of the virtual base class, and do not add comments or whitespace
between them, as the reader can refer to the virtual base class for more
information."

[Dai Mikurube (NOT FULLTIME), 2011/08/30 07:47:28]

Done.

   virtual PlatformFileError EnsureFileExists(
       FileSystemOperationContext* context,
-      const FilePath& file_path, bool* created);
+      const FilePath& file_path, bool* created) OVERRIDE;
 
-  virtual PlatformFileError GetLocalFilePath(
+  virtual PlatformFileError CreateDirectory(
       FileSystemOperationContext* context,
-      const FilePath& virtual_file,
-      FilePath* local_path);
+      const FilePath& file_path,
+      bool exclusive,
+      bool recursive) OVERRIDE;
 
   virtual PlatformFileError GetFileInfo(
       FileSystemOperationContext* context,
       const FilePath& file,
       base::PlatformFileInfo* file_info,
-      FilePath* platform_file);
+      FilePath* platform_file) OVERRIDE;
 
   virtual PlatformFileError ReadDirectory(
       FileSystemOperationContext* context,
       const FilePath& file_path,
-      std::vector<base::FileUtilProxy::Entry>* entries);
+      std::vector<base::FileUtilProxy::Entry>* entries) OVERRIDE;
 
-  virtual PlatformFileError CreateDirectory(
+  virtual AbstractFileEnumerator* CreateFileEnumerator(
+      FileSystemOperationContext* context,
+      const FilePath& root_path) OVERRIDE;
+
+  virtual PlatformFileError GetLocalFilePath(
+      FileSystemOperationContext* context,
+      const FilePath& virtual_file,
+      FilePath* local_path) OVERRIDE;
+
+  virtual PlatformFileError Touch(
       FileSystemOperationContext* context,
       const FilePath& file_path,
-      bool exclusive,
-      bool recursive);
+      const base::Time& last_access_time,
+      const base::Time& last_modified_time) OVERRIDE;
+
+  virtual PlatformFileError Truncate(
+      FileSystemOperationContext* context,
+      const FilePath& path,
+      int64 length) OVERRIDE;
+
+  virtual bool PathExists(
+      FileSystemOperationContext* context,
+      const FilePath& file_path) OVERRIDE;
+
+  virtual bool DirectoryExists(
+      FileSystemOperationContext* context,
+      const FilePath& file_path) OVERRIDE;
+
+  virtual bool IsDirectoryEmpty(
+      FileSystemOperationContext* context,
+      const FilePath& file_path) OVERRIDE;
 
   virtual PlatformFileError CopyOrMoveFile(
       FileSystemOperationContext* context,
       const FilePath& src_file_path,
       const FilePath& dest_file_path,
-      bool copy);
+      bool copy) OVERRIDE;
 
   virtual PlatformFileError CopyInForeignFile(
         FileSystemOperationContext* context,
         const FilePath& src_file_path,
-        const FilePath& dest_file_path);
+        const FilePath& dest_file_path) OVERRIDE;
 
   virtual PlatformFileError DeleteFile(
       FileSystemOperationContext* context,
-      const FilePath& file_path);
+      const FilePath& file_path) OVERRIDE;
 
   virtual PlatformFileError DeleteSingleDirectory(
       FileSystemOperationContext* context,
-      const FilePath& file_path);
-
-  virtual PlatformFileError Touch(
-      FileSystemOperationContext* context,
-      const FilePath& file_path,
-      const base::Time& last_access_time,
-      const base::Time& last_modified_time);
-
-  virtual PlatformFileError Truncate(
-      FileSystemOperationContext* context,
-      const FilePath& path,
-      int64 length);
-
-  virtual bool PathExists(
-      FileSystemOperationContext* context,
-      const FilePath& file_path);
-
-  virtual bool DirectoryExists(
-      FileSystemOperationContext* context,
-      const FilePath& file_path);
-
-  virtual bool IsDirectoryEmpty(
-      FileSystemOperationContext* context,
-      const FilePath& file_path);
-
-  virtual AbstractFileEnumerator* CreateFileEnumerator(
-      FileSystemOperationContext* context,
-      const FilePath& root_path);
+      const FilePath& file_path) OVERRIDE;
 
  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(LocalFileUtil);
 };
 
 }  // namespace fileapi
 
-#endif  // WEBKIT_FILEAPI_LOCAL_FILE_SYSTEM_FILE_UTIL_H_
+#endif  // WEBKIT_FILEAPI_LOCAL_FILE_UTIL_H_