Cleanup FileSystemOperation for preparing for adding FSO-factory method

webkit/fileapi/file_system_mount_point_provider.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_MOUNT_POINT_PROVIDER_H_
 #define WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_
 
 #include <string>
 #include <vector>
 
 #include "base/callback_forward.h"
+#include "base/platform_file.h"

[tzik, 2012/01/10 08:31:13]

sort?

[kinuko, 2012/01/10 09:48:53]

Done.

 #include "base/file_path.h"
 #include "webkit/fileapi/file_system_types.h"
 
 class GURL;
 
 namespace fileapi {
 
 class FileSystemFileUtil;
 
 // An interface to provide mount-point-specific path-related utilities
 // and specialized FileSystemFileUtil instance.
 class FileSystemMountPointProvider {
  public:
-  // Callback for GetFileSystemRootPath.
-  // If the request is accepted and the root filesystem for the origin exists
-  // the callback is called with success=true and valid root_path and name.
-  // If the request is accepted, |create| is specified for
-  // GetFileSystemRootPath, and the root directory does not exist, it creates
-  // a new one and calls back with success=true if the creation has succeeded.
-  typedef base::Callback<void(bool /* success */,
-                              const FilePath& /* root_path */,
-                              const std::string& /* name */)>
-      GetRootPathCallback;
+  // Callback for ValidateFileSystemRoot.
+  typedef base::Callback<void(base::PlatformFileError error)>
+      ValidateFileSystemCallback;
   virtual ~FileSystemMountPointProvider() {}
 
+  // Validates the filesystem for the given |origin_url| and |type|.
+  // This verifies if it is allowed to request (or create) the filesystem
+  // and if it can access (or create) the root directory of the mount point.
+  // If |create| is true this may also create the root directory for
+  // the filesystem if it doesn't exist.
+  virtual void ValidateFileSystemRoot(
+      const GURL& origin_url,
+      FileSystemType type,
+      bool create,
+      const ValidateFileSystemCallback& callback) = 0;
+
+  // Retrieves the root path of the filesystem specified by the given
+  // |origin_url| and |type| on the file thread.
+  // If |create| is true this may also create the root directory for
+  // the filesystem if it doesn't exist.
+  virtual FilePath GetFileSystemRootPathOnFileThread(
+      const GURL& origin_url,
+      FileSystemType type,
+      const FilePath& virtual_path,
+      bool create) = 0;
+
   // Checks if access to |virtual_path| is allowed from |origin_url|.
   virtual bool IsAccessAllowed(const GURL& origin_url,
                                FileSystemType type,
                                const FilePath& virtual_path) = 0;
 
-  // Retrieves the root path for the given |origin_url| and |type|, and
-  // calls the given |callback| with the root path and name.
-  // If |create| is true this also creates the directory if it doesn't exist.
-  virtual void ValidateFileSystemRootAndGetURL(
-      const GURL& origin_url,
-      FileSystemType type,
-      bool create,
-      const GetRootPathCallback& callback) = 0;
-
-  // Like GetFileSystemRootPath, but synchronous, and can be called only while
-  // running on the file thread.
-  virtual FilePath ValidateFileSystemRootAndGetPathOnFileThread(
-      const GURL& origin_url,
-      FileSystemType type,
-      const FilePath& virtual_path,
-      bool create) = 0;
-
   // Checks if a given |name| contains any restricted names/chars in it.
   // Callable on any thread.
   virtual bool IsRestrictedFileName(const FilePath& filename) const = 0;
 
   // Returns the list of top level directories that are exposed by this
   // provider. This list is used to set appropriate child process file access
   // permissions.
   virtual std::vector<FilePath> GetRootDirectories() const = 0;
 
   // Returns the specialized FileSystemFileUtil for this mount point.
@@ skipping 20 matching lines @@
   virtual void RemoveMountPoint(FilePath mount_point) = 0;
   // Gets virtual path by known filesystem path. Returns false when filesystem
   // path is not exposed by this provider.
   virtual bool GetVirtualPath(const FilePath& filesystem_path,
                               FilePath* virtual_path) = 0;
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_FILE_SYSTEM_MOUNT_POINT_PROVIDER_H_