Add FileUtileHelper

webkit/fileapi/file_system_file_util.h

 // 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.
 
 #ifndef WEBKIT_FILEAPI_FILE_SYSTEM_FILE_UTIL_H_
 #define WEBKIT_FILEAPI_FILE_SYSTEM_FILE_UTIL_H_
 
 #include "base/file_path.h"
 #include "base/file_util_proxy.h"
 #include "base/platform_file.h"
@@ skipping 29 matching lines @@
   };
 
   class EmptyFileEnumerator : public AbstractFileEnumerator {
     virtual FilePath Next() OVERRIDE { return FilePath(); }
     virtual int64 Size() OVERRIDE { return 0; }
     virtual bool IsDirectory() OVERRIDE { return false; }
   };
 
   virtual ~FileSystemFileUtil();
 
-  // Deletes a file or a directory.
-  // It is an error to delete a non-empty directory with recursive=false.
-  //
-  // This method calls one of the following methods depending on whether the
-  // target is a directory or not, and whether the |recursive| flag is given or
-  // not.
-  // - (virtual) DeleteFile,
-  // - (virtual) DeleteSingleDirectory or
-  // - (non-virtual) DeleteDirectoryRecursive which calls two methods above.
-  //
-  // TODO(kinuko): Move this implementation outside FileSystemFileUtil.
-  PlatformFileError Delete(
-      FileSystemOperationContext* context,
-      const FileSystemPath& path,
-      bool recursive);
-
   // Creates or opens a file with the given flags.
   // If PLATFORM_FILE_CREATE is set in |file_flags| it always tries to create
   // a new file at the given |path| and calls back with
   // PLATFORM_FILE_ERROR_FILE_EXISTS if the |path| already exists.
   virtual PlatformFileError CreateOrOpen(
       FileSystemOperationContext* context,
       const FileSystemPath& path,
       int file_flags,
       PlatformFile* file_handle,
       bool* created);
@@ skipping 120 matching lines @@
 
   // Deletes a directory and all entries under the directory.
   //
   // This method is called from Delete.  It internally calls two following
   // virtual methods,
   // - (virtual) DeleteFile to delete files, and
   // - (virtual) DeleteSingleDirectory to delete empty directories after all
   // the files are deleted.
   //
   // TODO(kinuko): Move this method out of this class.
   PlatformFileError DeleteDirectoryRecursive(

[kinuko, 2012/03/02 19:29:22]

Should this be also removed?

[tzik, 2012/03/02 21:25:47]

Done.

       FileSystemOperationContext* context,
       const FileSystemPath& path);
 
   // TODO(kinuko): We should stop FileUtil layering.
   FileSystemFileUtil* underlying_file_util() const {
     return underlying_file_util_.get();
   }
 
  private:
   scoped_ptr<FileSystemFileUtil> underlying_file_util_;
 
   DISALLOW_COPY_AND_ASSIGN(FileSystemFileUtil);
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_FILE_SYSTEM_FILE_UTIL_H_