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

Unified Diff: webkit/fileapi/file_system_mount_point_provider.h

Issue 9004019: Cleanup: Removing FileSystemPathManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: '' Created 9 years 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
Index: webkit/fileapi/file_system_mount_point_provider.h
diff --git a/webkit/fileapi/file_system_mount_point_provider.h b/webkit/fileapi/file_system_mount_point_provider.h
index 36768e34fca5694c905fa2115d5392f6bffa28db..4174c63369758d7e37cf2349cf9898c3316ab5cb 100644
--- a/webkit/fileapi/file_system_mount_point_provider.h
+++ b/webkit/fileapi/file_system_mount_point_provider.h
@@ -8,17 +8,30 @@
#include <string>
#include <vector>
+#include "base/callback.h"
tzik 2011/12/21 02:52:33 can be #include "base/callback_forward.h"?
kinuko 2011/12/21 13:00:50 Done.
#include "base/file_path.h"
-#include "googleurl/src/gurl.h"
-#include "webkit/fileapi/file_system_path_manager.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;
virtual ~FileSystemMountPointProvider() {}
// Checks if access to |virtual_path| is allowed from |origin_url|.
@@ -33,7 +46,7 @@ class FileSystemMountPointProvider {
const GURL& origin_url,
FileSystemType type,
bool create,
- const FileSystemPathManager::GetRootPathCallback& callback) = 0;
+ const GetRootPathCallback& callback) = 0;
// Like GetFileSystemRootPath, but synchronous, and can be called only while
// running on the file thread.

Powered by Google App Engine
This is Rietveld 408576698