Modify test case to support streaming operation in sandbox file system.

content/browser/fileapi/file_system_operation_impl_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/browser/fileapi/file_system_operation_impl.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/logging.h"
@@ skipping 32 matching lines @@
 namespace {
 
 const int kFileOperationStatusNotSet = 1;
 
 void AssertFileErrorEq(const tracked_objects::Location& from_here,
                        base::File::Error expected,
                        base::File::Error actual) {
   ASSERT_EQ(expected, actual) << from_here.ToString();
 }
 
-void AssertFileErrorEqWithClosure(
-    const tracked_objects::Location& from_here,
-    base::File::Error expected,
-    base::Closure closure,
-    base::File::Error actual) {
+void AssertFileErrorEqWithClosure(const tracked_objects::Location& from_here,
+                                  base::File::Error expected,
+                                  base::Closure closure,
+                                  base::File::Error actual) {
   ASSERT_EQ(expected, actual) << from_here.ToString();
   closure.Run();
 }
 
 }  // namespace
 
 // Test class for FileSystemOperationImpl.
 class FileSystemOperationImplTest
     : public testing::Test {
  public:
@@ skipping 211 matching lines @@
 
   void AddQuota(int64 quota_delta) {
     int64 quota;
     GetUsageAndQuota(NULL, &quota);
     quota_manager()->SetQuota(sandbox_file_system_.origin(),
                               sandbox_file_system_.storage_type(),
                               quota + quota_delta);
   }
 
  private:
-  base::MessageLoop message_loop_;
+  base::MessageLoopForIO message_loop_;
   scoped_refptr<QuotaManager> quota_manager_;
   scoped_refptr<QuotaManagerProxy> quota_manager_proxy_;
 
   // Common temp base for nondestructive uses.
   base::ScopedTempDir base_;
 
   SandboxFileSystemTestHelper sandbox_file_system_;
 
   // For post-operation status.
   int status_;
@@ skipping 307 matching lines @@
   FileSystemURL dest_file(CreateFile("dest"));
 
   operation_runner()->Copy(src_file, dest_file,
                            FileSystemOperation::OPTION_NONE,
                            FileSystemOperationRunner::CopyProgressCallback(),
                            RecordStatusCallback());
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(base::File::FILE_OK, status());
   EXPECT_TRUE(FileExists("dest"));
   EXPECT_EQ(2, quota_manager_proxy()->notify_storage_accessed_count());
+  EXPECT_EQ(1, change_observer()->get_and_reset_modify_file_count());
 
-  EXPECT_EQ(1, change_observer()->get_and_reset_modify_file_count());
   EXPECT_TRUE(change_observer()->HasNoChange());
 }
 
 TEST_F(FileSystemOperationImplTest, TestCopySuccessSrcFileAndNew) {
   FileSystemURL src_file(CreateFile("src"));
 
   operation_runner()->Copy(src_file, URLForPath("new"),
                            FileSystemOperation::OPTION_NONE,
                            FileSystemOperationRunner::CopyProgressCallback(),
                            RecordStatusCallback());
@@ skipping 15 matching lines @@
                            FileSystemOperationRunner::CopyProgressCallback(),
                            RecordStatusCallback());
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(base::File::FILE_OK, status());
 
   // Make sure we've overwritten but not copied the source under the |dest_dir|.
   EXPECT_TRUE(DirectoryExists("dest"));
   EXPECT_FALSE(DirectoryExists("dest/src"));
   EXPECT_GE(quota_manager_proxy()->notify_storage_accessed_count(), 3);
 
+  EXPECT_EQ(1, change_observer()->get_and_reset_create_directory_count());
   EXPECT_EQ(1, change_observer()->get_and_reset_remove_directory_count());
-  EXPECT_EQ(1, change_observer()->get_and_reset_create_directory_count());
   EXPECT_TRUE(change_observer()->HasNoChange());
 }
 
 TEST_F(FileSystemOperationImplTest, TestCopySuccessSrcDirAndNew) {
   FileSystemURL src_dir(CreateDirectory("src"));
   FileSystemURL dest_dir_new(URLForPath("dest"));
 
   operation_runner()->Copy(src_dir, dest_dir_new,
                            FileSystemOperation::OPTION_NONE,
                            FileSystemOperationRunner::CopyProgressCallback(),
@@ skipping 593 matching lines @@
 
   const int64 child_file_size = 8000 + 700;
   const int64 grandchild_file_size = 60 + 5;
   const int64 all_file_size = child_file_size + grandchild_file_size;
   int64 expected_usage = all_file_size + total_path_cost;
 
   usage = GetUsage();
   EXPECT_EQ(all_file_size, GetDataSizeOnDisk());
   EXPECT_EQ(expected_usage, usage);
 
-  // Copy src to dest1.
-  operation_runner()->Copy(
-      src, dest1, FileSystemOperation::OPTION_NONE,
-      FileSystemOperationRunner::CopyProgressCallback(),
-      base::Bind(&AssertFileErrorEq, FROM_HERE, base::File::FILE_OK));
-  base::RunLoop().RunUntilIdle();
+  {
+    base::RunLoop run_loop;
+    // Copy src to dest1.
+    operation_runner()->Copy(src,
+                             dest1,
+                             FileSystemOperation::OPTION_NONE,
+                             FileSystemOperationRunner::CopyProgressCallback(),
+                             base::Bind(&AssertFileErrorEqWithClosure,
+                                        FROM_HERE,
+                                        base::File::FILE_OK,
+                                        run_loop.QuitClosure()));
+    run_loop.Run();
+  }
 
   expected_usage += all_file_size + child_path_cost + grandchild_path_cost;
   EXPECT_TRUE(DirectoryExists("src/dir"));
   EXPECT_TRUE(FileExists("src/dir/file2"));
   EXPECT_TRUE(DirectoryExists("dest1/dir"));
   EXPECT_TRUE(FileExists("dest1/dir/file2"));
 
   EXPECT_EQ(2 * all_file_size, GetDataSizeOnDisk());
   EXPECT_EQ(expected_usage, GetUsage());
 
-  // Copy src/dir to dest2.
-  operation_runner()->Copy(
-      child_dir, dest2, FileSystemOperation::OPTION_NONE,
-      FileSystemOperationRunner::CopyProgressCallback(),
-      base::Bind(&AssertFileErrorEq, FROM_HERE, base::File::FILE_OK));
-  base::RunLoop().RunUntilIdle();
+  {
+    base::RunLoop run_loop;
+    // Copy src/dir to dest2.
+    operation_runner()->Copy(child_dir,
+                             dest2,
+                             FileSystemOperation::OPTION_NONE,
+                             FileSystemOperationRunner::CopyProgressCallback(),
+                             base::Bind(&AssertFileErrorEqWithClosure,
+                                        FROM_HERE,
+                                        base::File::FILE_OK,
+                                        run_loop.QuitClosure()));
+    run_loop.Run();
+  }
 
   expected_usage += grandchild_file_size + grandchild_path_cost;
   usage = GetUsage();
   EXPECT_EQ(2 * child_file_size + 3 * grandchild_file_size,
             GetDataSizeOnDisk());
   EXPECT_EQ(expected_usage, usage);
 }
 
 TEST_F(FileSystemOperationImplTest,
        TestCopySuccessSrcFileWithDifferentFileSize) {
@@ skipping 19 matching lines @@
         base::Bind(&AssertFileErrorEqWithClosure,
                    FROM_HERE,
                    base::File::FILE_OK,
                    run_loop.QuitClosure()));
     run_loop.Run();
   }
   EXPECT_EQ(0, GetFileSize("dest"));
 }
 
 }  // namespace content