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

Unified Diff: webkit/fileapi/file_system_path_manager.cc

Issue 7174002: Change {Obfuscated|Local}FileSystemFileUtil non-Singleton to take an underlying *FileUtil. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Fixed memery leak at LocalFSFUTest, and rebased. Created 9 years, 6 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/file_system_path_manager.cc
diff --git a/webkit/fileapi/file_system_path_manager.cc b/webkit/fileapi/file_system_path_manager.cc
index 395f88021ad86caba11d6ace8c4dba08d39c5542..b84eebd8eb3140876243d18a5479fa6dc3625c03 100644
--- a/webkit/fileapi/file_system_path_manager.cc
+++ b/webkit/fileapi/file_system_path_manager.cc
@@ -42,6 +42,8 @@ FileSystemPathManager::FileSystemPathManager(
bool allow_file_access_from_files)
: is_incognito_(is_incognito),
allow_file_access_from_files_(allow_file_access_from_files),
+ local_file_util_(
+ new LocalFileSystemFileUtil(FileSystemFileUtil::GetInstance())),
sandbox_provider_(
new SandboxMountPointProvider(
ALLOW_THIS_IN_INITIALIZER_LIST(this),
@@ -168,7 +170,7 @@ FileSystemFileUtil* FileSystemPathManager::GetFileSystemFileUtil(
case kFileSystemTypePersistent:
return sandbox_provider_->GetFileSystemFileUtil();
case kFileSystemTypeExternal:
- return LocalFileSystemFileUtil::GetInstance();
+ return local_file_util_.get();
case kFileSystemTypeUnknown:
default:
NOTREACHED();

Powered by Google App Engine
This is Rietveld 408576698