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

chrome/browser/chromeos/login/wizard_controller.cc

Index: chrome/browser/chromeos/login/wizard_controller.cc
diff --git a/chrome/browser/chromeos/login/wizard_controller.cc b/chrome/browser/chromeos/login/wizard_controller.cc
index 09b4ed8e0ea4c95f77ea318584bff7ced7323d8b..49430286b027dce450d69d743a2298e08e942495 100644
--- a/chrome/browser/chromeos/login/wizard_controller.cc
+++ b/chrome/browser/chromeos/login/wizard_controller.cc
@@ -46,6 +46,7 @@
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
+#include "content/public/browser/notification_source.h"
 #include "content/public/browser/notification_types.h"
 #include "ui/base/accelerators/accelerator.h"
 #include "ui/base/l10n/l10n_util.h"
@@ -462,6 +463,12 @@ void WizardController::OnEulaAccepted() {
   MarkEulaAccepted();
   bool enabled =
       OptionsUtil::ResolveMetricsReportingEnabled(usage_statistics_reporting_);
+
+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_WIZARD_EULA_ACCEPTED,
+      content::NotificationSource(content::Source<WizardController>(this)),

[Nikita (slow), 2012/09/25 16:23:46]

As you're essentially using AllSources in ResourceRequestAllowedNotifier, do you
need to pass source here?

[SteveT, 2012/09/25 18:49:11]

This was suggested by Ilya. I believe we want to be as specific as we can, when
we can. It'll make things a step easier to update if we ever clean up this
dependency issue. It's also more precise.

+      content::NotificationService::NoDetails());
+
   CrosSettings::Get()->SetBoolean(kStatsReportingPref, enabled);
   if (enabled) {
 #if defined(USE_LINUX_BREAKPAD)