[Sync] Refactor sync configuration logic.

chrome/browser/sync/test_profile_sync_service.h

Index: chrome/browser/sync/test_profile_sync_service.h
diff --git a/chrome/browser/sync/test_profile_sync_service.h b/chrome/browser/sync/test_profile_sync_service.h
index 1227e5f8043a1e933048e6b890d0a7cc1acd8f5f..0703bbc13caece360851ac9df218cfdfca2a2266 100644
--- a/chrome/browser/sync/test_profile_sync_service.h
+++ b/chrome/browser/sync/test_profile_sync_service.h
@@ -45,11 +45,12 @@ class SyncBackendHostForProfileSyncTest : public SyncBackendHost {
 
   MOCK_METHOD1(RequestNudge, void(const tracked_objects::Location&));
 
-  // Called when a nudge comes in.
-  void SimulateSyncCycleCompletedInitialSyncEnded(
-      const tracked_objects::Location&);
-
-  virtual void StartConfiguration(const base::Closure& callback) OVERRIDE;
+  virtual void RequestConfigureSyncer(
+      sync_api::ConfigureReason reason,
+      syncable::ModelTypeSet types_to_config,
+      const browser_sync::ModelSafeRoutingInfo routing_info,

[rlarocque, 2012/06/09 01:44:35]

I think we usually pass around ModelTypeSet and ModelSafeRoutingInfo as const&.

[Nicolas Zea, 2012/06/11 23:05:20]

Done.

+      const base::Callback<void(syncable::ModelTypeSet)>& ready_task,
+      const base::Closure& retry_callback) OVERRIDE;
 
   static void SetHistoryServiceExpectations(ProfileMock* profile);