Refactor SetClientID such that metrics rather than crash backs up the client id in Google Update set

chrome/common/child_process_logging_win.cc

Index: chrome/common/child_process_logging_win.cc
diff --git a/chrome/common/child_process_logging_win.cc b/chrome/common/child_process_logging_win.cc
index 1befb33b23a2eb971ddecee0e1c277a40b51e471..e072223913febd3fbca045fff51b545b42ef93af 100644
--- a/chrome/common/child_process_logging_win.cc
+++ b/chrome/common/child_process_logging_win.cc
@@ -67,10 +67,11 @@ void Init() {
 
   // This would be handled by BreakpadClient::SetClientID(), but because of the
   // aforementioned issue, crash keys aren't ready yet at the time of Breakpad
-  // initialization.
+  // initialization, retrieve the client id backed up in Google Update settings
+  // instead.
   std::string client_id;
-  if (GoogleUpdateSettings::GetMetricsId(&client_id))
-    base::debug::SetCrashKeyValue(crash_keys::kClientID, client_id);
+  if (GoogleUpdateSettings::RetrieveMetricsID(&client_id))
+    crash_keys::SetClientIDFromGUID(client_id);
 }
 
 }  // namespace child_process_logging