diff options
author | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-11 10:26:10 +0000 |
---|---|---|
committer | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-11 10:26:10 +0000 |
commit | 60f60f854736bd82e9e8d5967ec160df38fcd660 (patch) | |
tree | 41fdbe702084398954904afe6993be40c8c4e909 /webkit/fileapi/file_system_mount_point_provider_unittest.cc | |
parent | 9569b218572ab58deb22546cde49d2bd8ea20c48 (diff) | |
download | chromium_src-60f60f854736bd82e9e8d5967ec160df38fcd660.zip chromium_src-60f60f854736bd82e9e8d5967ec160df38fcd660.tar.gz chromium_src-60f60f854736bd82e9e8d5967ec160df38fcd660.tar.bz2 |
Cleanup FileSystemOperation for preparing for adding FSO-factory method
1. Move OpenFileSystem() to FileSystemContext
2. Change Cancel() not to take another FileSystemOperation
These two changes are made so that all the operations that require FileSystemOperation take target path URL.
3. Did some related code cleanups in FileSystemMountPointProvider:
- Renamed ValidateFileSystemRootAndGetURL() to ValidateFileSystemRoot() as we no longer need to return RootURL
- Renamed ValidateFileSystemRootAndGetPathOnFileThread() to GetFileSystemRootPathOnFileThread() for the sake of simplicity
- Reimplemented SandboxMPP::GetRootPathTask using PostTaskAndReply
Patch from http://codereview.chromium.org/9004019/
BUG=none
TEST=existing tests should pass
Review URL: http://codereview.chromium.org/9016020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117197 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/fileapi/file_system_mount_point_provider_unittest.cc')
-rw-r--r-- | webkit/fileapi/file_system_mount_point_provider_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/webkit/fileapi/file_system_mount_point_provider_unittest.cc b/webkit/fileapi/file_system_mount_point_provider_unittest.cc index c000535..2857c53 100644 --- a/webkit/fileapi/file_system_mount_point_provider_unittest.cc +++ b/webkit/fileapi/file_system_mount_point_provider_unittest.cc @@ -237,7 +237,7 @@ class FileSystemMountPointProviderTest : public testing::Test { if (type == kFileSystemTypeExternal) virtual_path = FilePath(kVirtualPath); FilePath returned_root_path = - provider(type)->ValidateFileSystemRootAndGetPathOnFileThread( + provider(type)->GetFileSystemRootPathOnFileThread( origin_url, type, virtual_path, create); if (root_path) *root_path = returned_root_path; |