Cleanup FileSystemOperation for preparing for adding FSO-factory method

webkit/fileapi/file_system_context.h

Index: webkit/fileapi/file_system_context.h
diff --git a/webkit/fileapi/file_system_context.h b/webkit/fileapi/file_system_context.h
index 0864f7f0a2601dddb595e85fd4b9fd2d1f6da5a3..924dfb53cc99f6683174c6a53a01727674976a63 100644
--- a/webkit/fileapi/file_system_context.h
+++ b/webkit/fileapi/file_system_context.h
@@ -7,6 +7,7 @@
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/platform_file.h"
 #include "webkit/fileapi/file_system_types.h"
 #include "webkit/quota/special_storage_policy.h"
 
@@ -24,6 +25,7 @@ class QuotaManagerProxy;
 namespace fileapi {
 
 class ExternalFileSystemMountPointProvider;
+class FileSystemCallbackDispatcher;
 class FileSystemContext;
 class FileSystemFileUtil;
 class FileSystemMountPointProvider;
@@ -85,6 +87,18 @@ class FileSystemContext
   // calling GetMountPointProvider(kFileSystemTypeExternal).
   ExternalFileSystemMountPointProvider* external_provider() const;
 
+  // Opens the filesystem for the given origin and type and dispatches

[satorux1, 2011/12/27 18:56:37]

"and type and dispatches" is a bit confusing. adding a comma may be helpful.

"and type, and dispatches"

[kinuko, 2011/12/28 10:59:35]

Done.

+  // the DidOpenFileSystem callback of the given |dispatcher|.
+  // If |create| is true this may actually setup a filesystem instance
+  // (e.g. by creating the root directory or initializing the database
+  // entry etc).
+  // TODO(kinuko): replace the dispatcher with regular callback.
+  void OpenFileSystem(
+      const GURL& origin_url,
+      FileSystemType type,
+      bool create,
+      FileSystemCallbackDispatcher* dispatcher);
+
  private:
   friend struct DefaultContextDeleter;
   void DeleteOnCorrectThread() const;