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

Unified Diff: webkit/fileapi/sandbox_mount_point_provider.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: Fixed for ChromeOS again. Created 9 years, 4 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
Index: webkit/fileapi/sandbox_mount_point_provider.cc
diff --git a/webkit/fileapi/sandbox_mount_point_provider.cc b/webkit/fileapi/sandbox_mount_point_provider.cc
index 920a8209831775d62b745735eac15f0e39838f7f..c7b9a9fff15e773623f93e54456bb3fa8a1d400a 100644
--- a/webkit/fileapi/sandbox_mount_point_provider.cc
+++ b/webkit/fileapi/sandbox_mount_point_provider.cc
@@ -20,8 +20,8 @@
#include "webkit/fileapi/file_system_types.h"
#include "webkit/fileapi/file_system_usage_cache.h"
#include "webkit/fileapi/file_system_util.h"
-#include "webkit/fileapi/local_file_system_file_util.h"
-#include "webkit/fileapi/obfuscated_file_system_file_util.h"
+#include "webkit/fileapi/local_file_util.h"
ericu 2011/08/22 17:59:37 Is local_file_util.h needed?
Dai Mikurube (NOT FULLTIME) 2011/08/24 11:57:30 Done.
+#include "webkit/fileapi/obfuscated_file_util.h"
#include "webkit/fileapi/quota_file_util.h"
#include "webkit/fileapi/sandbox_mount_point_provider.h"
#include "webkit/glue/webkit_glue.h"
@@ -97,7 +97,7 @@ class ObfuscatedOriginEnumerator
: public fileapi::SandboxMountPointProvider::OriginEnumerator {
public:
explicit ObfuscatedOriginEnumerator(
- fileapi::ObfuscatedFileSystemFileUtil* file_util) {
+ fileapi::ObfuscatedFileUtil* file_util) {
enum_.reset(file_util->CreateOriginEnumerator());
}
virtual ~ObfuscatedOriginEnumerator() {}
@@ -111,8 +111,7 @@ class ObfuscatedOriginEnumerator
}
private:
- scoped_ptr<fileapi::ObfuscatedFileSystemFileUtil::AbstractOriginEnumerator>
- enum_;
+ scoped_ptr<fileapi::ObfuscatedFileUtil::AbstractOriginEnumerator> enum_;
};
class OldSandboxOriginEnumerator
@@ -173,7 +172,7 @@ FilePath OldGetBaseDirectoryForOriginAndType(
}
bool MigrateOneOldFileSystem(
- fileapi::ObfuscatedFileSystemFileUtil* file_util,
+ fileapi::ObfuscatedFileUtil* file_util,
const FilePath& old_base_path, const GURL& origin,
fileapi::FileSystemType type) {
FilePath base_path = OldGetBaseDirectoryForOriginAndType(
@@ -202,7 +201,7 @@ bool MigrateOneOldFileSystem(
}
void MigrateAllOldFileSystems(
- fileapi::ObfuscatedFileSystemFileUtil* file_util,
+ fileapi::ObfuscatedFileUtil* file_util,
const FilePath& old_base_path) {
scoped_ptr<OldSandboxOriginEnumerator> old_origins(
new OldSandboxOriginEnumerator(old_base_path));
@@ -251,7 +250,7 @@ void MigrateAllOldFileSystems(
// to look up the filesystem's root, so we can take care of most of them by
// putting a check there.
void MigrateIfNeeded(
- fileapi::ObfuscatedFileSystemFileUtil* file_util,
+ fileapi::ObfuscatedFileUtil* file_util,
const FilePath& old_base_path) {
if (file_util::DirectoryExists(old_base_path))
MigrateAllOldFileSystems(file_util, old_base_path);
@@ -281,7 +280,7 @@ SandboxMountPointProvider::SandboxMountPointProvider(
profile_path_(profile_path),
quota_file_util_(QuotaFileUtil::CreateDefault()),
sandbox_file_util_(
- new ObfuscatedFileSystemFileUtil(
+ new ObfuscatedFileUtil(
profile_path.Append(kNewFileSystemDirectory),
quota_file_util_)) {
}
@@ -308,7 +307,7 @@ class SandboxMountPointProvider::GetFileSystemRootPathTask
scoped_refptr<base::MessageLoopProxy> file_message_loop,
const GURL& origin_url,
FileSystemType type,
- ObfuscatedFileSystemFileUtil* file_util,
+ ObfuscatedFileUtil* file_util,
const FilePath& old_base_path,
FileSystemPathManager::GetRootPathCallback* callback)
: file_message_loop_(file_message_loop),
@@ -362,7 +361,7 @@ class SandboxMountPointProvider::GetFileSystemRootPathTask
scoped_refptr<base::MessageLoopProxy> origin_message_loop_proxy_;
GURL origin_url_;
FileSystemType type_;
- scoped_refptr<ObfuscatedFileSystemFileUtil> file_util_;
+ scoped_refptr<ObfuscatedFileUtil> file_util_;
FilePath old_base_path_;
scoped_ptr<FileSystemPathManager::GetRootPathCallback> callback_;
};
@@ -541,7 +540,7 @@ int64 SandboxMountPointProvider::GetOriginUsageOnFileThread(
FileSystemOperationContext context(NULL, sandbox_file_util_);
context.set_src_origin_url(origin_url);
context.set_src_type(type);
- scoped_ptr<FileSystemFileUtil::AbstractFileEnumerator> enumerator(
+ scoped_ptr<FileApiFileUtil::AbstractFileEnumerator> enumerator(
sandbox_file_util_->CreateFileEnumerator(&context, FilePath()));
FilePath file_path_each;
@@ -620,7 +619,7 @@ void SandboxMountPointProvider::EndUpdateOriginOnFileThread(
FileSystemUsageCache::DecrementDirty(usage_file_path);
}
-FileSystemFileUtil* SandboxMountPointProvider::GetFileSystemFileUtil() {
+FileApiFileUtil* SandboxMountPointProvider::GetFileUtil() {
return sandbox_file_util_.get();
}

Powered by Google App Engine
This is Rietveld 408576698