Activate the VariationsService for ChromeOS and ensure that it does not ping the server until the E…

chrome/browser/chrome_browser_main.cc

Index: chrome/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index 315f7a07030fe51ecf6090bf493edde9c717657f..97243e31b9bd8321f472011d3d48db4feef489c3 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -1426,23 +1426,21 @@ int ChromeBrowserMainParts::PreMainMessageLoopRunImpl() {
     RecordPreReadExperimentTime("Startup.BrowserOpenTabs",
                                 base::TimeTicks::Now() - browser_open_start);
 
-    // TODO(mad): Move this call in a proper place on CrOS.
-    // http://crosbug.com/17687
-#if !defined(OS_CHROMEOS)
     // If we're running tests (ui_task is non-null), then we don't want to
-    // call FetchLanguageListFromTranslateServer or
-    // StartRepeatedVariationsSeedFetch.
+    // call FetchLanguageListFromTranslateServer.

[Nikita (slow), 2012/09/10 14:48:55]

nit: Comment should be updated so that it doesn't mention only translate
service.

[SteveT, 2012/09/10 16:33:02]

Ah you're right. Should have left it the way it was before. Reverted.

     if (parameters().ui_task == NULL) {
-      // Request new variations seed information from server.
-      browser_process_->variations_service()->
-          StartRepeatedVariationsSeedFetch();
-
+#if !defined(OS_CHROMEOS)
+      // TODO(mad): Move this call in a proper place on CrOS.
+      // http://crosbug.com/17687
       if (translate_manager_ != NULL) {
         translate_manager_->FetchLanguageListFromTranslateServer(
             profile_->GetPrefs());
       }
-    }
 #endif
+      // Request new variations seed information from server.
+      browser_process_->variations_service()->
+          StartRepeatedVariationsSeedFetch();
+    }
 
     run_message_loop_ = true;
   } else {