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

Unified Diff: webkit/fileapi/file_system_url_request_job_unittest.cc

Issue 9004019: Cleanup: Removing FileSystemPathManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 8 years, 12 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
« no previous file with comments | « webkit/fileapi/file_system_url_request_job.cc ('k') | webkit/fileapi/file_system_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/fileapi/file_system_url_request_job_unittest.cc
diff --git a/webkit/fileapi/file_system_url_request_job_unittest.cc b/webkit/fileapi/file_system_url_request_job_unittest.cc
index 10f4d0c7c0e2d1ed75851d3aad84144e4d4a3eca..ccc980aef78c410fa2ac9519db5f38ea14ade14c 100644
--- a/webkit/fileapi/file_system_url_request_job_unittest.cc
+++ b/webkit/fileapi/file_system_url_request_job_unittest.cc
@@ -37,7 +37,7 @@
#include "webkit/fileapi/file_system_context.h"
#include "webkit/fileapi/file_system_file_util.h"
#include "webkit/fileapi/file_system_operation_context.h"
-#include "webkit/fileapi/file_system_path_manager.h"
+#include "webkit/fileapi/mock_file_system_options.h"
#include "webkit/fileapi/sandbox_mount_point_provider.h"
#include "webkit/quota/mock_special_storage_policy.h"
@@ -83,12 +83,10 @@ class FileSystemURLRequestJobTest : public testing::Test {
base::MessageLoopProxy::current(),
base::MessageLoopProxy::current(),
special_storage_policy_, NULL,
- FilePath(), false /* is_incognito */, true /* allow_file_access */,
- new FileSystemPathManager(
- base::MessageLoopProxy::current(),
- temp_dir_.path(), NULL, false, false));
+ temp_dir_.path(),
+ CreateDisallowFileAccessOptions());
- file_system_context_->path_manager()->ValidateFileSystemRootAndGetURL(
+ file_system_context_->sandbox_provider()->ValidateFileSystemRootAndGetURL(
GURL("http://remote/"), kFileSystemTypeTemporary, true, // create
base::Bind(&FileSystemURLRequestJobTest::OnGetRootPath,
weak_factory_.GetWeakPtr()));
@@ -144,7 +142,7 @@ class FileSystemURLRequestJobTest : public testing::Test {
void CreateDirectory(const base::StringPiece& dir_name) {
FilePath path = FilePath().AppendASCII(dir_name);
- FileSystemFileUtil* file_util = file_system_context_->path_manager()->
+ FileSystemFileUtil* file_util = file_system_context_->
sandbox_provider()->GetFileUtil();
FileSystemOperationContext context(file_system_context_, file_util);
context.set_src_origin_url(GURL("http://remote"));
@@ -161,7 +159,7 @@ class FileSystemURLRequestJobTest : public testing::Test {
void WriteFile(const base::StringPiece& file_name,
const char* buf, int buf_size) {
FilePath path = FilePath().AppendASCII(file_name);
- FileSystemFileUtil* file_util = file_system_context_->path_manager()->
+ FileSystemFileUtil* file_util = file_system_context_->
sandbox_provider()->GetFileUtil();
FileSystemOperationContext context(file_system_context_, file_util);
context.set_src_origin_url(GURL("http://remote"));
« no previous file with comments | « webkit/fileapi/file_system_url_request_job.cc ('k') | webkit/fileapi/file_system_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698