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 53a9b0007fac28bb82fc4568c226ff920017e844..64f68db582626dd8c8b5b680101de0866103fc6a 100644
--- a/chrome/browser/chromeos/login/wizard_controller.cc
+++ b/chrome/browser/chromeos/login/wizard_controller.cc
@@ -442,6 +442,12 @@ void WizardController::OnEulaAccepted() {
   MarkEulaAccepted();
   bool enabled =
       OptionsUtil::ResolveMetricsReportingEnabled(usage_statistics_reporting_);
+
+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_WIZARD_EULA_ACCEPTED,
+      content::NotificationService::AllSources(),

[Ilya Sherman, 2012/09/11 22:51:27]

nit: Please specify a source rather than using AllSources()

[SteveT, 2012/09/12 17:56:10]

I knew I would get in trouble for this :) - Unfortunately it is currently not
easy on the receiving end to specify the specific WizardController that will be
the Source of this Notification. The WizardController is created deep in the
bowels of BaseLoginDisplayHost, where it is not easy for our
ResourceRequestAllowedNotifier to access it.

All existing listeners of this notification also listen on AllSources.

What's the usual course of action for these situations? I can see if we can
plumb the WizardController around, but it doesn't look clean.

[Ilya Sherman, 2012/09/12 20:30:01]

It looks like this is a new notification that you're adding, so I'm not sure
what you mean about the existing listeners ;)
I think we should do the work to plumb this through, or else specify a different
source (user account/profile?) that maps one-to-one to the controller.  The
second option is probably cleaner, if there's a suitable alternate source.

The danger with using AllSources() here is that if ChromeOS ever supports
multiple users being simultaneously logged in (and each having their own EULA?),
this will start listening to notifications from *all* users, which seems wrong.

[SteveT, 2012/09/14 20:25:09]

Done.

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