[sync] Make SetSetupInProgress kick sync engine when needed

chrome/browser/sync/profile_sync_service.cc

Index: chrome/browser/sync/profile_sync_service.cc
diff --git a/chrome/browser/sync/profile_sync_service.cc b/chrome/browser/sync/profile_sync_service.cc
index 4ddaf6dadf56327a41c5bfd2935a018733d8450b..60587a43382f12198d410b3d1fd113b1942e7018 100644
--- a/chrome/browser/sync/profile_sync_service.cc
+++ b/chrome/browser/sync/profile_sync_service.cc
@@ -1001,6 +1001,17 @@ bool ProfileSyncService::FirstSetupInProgress() const {
   return !HasSyncSetupCompleted() && setup_in_progress_;
 }
 
+void ProfileSyncService::SetSetupInProgress(bool setup_in_progress) {
+  setup_in_progress_ = setup_in_progress;
+  if (!setup_in_progress) {
+    if (sync_initialized()) {
+      SetSyncSetupCompleted();

[Andrew T Wilson (Slow), 2012/06/11 18:17:59]

I'm somewhat leery about calling SetSyncSetupCompleted() here - I wonder if
instead we should require the caller to call it, *before* calling
SetSetupInProgress(false).

Also, I think we should probably just call ReconfigureDatatypeManager() instead
of OnUserChoseDatatypes().

+      OnUserChoseDatatypes(sync_prefs_.HasKeepEverythingSynced(),
+                           GetPreferredDataTypes());
+    }
+  }
+}
+
 bool ProfileSyncService::sync_initialized() const {
   return backend_initialized_;
 }