Let SyncBackupManager keep backup data in memory until shutdown. Only persist

sync/internal_api/sync_rollback_manager_unittest.cc

Index: sync/internal_api/sync_rollback_manager_unittest.cc
diff --git a/sync/internal_api/sync_rollback_manager_unittest.cc b/sync/internal_api/sync_rollback_manager_unittest.cc
index b2435525b3d88accb651c1d1b7bc54d2c4b25526..961a14c869f6b7d9e4f2d06381f60ffbda05253d 100644
--- a/sync/internal_api/sync_rollback_manager_unittest.cc
+++ b/sync/internal_api/sync_rollback_manager_unittest.cc
@@ -102,7 +102,8 @@ class SyncRollbackManagerTest : public testing::Test,
   }
 
   void InitManager(SyncManager* manager, ModelTypeSet types,
-                   TestChangeDelegate* delegate, StorageOption storage_option) {
+                   TestChangeDelegate* delegate,
+                   InternalComponentsFactory::StorageOption storage_option) {
     manager_ = manager;
     types_ = types;
 
@@ -119,7 +120,7 @@ class SyncRollbackManagerTest : public testing::Test,
     args.workers.push_back(worker_);
     args.change_delegate = delegate;
     args.internal_components_factory.reset(new TestInternalComponentsFactory(
-        InternalComponentsFactory::Switches(), storage_option));
+        InternalComponentsFactory::Switches(), storage_option, storage_option));
     manager->Init(&args);
     loop_.PostTask(FROM_HERE, run_loop.QuitClosure());
     run_loop.Run();
@@ -130,9 +131,9 @@ class SyncRollbackManagerTest : public testing::Test,
     SyncBackupManager backup_manager;
     TestChangeDelegate delegate;
     InitManager(&backup_manager, ModelTypeSet(type), &delegate,
-                STORAGE_ON_DISK);
+                InternalComponentsFactory::STORAGE_ON_DISK_DEFERRED);
     CreateEntry(backup_manager.GetUserShare(), type, client_tag);
-    backup_manager.ShutdownOnSyncThread(STOP_SYNC);
+    backup_manager.ShutdownOnSyncThread(SWITCH_MODE_SYNC);
   }
 
   // Verify entry with |client_tag| exists in sync directory.
@@ -183,7 +184,7 @@ TEST_F(SyncRollbackManagerTest, RollbackBasic) {
   TestChangeDelegate delegate;
   SyncRollbackManager rollback_manager;
   InitManager(&rollback_manager, ModelTypeSet(PREFERENCES), &delegate,
-              STORAGE_ON_DISK);
+              InternalComponentsFactory::STORAGE_ON_DISK);
 
   // Simulate a new entry added during type initialization.
   int64 new_pref_id =
@@ -207,7 +208,7 @@ TEST_F(SyncRollbackManagerTest, NoRollbackOfTypesNotBackedUp) {
   TestChangeDelegate delegate;
   SyncRollbackManager rollback_manager;
   InitManager(&rollback_manager, ModelTypeSet(PREFERENCES, APPS), &delegate,
-              STORAGE_ON_DISK);
+              InternalComponentsFactory::STORAGE_ON_DISK);
 
   // Simulate new entry added during type initialization.
   int64 new_pref_id =
@@ -235,7 +236,7 @@ TEST_F(SyncRollbackManagerTest, BackupDbNotChangedOnAbort) {
   scoped_ptr<SyncRollbackManager> rollback_manager(
       new SyncRollbackManager);
   InitManager(rollback_manager.get(), ModelTypeSet(PREFERENCES), &delegate,
-              STORAGE_ON_DISK);
+              InternalComponentsFactory::STORAGE_ON_DISK);
 
   // Simulate a new entry added during type initialization.
   CreateEntry(rollback_manager->GetUserShare(), PREFERENCES, "pref2");
@@ -246,7 +247,7 @@ TEST_F(SyncRollbackManagerTest, BackupDbNotChangedOnAbort) {
   // Verify new entry was not persisted.
   rollback_manager.reset(new SyncRollbackManager);
   InitManager(rollback_manager.get(), ModelTypeSet(PREFERENCES), &delegate,
-              STORAGE_ON_DISK);
+              InternalComponentsFactory::STORAGE_ON_DISK);
   EXPECT_FALSE(VerifyEntry(rollback_manager->GetUserShare(), PREFERENCES,
                            "pref2"));
 }
@@ -256,7 +257,7 @@ TEST_F(SyncRollbackManagerTest, OnInitializationFailure) {
   scoped_ptr<SyncRollbackManager> rollback_manager(
       new SyncRollbackManager);
   InitManager(rollback_manager.get(), ModelTypeSet(PREFERENCES), NULL,
-              STORAGE_ON_DISK);
+              InternalComponentsFactory::STORAGE_ON_DISK);
 }
 
 }  // anonymous namespace