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

Unified Diff: webkit/fileapi/sandbox_mount_point_provider_unittest.cc

Issue 9016020: Cleanup FileSystemOperation for preparing for adding FSO-factory method (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: build fix 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
Index: webkit/fileapi/sandbox_mount_point_provider_unittest.cc
diff --git a/webkit/fileapi/sandbox_mount_point_provider_unittest.cc b/webkit/fileapi/sandbox_mount_point_provider_unittest.cc
index 4df10321c028cd4f1a166e658f40aec7860249e2..2f07dc562516cfc4ee4816bef5591e147eff9b13 100644
--- a/webkit/fileapi/sandbox_mount_point_provider_unittest.cc
+++ b/webkit/fileapi/sandbox_mount_point_provider_unittest.cc
@@ -157,13 +157,13 @@ class SandboxMountPointProviderMigrationTest : public testing::Test {
return sandbox_provider()->GetFileUtil();
}
- void OnGetRootPath(bool success, const FilePath& unused,
- const std::string& unused_also) {
- EXPECT_FALSE(success); // We told it not to create.
+ void OnValidate(base::PlatformFileError result) {
+ EXPECT_NE(base::PLATFORM_FILE_OK, result); // We told it not to create.
}
- FileSystemMountPointProvider::GetRootPathCallback GetRootPathCallback() {
- return base::Bind(&SandboxMountPointProviderMigrationTest::OnGetRootPath,
+ FileSystemMountPointProvider::ValidateFileSystemCallback
+ GetValidateCallback() {
+ return base::Bind(&SandboxMountPointProviderMigrationTest::OnValidate,
weak_factory_.GetWeakPtr());
}
@@ -283,12 +283,12 @@ class SandboxMountPointProviderMigrationTest : public testing::Test {
// migration if one is needed.
switch (method) {
case 0:
- sandbox_provider()->ValidateFileSystemRootAndGetURL(
- origin_url, type, create, GetRootPathCallback());
+ sandbox_provider()->ValidateFileSystemRoot(
+ origin_url, type, create, GetValidateCallback());
MessageLoop::current()->RunAllPending();
break;
case 1:
- sandbox_provider()->ValidateFileSystemRootAndGetPathOnFileThread(
+ sandbox_provider()->GetFileSystemRootPathOnFileThread(
origin_url, type, FilePath(), create);
break;
case 2:

Powered by Google App Engine
This is Rietveld 408576698