Use SNAPSHOT sync mode for LocalSync

chrome/browser/sync_file_system/local/local_file_sync_service_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 <vector>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/location.h"
@@ skipping 303 matching lines @@
   file_system2.TearDown();
 }
 
 TEST_F(LocalFileSyncServiceTest, ProcessLocalChange_CreateFile) {
   const FileSystemURL kFile(file_system_->URL("foo"));
   const char kTestFileData[] = "0123456789";
   const int kTestFileDataSize = static_cast<int>(arraysize(kTestFileData) - 1);
 
   base::RunLoop run_loop;
 
-  // We should get called OnSyncEnabled on kFile.
+  // We should get called OnSyncEnabled and OnWriteEnabled on kFile.
+  // (OnWriteEnabled is called because we release lock before returning
+  // from ApplyLocalChange)
   StrictMock<MockSyncStatusObserver> status_observer;
-  EXPECT_CALL(status_observer, OnSyncEnabled(kFile))
-      .Times(AtLeast(1));
+  EXPECT_CALL(status_observer, OnSyncEnabled(kFile)).Times(AtLeast(1));
+  EXPECT_CALL(status_observer, OnWriteEnabled(kFile)).Times(AtLeast(1));
   file_system_->AddSyncStatusObserver(&status_observer);
 
   // Creates and writes into a file.
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system_->CreateFile(kFile));
   EXPECT_EQ(kTestFileDataSize,
             file_system_->WriteString(kFile, std::string(kTestFileData)));
 
-  // Retrieve the expected platform_path.
+  // Retrieve the expected file info.
   base::PlatformFileInfo info;
   base::FilePath platform_path;
   EXPECT_EQ(base::PLATFORM_FILE_OK,
             file_system_->GetMetadataAndPlatformPath(
                 kFile, &info, &platform_path));
 
   ASSERT_FALSE(info.is_directory);
   ASSERT_EQ(kTestFileDataSize, info.size);
 
   SyncFileMetadata metadata;
   metadata.file_type = SYNC_FILE_TYPE_FILE;
   metadata.size = info.size;
   metadata.last_modified = info.last_modified;
 
   // The local_change_processor's ApplyLocalChange should be called once
   // with ADD_OR_UPDATE change for TYPE_FILE.
   StrictMock<MockLocalChangeProcessor> local_change_processor;
   const FileChange change(FileChange::FILE_CHANGE_ADD_OR_UPDATE,
                           SYNC_FILE_TYPE_FILE);
   EXPECT_CALL(local_change_processor,
-              ApplyLocalChange(change, platform_path, metadata, kFile, _))
+              ApplyLocalChange(change, _, metadata, kFile, _))
       .WillOnce(MockStatusCallback(SYNC_STATUS_OK));
 
   local_service_->SetLocalChangeProcessor(&local_change_processor);
   local_service_->ProcessLocalChange(
       base::Bind(&OnSyncCompleted, FROM_HERE, run_loop.QuitClosure(),
                  SYNC_STATUS_OK, kFile));
 
   run_loop.Run();
 
   file_system_->RemoveSyncStatusObserver(&status_observer);
 
   EXPECT_EQ(0, GetNumChangesInTracker());
 }
 
 TEST_F(LocalFileSyncServiceTest, ProcessLocalChange_CreateAndRemoveFile) {
   const FileSystemURL kFile(file_system_->URL("foo"));
 
   base::RunLoop run_loop;
 
   // We should get called OnSyncEnabled and OnWriteEnabled on kFile.
   StrictMock<MockSyncStatusObserver> status_observer;
-  EXPECT_CALL(status_observer, OnSyncEnabled(kFile))
-      .Times(AtLeast(1));
+  EXPECT_CALL(status_observer, OnSyncEnabled(kFile)).Times(AtLeast(1));
+  EXPECT_CALL(status_observer, OnWriteEnabled(kFile)).Times(AtLeast(1));
   file_system_->AddSyncStatusObserver(&status_observer);
 
   // Creates and then deletes a file.
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system_->CreateFile(kFile));
   EXPECT_EQ(base::PLATFORM_FILE_OK, file_system_->Remove(kFile, false));
 
   // The local_change_processor's ApplyLocalChange should be called once
   // with DELETE change for TYPE_FILE.
   // The file will NOT exist in the remote side and the processor might
   // return SYNC_FILE_ERROR_NOT_FOUND (as mocked).
@@ skipping 68 matching lines @@
 
   // The local_change_processor's ApplyLocalChange will be called
   // twice for FILE_TYPE and FILE_DIRECTORY.
   StrictMock<MockLocalChangeProcessor> local_change_processor;
   std::vector<FileChange> changes;
   EXPECT_CALL(local_change_processor, ApplyLocalChange(_, _, _, kPath, _))
       .Times(2)
       .WillOnce(MockStatusCallbackAndRecordChange(SYNC_STATUS_OK, &changes))
       .WillOnce(MockStatusCallbackAndRecordChange(SYNC_STATUS_OK, &changes));
   local_service_->SetLocalChangeProcessor(&local_change_processor);
+
+  // OnWriteEnabled will be notified on kPath.
+  EXPECT_CALL(status_observer, OnWriteEnabled(kPath)).Times(AtLeast(1));
+
   local_service_->ProcessLocalChange(
       base::Bind(&OnSyncCompleted, FROM_HERE, run_loop.QuitClosure(),
                  SYNC_STATUS_OK, kPath));
 
   run_loop.Run();
 
   EXPECT_EQ(2U, changes.size());
   EXPECT_EQ(FileChange(FileChange::FILE_CHANGE_DELETE, SYNC_FILE_TYPE_FILE),
             changes[0]);
   EXPECT_EQ(FileChange(FileChange::FILE_CHANGE_ADD_OR_UPDATE,
@@ skipping 225 matching lines @@
   all_origins.insert(kOrigin2);
   all_origins.insert(kOrigin3);
   while (!all_origins.empty()) {
     ASSERT_TRUE(NextOriginToProcess(&origin));
     ASSERT_TRUE(ContainsKey(all_origins, origin));
     all_origins.erase(origin);
   }
 }
 
 }  // namespace sync_file_system