FYI: Control Data + Per-Device Metadata

chrome/browser/sync/test_profile_sync_service.cc

 // Copyright (c) 2012 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 "chrome/browser/sync/test_profile_sync_service.h"
 
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/sync/abstract_profile_sync_service_test.h"
 #include "chrome/browser/sync/glue/data_type_controller.h"
 #include "chrome/browser/sync/glue/sync_backend_host.h"
 #include "chrome/browser/sync/profile_sync_components_factory.h"
 #include "chrome/browser/sync/test/test_http_bridge_factory.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "sync/internal_api/public/sessions/sync_session_snapshot.h"
 #include "sync/internal_api/public/user_share.h"
 #include "sync/js/js_reply_handler.h"
 #include "sync/protocol/encryption.pb.h"
 #include "sync/syncable/directory.h"
 
 using syncer::ModelSafeRoutingInfo;
 using syncer::TestInternalComponentsFactory;
 using syncer::sessions::ModelNeutralState;
 using syncer::sessions::SyncSessionSnapshot;
 using syncer::sessions::SyncSourceInfo;
 using syncer::UserShare;
 using syncer::syncable::Directory;
+using syncer::NIGORI;
+using syncer::PER_DEVICE_METADATA;
+using syncer::PER_USER_METADATA;
 
 namespace browser_sync {
 
 SyncBackendHostForProfileSyncTest::SyncBackendHostForProfileSyncTest(
     Profile* profile,
     const base::WeakPtr<SyncPrefs>& sync_prefs,
     const base::WeakPtr<InvalidatorStorage>& invalidator_storage,
     syncer::TestIdFactory& id_factory,
     base::Closure& callback,
     bool set_initial_sync_ended_on_init,
@@ skipping 69 matching lines @@
     if (!callback_.is_null()) {
       callback_.Run();
     }
 
     // Pretend we downloaded initial updates and set initial sync ended bits
     // if we were asked to.
     if (set_initial_sync_ended_on_init_) {
       UserShare* user_share = GetUserShare();
       Directory* directory = user_share->directory.get();
 
-      if (!directory->initial_sync_ended_for_type(syncer::NIGORI)) {
+      if (!directory->initial_sync_ended_for_type(NIGORI)) {
         ProfileSyncServiceTestHelper::CreateRoot(
-            syncer::NIGORI, user_share, &id_factory_);
+            NIGORI, user_share, &id_factory_);
 
         // A side effect of adding the NIGORI mode (normally done by the
         // syncer) is a decryption attempt, which will fail the first time.
       }
 
+      if (!directory->initial_sync_ended_for_type(PER_DEVICE_METADATA)) {

[rlarocque, 2012/08/11 01:31:52]

These if statements are probably not required...

+        ProfileSyncServiceTestHelper::CreateRoot(
+            PER_DEVICE_METADATA, user_share, &id_factory_);
+      }
+
+      if (!directory->initial_sync_ended_for_type(PER_USER_METADATA)) {
+        ProfileSyncServiceTestHelper::CreateRoot(
+            PER_USER_METADATA, user_share, &id_factory_);
+      }
+
       SetInitialSyncEndedForAllTypes();
       restored_types = syncer::ModelTypeSet::All();
     }
   }
 
   SyncBackendHost::HandleSyncManagerInitializationOnFrontendLoop(
       js_backend, success, restored_types);
 }
 
 void SyncBackendHostForProfileSyncTest::SetInitialSyncEndedForAllTypes() {
@@ skipping 103 matching lines @@
       profile(),
       sync_prefs_.AsWeakPtr(),
       invalidator_storage_.AsWeakPtr(),
       id_factory_,
       callback_,
       set_initial_sync_ended_on_init_,
       synchronous_backend_initialization_,
       fail_initial_download_,
       storage_option_));
 }