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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: webkit/fileapi/file_system_operation.cc

Issue 7470037: [Refactor] to rename and re-layer the file_util stack layers. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Rebased. Created 9 years, 5 months 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
« webkit/fileapi/file_system_file_util_proxy.cc ('K') | « webkit/fileapi/file_system_operation.h ('k') | webkit/fileapi/file_system_operation_context.h » ('j') | webkit/fileapi/fileapi_file_util.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/fileapi/file_system_operation.cc
diff --git a/webkit/fileapi/file_system_operation.cc b/webkit/fileapi/file_system_operation.cc
index c2281ccc43fc28477afea4a5506e9b3b3fb211e1..071b724a12c5c5c9af9005664abc0f28367e3ae5 100644
--- a/webkit/fileapi/file_system_operation.cc
+++ b/webkit/fileapi/file_system_operation.cc
@@ -17,7 +17,7 @@
#include "webkit/fileapi/file_system_types.h"
#include "webkit/fileapi/file_system_util.h"
#include "webkit/fileapi/file_writer_delegate.h"
-#include "webkit/fileapi/local_file_system_file_util.h"
+#include "webkit/fileapi/local_file_util.h"
#include "webkit/fileapi/quota_file_util.h"
#include "webkit/quota/quota_types.h"
@@ -27,11 +27,10 @@ FileSystemOperation::FileSystemOperation(
FileSystemCallbackDispatcher* dispatcher,
scoped_refptr<base::MessageLoopProxy> proxy,
FileSystemContext* file_system_context,
- FileSystemFileUtil* file_system_file_util)
+ FileUtil* file_util)
: proxy_(proxy),
dispatcher_(dispatcher),
- file_system_operation_context_(
- file_system_context, file_system_file_util),
+ file_system_operation_context_(file_system_context, file_util),
callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
DCHECK(dispatcher);
#ifndef NDEBUG
@@ -77,19 +76,17 @@ void FileSystemOperation::CreateFile(const GURL& path,
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForWrite(
- path, true /* create */, &origin_url, &type, &virtual_path,
- &file_system_file_util)) {
+ path, true /* create */, &origin_url, &type, &virtual_path, &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
file_system_operation_context_.set_src_virtual_path(virtual_path);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
exclusive_ = exclusive;
GetUsageAndQuotaThenCallback(origin_url, callback_factory_.NewCallback(
@@ -125,20 +122,18 @@ void FileSystemOperation::CreateDirectory(const GURL& path,
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForWrite(
- path, true /* create */, &origin_url, &type, &virtual_path,
- &file_system_file_util)) {
+ path, true /* create */, &origin_url, &type, &virtual_path, &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
file_system_operation_context_.set_src_virtual_path(virtual_path);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
exclusive_ = exclusive;
recursive_ = recursive;
@@ -178,14 +173,13 @@ void FileSystemOperation::Copy(const GURL& src_path,
GURL dest_origin_url;
FileSystemType src_type;
FileSystemType dest_type;
- FileSystemFileUtil* src_file_system_file_util;
- FileSystemFileUtil* dest_file_system_file_util;
+ FileUtil* src_file_util;
+ FileUtil* dest_file_util;
if (!VerifyFileSystemPathForRead(src_path, &src_origin_url, &src_type,
- &virtual_path_0, &src_file_system_file_util) ||
+ &virtual_path_0, &src_file_util) ||
!VerifyFileSystemPathForWrite(dest_path, true /* create */,
- &dest_origin_url, &dest_type, &virtual_path_1,
- &dest_file_system_file_util)) {
+ &dest_origin_url, &dest_type, &virtual_path_1, &dest_file_util)) {
delete this;
return;
}
@@ -195,12 +189,10 @@ void FileSystemOperation::Copy(const GURL& src_path,
file_system_operation_context_.set_dest_type(dest_type);
file_system_operation_context_.set_src_virtual_path(virtual_path_0);
file_system_operation_context_.set_dest_virtual_path(virtual_path_1);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- src_file_system_file_util);
- if (!file_system_operation_context_.dest_file_system_file_util())
- file_system_operation_context_.set_dest_file_system_file_util(
- dest_file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(src_file_util);
+ if (!file_system_operation_context_.dest_file_util())
+ file_system_operation_context_.set_dest_file_util(dest_file_util);
GetUsageAndQuotaThenCallback(dest_origin_url, callback_factory_.NewCallback(
&FileSystemOperation::DelayedCopyForQuota));
@@ -237,14 +229,13 @@ void FileSystemOperation::Move(const GURL& src_path,
GURL dest_origin_url;
FileSystemType src_type;
FileSystemType dest_type;
- FileSystemFileUtil* src_file_system_file_util;
- FileSystemFileUtil* dest_file_system_file_util;
+ FileUtil* src_file_util;
+ FileUtil* dest_file_util;
if (!VerifyFileSystemPathForWrite(src_path, false, &src_origin_url, &src_type,
- &virtual_path_0, &src_file_system_file_util) ||
+ &virtual_path_0, &src_file_util) ||
!VerifyFileSystemPathForWrite(dest_path, true /* create */,
- &dest_origin_url, &dest_type, &virtual_path_1,
- &dest_file_system_file_util)) {
+ &dest_origin_url, &dest_type, &virtual_path_1, &dest_file_util)) {
delete this;
return;
}
@@ -254,12 +245,10 @@ void FileSystemOperation::Move(const GURL& src_path,
file_system_operation_context_.set_dest_type(dest_type);
file_system_operation_context_.set_src_virtual_path(virtual_path_0);
file_system_operation_context_.set_dest_virtual_path(virtual_path_1);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- src_file_system_file_util);
- if (!file_system_operation_context_.dest_file_system_file_util())
- file_system_operation_context_.set_dest_file_system_file_util(
- dest_file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(src_file_util);
+ if (!file_system_operation_context_.dest_file_util())
+ file_system_operation_context_.set_dest_file_util(dest_file_util);
GetUsageAndQuotaThenCallback(dest_origin_url, callback_factory_.NewCallback(
&FileSystemOperation::DelayedMoveForQuota));
@@ -293,17 +282,16 @@ void FileSystemOperation::DirectoryExists(const GURL& path) {
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForRead(path, &origin_url, &type, &virtual_path,
- &file_system_file_util)) {
+ &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
FileSystemFileUtilProxy::GetFileInfo(
file_system_operation_context_,
proxy_, virtual_path, callback_factory_.NewCallback(
@@ -319,17 +307,16 @@ void FileSystemOperation::FileExists(const GURL& path) {
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForRead(path, &origin_url, &type, &virtual_path,
- &file_system_file_util)) {
+ &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
FileSystemFileUtilProxy::GetFileInfo(
file_system_operation_context_,
proxy_, virtual_path, callback_factory_.NewCallback(
@@ -345,17 +332,16 @@ void FileSystemOperation::GetMetadata(const GURL& path) {
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForRead(path, &origin_url, &type, &virtual_path,
- &file_system_file_util)) {
+ &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
FileSystemFileUtilProxy::GetFileInfo(
file_system_operation_context_,
proxy_, virtual_path, callback_factory_.NewCallback(
@@ -371,17 +357,16 @@ void FileSystemOperation::ReadDirectory(const GURL& path) {
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForRead(path, &origin_url, &type, &virtual_path,
- &file_system_file_util)) {
+ &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
FileSystemFileUtilProxy::ReadDirectory(
file_system_operation_context_,
proxy_, virtual_path, callback_factory_.NewCallback(
@@ -397,17 +382,16 @@ void FileSystemOperation::Remove(const GURL& path, bool recursive) {
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForWrite(path, false /* create */, &origin_url,
- &type, &virtual_path, &file_system_file_util)) {
+ &type, &virtual_path, &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
FileSystemFileUtilProxy::Delete(
file_system_operation_context_,
proxy_, virtual_path, recursive, callback_factory_.NewCallback(
@@ -426,18 +410,17 @@ void FileSystemOperation::Write(
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForWrite(path, true /* create */, &origin_url,
- &type, &virtual_path, &file_system_file_util)) {
+ &type, &virtual_path, &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
file_system_operation_context_.set_src_virtual_path(virtual_path);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
DCHECK(blob_url.is_valid());
file_writer_delegate_.reset(new FileWriterDelegate(this, offset, proxy_));
blob_request_.reset(
@@ -476,18 +459,17 @@ void FileSystemOperation::Truncate(const GURL& path, int64 length) {
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForWrite(path, false /* create */, &origin_url,
- &type, &virtual_path, &file_system_file_util)) {
+ &type, &virtual_path, &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
file_system_operation_context_.set_src_virtual_path(virtual_path);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
length_ = length;
GetUsageAndQuotaThenCallback(origin_url, callback_factory_.NewCallback(
@@ -523,17 +505,16 @@ void FileSystemOperation::TouchFile(const GURL& path,
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (!VerifyFileSystemPathForWrite(path, true /* create */, &origin_url,
- &type, &virtual_path, &file_system_file_util)) {
+ &type, &virtual_path, &file_util)) {
delete this;
return;
}
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
FileSystemFileUtilProxy::Touch(
file_system_operation_context_,
proxy_, virtual_path, last_access_time, last_modified_time,
@@ -552,7 +533,7 @@ void FileSystemOperation::OpenFile(const GURL& path,
FilePath virtual_path;
GURL origin_url;
FileSystemType type;
- FileSystemFileUtil* file_system_file_util;
+ FileUtil* file_util;
if (file_flags & (
(base::PLATFORM_FILE_ENUMERATE | base::PLATFORM_FILE_TEMPORARY |
base::PLATFORM_FILE_HIDDEN))) {
@@ -566,13 +547,13 @@ void FileSystemOperation::OpenFile(const GURL& path,
base::PLATFORM_FILE_DELETE_ON_CLOSE |
base::PLATFORM_FILE_WRITE_ATTRIBUTES)) {
if (!VerifyFileSystemPathForWrite(path, true /* create */, &origin_url,
- &type, &virtual_path, &file_system_file_util)) {
+ &type, &virtual_path, &file_util)) {
delete this;
return;
}
} else {
if (!VerifyFileSystemPathForRead(path, &origin_url, &type, &virtual_path,
- &file_system_file_util)) {
+ &file_util)) {
delete this;
return;
}
@@ -580,9 +561,8 @@ void FileSystemOperation::OpenFile(const GURL& path,
file_system_operation_context_.set_src_origin_url(origin_url);
file_system_operation_context_.set_src_type(type);
file_system_operation_context_.set_src_virtual_path(virtual_path);
- if (!file_system_operation_context_.src_file_system_file_util())
- file_system_operation_context_.set_src_file_system_file_util(
- file_system_file_util);
+ if (!file_system_operation_context_.src_file_util())
+ file_system_operation_context_.set_src_file_util(file_util);
file_flags_ = file_flags;
GetUsageAndQuotaThenCallback(origin_url, callback_factory_.NewCallback(
@@ -813,7 +793,7 @@ void FileSystemOperation::OnFileOpenedForWrite(
bool FileSystemOperation::VerifyFileSystemPathForRead(
const GURL& path, GURL* origin_url, FileSystemType* type,
- FilePath* virtual_path, FileSystemFileUtil** file_system_file_util) {
+ FilePath* virtual_path, FileUtil** file_util) {
// If we have no context, we just allow any operations, for testing.
// TODO(ericu): Revisit this hack for security.
@@ -832,7 +812,7 @@ bool FileSystemOperation::VerifyFileSystemPathForRead(
#endif
*type = file_system_operation_context_.src_type();
*origin_url = file_system_operation_context_.src_origin_url();
- *file_system_file_util = NULL;
+ *file_util = NULL;
return true;
}
@@ -847,10 +827,9 @@ bool FileSystemOperation::VerifyFileSystemPathForRead(
dispatcher_->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
return false;
}
- DCHECK(file_system_file_util);
- *file_system_file_util =
- file_system_context()->path_manager()->GetFileSystemFileUtil(*type);
- DCHECK(*file_system_file_util);
+ DCHECK(file_util);
+ *file_util = file_system_context()->path_manager()->GetFileUtil(*type);
+ DCHECK(*file_util);
// We notify this read access whether the read access succeeds or not.
// This must be ok since this is used to let the QM's eviction logic know
@@ -869,7 +848,7 @@ bool FileSystemOperation::VerifyFileSystemPathForRead(
bool FileSystemOperation::VerifyFileSystemPathForWrite(
const GURL& path, bool create, GURL* origin_url, FileSystemType* type,
- FilePath* virtual_path, FileSystemFileUtil** file_system_file_util) {
+ FilePath* virtual_path, FileUtil** file_util) {
// If we have no context, we just allow any operations, for testing.
// TODO(ericu): Revisit this hack for security.
@@ -888,7 +867,7 @@ bool FileSystemOperation::VerifyFileSystemPathForWrite(
#endif
*type = file_system_operation_context_.dest_type();
*origin_url = file_system_operation_context_.dest_origin_url();
- *file_system_file_util = NULL;
+ *file_util = NULL;
return true;
}
@@ -912,10 +891,9 @@ bool FileSystemOperation::VerifyFileSystemPathForWrite(
dispatcher_->DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
return false;
}
- DCHECK(file_system_file_util);
- *file_system_file_util =
- file_system_context()->path_manager()->GetFileSystemFileUtil(*type);
- DCHECK(*file_system_file_util);
+ DCHECK(file_util);
+ *file_util = file_system_context()->path_manager()->GetFileUtil(*type);
+ DCHECK(*file_util);
return true;
}
« webkit/fileapi/file_system_file_util_proxy.cc ('K') | « webkit/fileapi/file_system_operation.h ('k') | webkit/fileapi/file_system_operation_context.h » ('j') | webkit/fileapi/fileapi_file_util.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698