Move VariationsRegistrySyncerWin to ChromeVariationsServiceClient

chrome/browser/chrome_browser_main.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/chrome_browser_main.h"
 
 #include <set>
 #include <string>
 #include <vector>
 
@@ skipping 1636 matching lines @@
     base::TimeDelta delay = base::TimeTicks::Now() - browser_open_start;
     UMA_HISTOGRAM_LONG_TIMES_100("Startup.BrowserOpenTabs", delay);
 
     // If we're running tests (ui_task is non-null), then we don't want to
     // call RequestLanguageList or StartRepeatedVariationsSeedFetch or
     // RequestLanguageList
     if (parameters().ui_task == NULL) {
       // Request new variations seed information from server.
       chrome_variations::VariationsService* variations_service =
           browser_process_->variations_service();
       if (variations_service) {

[Alexei Svitkine (slow), 2015/08/24 16:36:31]

Nit: No {}s

[blundell, 2015/08/24 18:32:08]

Done.

-        variations_service->StartRepeatedVariationsSeedFetch();
-
-#if defined(OS_WIN)
-        variations_service->StartGoogleUpdateRegistrySync();
-#endif  // defined(OS_WIN)
+        variations_service->PerformInitialStartup();

[Alexei Svitkine (slow), 2015/08/24 16:36:31]

Hmm, this name isn't super accurate. In fact, this is the second stage of start
up - the initial start up is creating field trials which is done much earlier.

Maybe PerformPreMainMessageLoopStartup()?

[blundell, 2015/08/24 18:32:08]

Done.

       }
 
       translate::TranslateDownloadManager::RequestLanguageList(
           profile_->GetPrefs());
     }
 
     run_message_loop_ = true;
   } else {
     run_message_loop_ = false;
   }
@@ skipping 132 matching lines @@
   chromeos::CrosSettings::Shutdown();
 #endif  // defined(OS_CHROMEOS)
 #endif  // defined(OS_ANDROID)
 }
 
 // Public members:
 
 void ChromeBrowserMainParts::AddParts(ChromeBrowserMainExtraParts* parts) {
   chrome_extra_parts_.push_back(parts);
 }