PART2: Make SignedSettings use proper Value types instead of string all around the place.

chrome/browser/chromeos/login/wizard_controller.cc

 // Copyright (c) 2011 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 <sys/types.h>
 
 #include <string>
@@ skipping 373 matching lines @@
 }
 
 void WizardController::OnEulaAccepted() {
   MarkEulaAccepted();
   // TODO(pastarmovj): Make this code cache the value for the pref in a better
   // way until we can store it in the policy blob. See explanation below:
   // At this point we can not write this in the signed settings pref blob.
   // But we can at least create the consent file and Chrome would port that
   // if the device is owned by a local user. In case of enterprise enrolled
   // device the setting will be respected only until the policy is not set.
+  base::FundamentalValue value(usage_statistics_reporting_);
   SignedSettingsTempStorage::Store(
       kStatsReportingPref,
-      (usage_statistics_reporting_ ? "true" : "false"),
+      value,
       g_browser_process->local_state());
   bool enabled =
       OptionsUtil::ResolveMetricsReportingEnabled(usage_statistics_reporting_);
   // Make sure the local state cached value is updated too because the real
   // policy will only get written when the owner is created and the cache won't
   // be updated until the policy is reread.
   g_browser_process->local_state()->SetBoolean(kStatsReportingPref, enabled);
   if (enabled) {
 #if defined(USE_LINUX_BREAKPAD)
     // The crash reporter initialization needs IO to complete.
@@ skipping 301 matching lines @@
 
 bool WizardController::usage_statistics_reporting() const {
   return usage_statistics_reporting_;
 }
 
 void WizardController::SetZeroDelays() {
   kShowDelayMs = 0;
 }
 
 }  // namespace chromeos