Retrieve client_id from GoogleUpdateSettings when its missing from Local State.

chrome/installer/util/google_update_settings.h

 // 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.
 
 #ifndef CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_
 #define CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 68 matching lines @@
   // crash dumps to Google. This information is collected by the web server
   // used to download the chrome installer.
   static bool GetCollectStatsConsentAtLevel(bool system_install);
 
   // Sets the user consent to send UMA and crash dumps to Google. Returns
   // false if the setting could not be recorded.
   static bool SetCollectStatsConsentAtLevel(bool system_install,
                                             bool consented);
 #endif
 
-  // Returns the metrics client-id backed up in the registry. If none found,
-  // returns empty string.
-  static bool RetrieveMetricsID(std::string* metrics_id);
+  // Returns the metrics info backed up in the registry. Only non-null
+  // parameters will be retrieved. Returns false on retrieval failure for
+  // |metrics_id| (|installation_date| is not considered critical and is simply
+  // set to 0 on retrieval failure).
+  static bool RetrieveMetricsInfo(std::string* metrics_id,
+                                  int64* installation_date);
 
   // Saves a backup of the metrics client-id in the registry.
-  static bool SaveMetricsID(const std::string& metrics_id);
+  static bool SaveMetricsInfo(const std::string& metrics_id,
+                              const int64 installation_date);

[Ilya Sherman, 2014/07/08 01:10:22]

nit: No need for the const in front of an int parameter.

[gab, 2014/07/09 19:16:59]

Indeed, not sure how this ended up there, it's not in the impl at least :-)!

 
   // Sets the machine-wide EULA consented flag required on OEM installs.
   // Returns false if the setting could not be recorded.
   static bool SetEULAConsent(const installer::InstallationState& machine_state,
                              BrowserDistribution* dist,
                              bool consented);
 
   // Returns the last time chrome was run in days. It uses a recorded value
   // set by SetLastRunTime(). Returns -1 if the value was not found or if
   // the value is corrupted.
@@ skipping 201 matching lines @@
   // this will do nothing to |experiment_labels|. This will return true if the
   // label did not exist, or was successfully read.
   static bool ReadExperimentLabels(bool system_install,
                                    base::string16* experiment_labels);
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(GoogleUpdateSettings);
 };
 
 #endif  // CHROME_INSTALLER_UTIL_GOOGLE_UPDATE_SETTINGS_H_