From d292409206bdfee695b72eae60279a15acb0acd8 Mon Sep 17 00:00:00 2001 From: "kinuko@chromium.org" Date: Tue, 5 Oct 2010 05:33:39 +0000 Subject: Support removeRecursively and new copy/move behaviors added to the spec recently. http://lists.w3.org/Archives/Public/public-webapps/2010JulSep/1101.html > For a move/copy of a file on top of existing file, or a directory on > top of an existing empty directory, you get an automatic overwrite. > A move/copy of a file on top of an existing directory, or of a > directory on top of an existing file, will always fail. > A move/copy of a file or directory on top of an existing non-empty > directory will always fail. BUG=32277 TEST=FileSystemOperationTest.* Review URL: http://codereview.chromium.org/3567012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61480 0039d316-1c4b-4281-b951-d872f2087c98 --- webkit/fileapi/file_system_operation.h | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'webkit/fileapi/file_system_operation.h') diff --git a/webkit/fileapi/file_system_operation.h b/webkit/fileapi/file_system_operation.h index ab6d56c..ba65b7d 100644 --- a/webkit/fileapi/file_system_operation.h +++ b/webkit/fileapi/file_system_operation.h @@ -45,9 +45,6 @@ class FileSystemOperation { void Copy(const FilePath& src_path, const FilePath& dest_path); - // If |dest_path| exists and is a directory, behavior is unspecified or - // varies for different platforms. TODO(kkanetkar): Fix this as per spec - // when it is addressed in spec. void Move(const FilePath& src_path, const FilePath& dest_path); @@ -59,7 +56,7 @@ class FileSystemOperation { void ReadDirectory(const FilePath& path); - void Remove(const FilePath& path); + void Remove(const FilePath& path, bool recursive); void Write(const FilePath& path, const GURL& blob_url, int64 offset); -- cgit v1.1