Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Unified Diff: webkit/fileapi/file_system_file_util.h

Issue 9564047: Add FileUtileHelper (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: s/DeletePath/Delete/g Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webkit/fileapi/cross_file_util_helper.cc ('k') | webkit/fileapi/file_system_file_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/fileapi/file_system_file_util.h
diff --git a/webkit/fileapi/file_system_file_util.h b/webkit/fileapi/file_system_file_util.h
index 1c76984d053ff2e5e3d65d62c55bf52a8a2d72d3..f71449a02e5a382f248d397d02540950077b5106 100644
--- a/webkit/fileapi/file_system_file_util.h
+++ b/webkit/fileapi/file_system_file_util.h
@@ -47,22 +47,6 @@ class FileSystemFileUtil {
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
@@ -194,19 +178,6 @@ class FileSystemFileUtil {
FileSystemFileUtil();
explicit FileSystemFileUtil(FileSystemFileUtil* underlying_file_util);
- // 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(
- FileSystemOperationContext* context,
- const FileSystemPath& path);
-
// TODO(kinuko): We should stop FileUtil layering.
FileSystemFileUtil* underlying_file_util() const {
return underlying_file_util_.get();
« no previous file with comments | « webkit/fileapi/cross_file_util_helper.cc ('k') | webkit/fileapi/file_system_file_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698