diff options
| author | kinaba@chromium.org <kinaba@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-11 02:18:40 +0000 |
|---|---|---|
| committer | kinaba@chromium.org <kinaba@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-11 02:18:40 +0000 |
| commit | 46d5304577719a2cd312b28db6410abd23fd2f45 (patch) | |
| tree | 1bed0a5b92c5eb2ed6269bcb954309a2f556660f /webkit/chromeos | |
| parent | 60d814f663a79e187f27cbdb1808e0cded72e018 (diff) | |
| download | chromium_src-46d5304577719a2cd312b28db6410abd23fd2f45.zip chromium_src-46d5304577719a2cd312b28db6410abd23fd2f45.tar.gz chromium_src-46d5304577719a2cd312b28db6410abd23fd2f45.tar.bz2 | |
Refactor FileSystemOperation to take callback for each method.
This patch is the first step for supporting cross-filesystem copy/move on
the Filesystem API implementation. To accomplish it, I'm planning to
crack FileSystemOperation::{Move,Copy} to a series of other FSO operations.
For it, per-method callback is more handy.
BUG=110121
TEST=*File*
Review URL: http://codereview.chromium.org/9372044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@121620 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/chromeos')
| -rw-r--r-- | webkit/chromeos/fileapi/cros_mount_point_provider.cc | 5 | ||||
| -rw-r--r-- | webkit/chromeos/fileapi/cros_mount_point_provider.h | 1 |
2 files changed, 1 insertions, 5 deletions
diff --git a/webkit/chromeos/fileapi/cros_mount_point_provider.cc b/webkit/chromeos/fileapi/cros_mount_point_provider.cc index 011ed76..07ab125 100644 --- a/webkit/chromeos/fileapi/cros_mount_point_provider.cc +++ b/webkit/chromeos/fileapi/cros_mount_point_provider.cc @@ -16,7 +16,6 @@ #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebFileSystem.h" #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h" #include "webkit/chromeos/fileapi/file_access_permissions.h" -#include "webkit/fileapi/file_system_callback_dispatcher.h" #include "webkit/fileapi/file_system_operation.h" #include "webkit/fileapi/file_system_util.h" #include "webkit/fileapi/native_file_util.h" @@ -185,13 +184,11 @@ CrosMountPointProvider::CreateFileSystemOperation( const GURL& origin_url, fileapi::FileSystemType file_system_type, const FilePath& virtual_path, - scoped_ptr<fileapi::FileSystemCallbackDispatcher> dispatcher, base::MessageLoopProxy* file_proxy, fileapi::FileSystemContext* context) const { // TODO(satorux,zel): instantiate appropriate FileSystemOperation that // implements async/remote operations. - return new fileapi::FileSystemOperation( - dispatcher.Pass(), file_proxy, context); + return new fileapi::FileSystemOperation(file_proxy, context); } bool CrosMountPointProvider::GetVirtualPath(const FilePath& filesystem_path, diff --git a/webkit/chromeos/fileapi/cros_mount_point_provider.h b/webkit/chromeos/fileapi/cros_mount_point_provider.h index a509c11..1ebdab8 100644 --- a/webkit/chromeos/fileapi/cros_mount_point_provider.h +++ b/webkit/chromeos/fileapi/cros_mount_point_provider.h @@ -57,7 +57,6 @@ class CrosMountPointProvider const GURL& origin_url, fileapi::FileSystemType file_system_type, const FilePath& virtual_path, - scoped_ptr<fileapi::FileSystemCallbackDispatcher> dispatcher, base::MessageLoopProxy* file_proxy, fileapi::FileSystemContext* context) const OVERRIDE; |