diff options
author | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-20 12:42:44 +0000 |
---|---|---|
committer | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-20 12:42:44 +0000 |
commit | 3c48b53f15d8438e6cac36dc28cd491e488424a5 (patch) | |
tree | 4ff1bababd8bbde1036fffb4a2e38275e0e65898 /webkit/fileapi/sandbox_mount_point_provider.h | |
parent | fe5ed19f21f9773d8c93d115035097a5f72161f5 (diff) | |
download | chromium_src-3c48b53f15d8438e6cac36dc28cd491e488424a5.zip chromium_src-3c48b53f15d8438e6cac36dc28cd491e488424a5.tar.gz chromium_src-3c48b53f15d8438e6cac36dc28cd491e488424a5.tar.bz2 |
Cleanup: Reorder methods in SandboxMountPointProvider
- To follow coding style to put overriding methods together in .h
- To make the method order match in .cc/.h
BUG=none
TEST=no functional changes
Review URL: http://codereview.chromium.org/8965041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115118 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/fileapi/sandbox_mount_point_provider.h')
-rw-r--r-- | webkit/fileapi/sandbox_mount_point_provider.h | 42 |
1 files changed, 16 insertions, 26 deletions
diff --git a/webkit/fileapi/sandbox_mount_point_provider.h b/webkit/fileapi/sandbox_mount_point_provider.h index 11a4773..5fa76ab2 100644 --- a/webkit/fileapi/sandbox_mount_point_provider.h +++ b/webkit/fileapi/sandbox_mount_point_provider.h @@ -49,51 +49,43 @@ class SandboxMountPointProvider virtual bool HasFileSystemType(FileSystemType type) const = 0; }; + // The legacy [pre-obfuscation] FileSystem directory name, kept around for + // migration and migration testing. + static const FilePath::CharType kOldFileSystemDirectory[]; + // The FileSystem directory name. + static const FilePath::CharType kNewFileSystemDirectory[]; + // Where we move the old filesystem directory if migration fails. + static const FilePath::CharType kRenamedOldFileSystemDirectory[]; + SandboxMountPointProvider( FileSystemPathManager* path_manager, scoped_refptr<base::MessageLoopProxy> file_message_loop, const FilePath& profile_path); virtual ~SandboxMountPointProvider(); - // Checks if access to |virtual_path| is allowed from |origin_url|. - virtual bool IsAccessAllowed(const GURL& origin_url, - FileSystemType type, - const FilePath& virtual_path) OVERRIDE; - - // 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. + // FileSystemMountPointProvider overrides. + virtual bool IsAccessAllowed( + const GURL& origin_url, + FileSystemType type, + const FilePath& virtual_path) OVERRIDE; virtual void ValidateFileSystemRootAndGetURL( const GURL& origin_url, FileSystemType type, bool create, const FileSystemPathManager::GetRootPathCallback& callback) OVERRIDE; - - // 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& unused, bool create) OVERRIDE; - - // The legacy [pre-obfuscation] FileSystem directory name, kept around for - // migration and migration testing. - static const FilePath::CharType kOldFileSystemDirectory[]; - // The FileSystem directory name. - static const FilePath::CharType kNewFileSystemDirectory[]; - // Where we move the old filesystem directory if migration fails. - static const FilePath::CharType kRenamedOldFileSystemDirectory[]; + virtual bool IsRestrictedFileName(const FilePath& filename) const OVERRIDE; + virtual std::vector<FilePath> GetRootDirectories() const OVERRIDE; + virtual FileSystemFileUtil* GetFileUtil() OVERRIDE; FilePath old_base_path() const; FilePath new_base_path() const; FilePath renamed_old_base_path() const; - // Checks if a given |name| contains any restricted names/chars in it. - virtual bool IsRestrictedFileName(const FilePath& filename) const OVERRIDE; - - virtual std::vector<FilePath> GetRootDirectories() const OVERRIDE; - // Returns an origin enumerator of this provider. // This method can only be called on the file thread. OriginEnumerator* CreateOriginEnumerator() const; @@ -109,8 +101,6 @@ class SandboxMountPointProvider FileSystemType type, bool create) const; - virtual FileSystemFileUtil* GetFileUtil() OVERRIDE; - // Deletes the data on the origin and reports the amount of deleted data // to the quota manager via |proxy|. bool DeleteOriginDataOnFileThread( |