Fix some migration-related bugs

chrome/browser/sync/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/profile_sync_service.h"
 
 #include <cstddef>
 #include <map>
 #include <set>
 #include <utility>
@@ skipping 1439 matching lines @@
       // enabled, and yet we still think we require a passphrase for decryption.
       DCHECK(!(IsPassphraseRequiredForDecryption() &&
                !IsEncryptedDatatypeEnabled()));
 
       // This must be done before we start syncing with the server to avoid
       // sending unencrypted data up on a first time sync.
       if (encryption_pending_)
         backend_->EnableEncryptEverything();
       NotifyObservers();
 
-      // In the old world, this would be a no-op.  With new syncer thread,
-      // this is the point where it is safe to switch from config-mode to
-      // normal operation.
-      backend_->StartSyncingWithServer();
+      if (migrator_.get() &&
+          migrator_->state() != browser_sync::BackendMigrator::IDLE) {
+        // Migration in progress.  Let the migrator know we just finished
+        // configuring something.  It will be up to the migrator to call
+        // StartSyncingWithServer() if migration is now finished.
+        migrator_->OnConfigureDone(*result);
+      } else {
+        StartSyncingWithServer();
+      }
 
       break;
     }
     case chrome::NOTIFICATION_GOOGLE_SIGNIN_SUCCESSFUL: {
       const GoogleServiceSigninSuccessDetails* successful =
           content::Details<const GoogleServiceSigninSuccessDetails>(
               details).ptr();
       DCHECK(!successful->password.empty());
       if (!sync_prefs_.IsStartSuppressed()) {
         cached_passphrase_ = successful->password;
@@ skipping 62 matching lines @@
         OnUnrecoverableError(FROM_HERE, "Sync credentials lost.");
       }
       break;
     }
     default: {
       NOTREACHED();
     }
   }
 }
 
+void ProfileSyncService::StartSyncingWithServer() {
+  if (backend_.get())
+    backend_->StartSyncingWithServer();
+}
+
 void ProfileSyncService::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void ProfileSyncService::RemoveObserver(Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
 bool ProfileSyncService::HasObserver(Observer* observer) const {
   return observers_.HasObserver(observer);
@@ skipping 87 matching lines @@
   // See http://stackoverflow.com/questions/6224121/is-new-this-myclass-undefined-behaviour-after-directly-calling-the-destru.
   ProfileSyncService* old_this = this;
   this->~ProfileSyncService();
   new(old_this) ProfileSyncService(
       new ProfileSyncComponentsFactoryImpl(profile,
                                            CommandLine::ForCurrentProcess()),
       profile,
       signin,
       behavior);
 }