Rename FileUtilProxy::Create to EnsureFileExists

per follow-up discussion on http://codereview.chromium.org/3717001/show

Rename FileUtilProxy::Create to EnsureFileExists to make it clear that
it doesn't open the file (or doesn't leave a file handle opened).

Also fixes CreatePlatformFile to set |created| correctly when
PLATFORM_FILE_CREATE flag is given (I think this is the correct
behavior).

BUG=none
TEST=FileSystemOperationTest.*

Review URL: http://codereview.chromium.org/3743004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62683 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 6 deletions

diff --git a/webkit/fileapi/file_system_operation.h b/webkit/fileapi/file_system_operation.h
index bfa4141..6b4599a 100644
--- a/webkit/fileapi/file_system_operation.h
+++ b/webkit/fileapi/file_system_operation.h
@@ -82,13 +82,13 @@ class FileSystemOperation {
   scoped_refptr<base::MessageLoopProxy> proxy_;
 
  private:
-  // Callbacks for above methods.
-  void DidCreateFileExclusive(
-      base::PlatformFileError rv, base::PassPlatformFile file, bool created);
+  // Callback for CreateFile for |exclusive|=true cases.
+  void DidEnsureFileExistsExclusive(base::PlatformFileError rv,
+                                    bool created);
 
-  // Returns success even if the file already existed.
-  void DidCreateFileNonExclusive(
-      base::PlatformFileError rv, base::PassPlatformFile file, bool created);
+  // Callback for CreateFile for |exclusive|=false cases.
+  void DidEnsureFileExistsNonExclusive(base::PlatformFileError rv,
+                                       bool created);
 
   // Generic callback that translates platform errors to WebKit error codes.
   void DidFinishFileOperation(base::PlatformFileError rv);