Consolidates accessing and setting the UMA pref to be within metrics code

chrome/browser/chromeos/login/wizard_controller.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/chromeos/login/wizard_controller.h"
 
 #include <signal.h>
 #include <stdlib.h>
 #include <sys/types.h>
 
@@ skipping 626 matching lines @@
   } else if (ShouldShowHostPairingScreen()) {
     ShowHostPairingScreen();
   } else {
     ShowAutoEnrollmentCheckScreen();
   }
 }
 
 void WizardController::OnEulaAccepted() {
   time_eula_accepted_ = base::Time::Now();
   StartupUtils::MarkEulaAccepted();
-  bool uma_enabled =
-      ResolveMetricsReportingEnabled(usage_statistics_reporting_);
-
-  CrosSettings::Get()->SetBoolean(kStatsReportingPref, uma_enabled);
-  if (uma_enabled) {
-#if defined(GOOGLE_CHROME_BUILD)
-    // The crash reporter initialization needs IO to complete.
-    base::ThreadRestrictions::ScopedAllowIO allow_io;
-    breakpad::InitCrashReporter(std::string());
-#endif
-  }
+  base::Callback<void(bool)> callback_fn = base::Bind(
+    &WizardController::InitiateMetricsReportingChangeCallback,

[Alexei Svitkine (slow), 2014/08/28 14:46:48]

Nit: Indent 2 more.

[gayane -on leave until 09-2017, 2014/09/03 20:20:08]

Done.

+    base::Unretained(this));

[Alexei Svitkine (slow), 2014/08/28 14:46:48]

Pass a weak ptr via weak_ptr_factory_ instead of base::Unretained() here.

[gayane -on leave until 09-2017, 2014/09/03 20:20:08]

Done.

+  InitiateMetricsReportingChange(callback_fn, usage_statistics_reporting_);
 
   if (skip_update_enroll_after_eula_) {
     PerformPostEulaActions();
     ShowAutoEnrollmentCheckScreen();
   } else {
     InitiateOOBEUpdate();
   }
 }
 
+void WizardController::InitiateMetricsReportingChangeCallback(bool success) {
+  if (success) {
+    CrosSettings::Get()->SetBoolean(kStatsReportingPref,
+                                    usage_statistics_reporting_);
+    if (usage_statistics_reporting_)
+    {

[Alexei Svitkine (slow), 2014/08/28 14:46:48]

Nit: Should go on previous line.

[gayane -on leave until 09-2017, 2014/09/03 20:20:08]

Done.

+#if defined(GOOGLE_CHROME_BUILD)
+      // The crash reporter initialization needs IO to complete.
+      base::ThreadRestrictions::ScopedAllowIO allow_io;
+      breakpad::InitCrashReporter(std::string());
+#endif
+    }
+  }
+}
+
 void WizardController::OnUpdateErrorCheckingForUpdate() {
   // TODO(nkostylev): Update should be required during OOBE.
   // We do not want to block users from being able to proceed to the login
   // screen if there is any error checking for an update.
   // They could use "browse without sign-in" feature to set up the network to be
   // able to perform the update later.
   OnUpdateCompleted();
 }
 
 void WizardController::OnUpdateErrorUpdating() {
@@ skipping 599 matching lines @@
 bool WizardController::SetOnTimeZoneResolvedForTesting(
     const base::Closure& callback) {
   if (timezone_resolved_)
     return false;
 
   on_timezone_resolved_for_testing_ = callback;
   return true;
 }
 
 }  // namespace chromeos