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

Unified Diff: webkit/fileapi/sandbox_mount_point_provider.h

Issue 9016020: Cleanup FileSystemOperation for preparing for adding FSO-factory method (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/sandbox_mount_point_provider.h
diff --git a/webkit/fileapi/sandbox_mount_point_provider.h b/webkit/fileapi/sandbox_mount_point_provider.h
index 8fa4719f75d683794dab9969731ccca90cbf6fc5..f6efcfa63c63ed693c7b0787b48b60a4d591d8c3 100644
--- a/webkit/fileapi/sandbox_mount_point_provider.h
+++ b/webkit/fileapi/sandbox_mount_point_provider.h
@@ -37,7 +37,8 @@ class SandboxMountPointProvider
: public FileSystemMountPointProvider,
public FileSystemQuotaUtil {
public:
- typedef FileSystemMountPointProvider::GetRootPathCallback GetRootPathCallback;
+ typedef FileSystemMountPointProvider::ValidateFileSystemCallback
+ ValidateFileSystemCallback;
// Origin enumerator interface.
// An instance of this interface is assumed to be called on the file thread.
@@ -67,20 +68,20 @@ class SandboxMountPointProvider
virtual ~SandboxMountPointProvider();
// FileSystemMountPointProvider overrides.
- virtual bool IsAccessAllowed(
- const GURL& origin_url,
- FileSystemType type,
- const FilePath& virtual_path) OVERRIDE;
- virtual void ValidateFileSystemRootAndGetURL(
+ virtual void ValidateFileSystemRoot(
const GURL& origin_url,
FileSystemType type,
bool create,
- const GetRootPathCallback& callback) OVERRIDE;
- virtual FilePath ValidateFileSystemRootAndGetPathOnFileThread(
+ const ValidateFileSystemCallback& callback) OVERRIDE;
+ virtual FilePath GetFileSystemRootPathOnFileThread(
const GURL& origin_url,
FileSystemType type,
- const FilePath& unused,
+ const FilePath& virtual_path,
bool create) OVERRIDE;
+ virtual bool IsAccessAllowed(
+ const GURL& origin_url,
+ FileSystemType type,
+ const FilePath& virtual_path) OVERRIDE;
virtual bool IsRestrictedFileName(const FilePath& filename) const OVERRIDE;
virtual std::vector<FilePath> GetRootDirectories() const OVERRIDE;
virtual FileSystemFileUtil* GetFileUtil() OVERRIDE;
@@ -155,8 +156,6 @@ class SandboxMountPointProvider
// filesystem.
bool IsAllowedScheme(const GURL& url) const;
- class GetFileSystemRootPathTask;
-
friend class FileSystemTestOriginHelper;
friend class SandboxMountPointProviderMigrationTest;
friend class SandboxMountPointProviderOriginEnumeratorTest;

Powered by Google App Engine
This is Rietveld 408576698