Set a "metrics_client_id" crash key instead of "guid" on Mac OS X

components/crash/app/crashpad_mac.mm

 // Copyright 2015 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 "components/crash/app/crashpad_mac.h"
 
 #include <string.h>
 #include <unistd.h>
 
 #include <algorithm>
@@ skipping 140 matching lines @@
   // fallback. Forwarding is turned off for debug-mode builds even for the
   // browser process, because the system's crash reporter can take a very long
   // time to chew on symbols.
   if (!browser_process || is_debug_build) {
     crashpad_info->set_system_crash_reporter_forwarding(
         crashpad::TriState::kDisabled);
   }
 
   g_simple_string_dictionary = new crashpad::SimpleStringDictionary();
   crashpad_info->set_simple_annotations(g_simple_string_dictionary);
+
+  base::debug::SetCrashKeyReportingFunctions(SetCrashKeyValue, ClearCrashKey);
+  crash_reporter_client->RegisterCrashKeys();
+
   SetCrashKeyValue("ptype", browser_process ? base::StringPiece("browser")
                                             : base::StringPiece(process_type));
   SetCrashKeyValue("pid", base::StringPrintf("%d", getpid()));
 
   logging::SetLogMessageHandler(LogMessageHandler);
 
   // If clients called CRASHPAD_SIMULATE_CRASH() instead of
   // base::debug::DumpWithoutCrashing(), these dumps would appear as crashes in
   // the correct function, at the correct file and line. This would be
   // preferable to having all occurrences show up in DumpWithoutCrashing() at
   // the same file and line.
   base::debug::SetDumpWithoutCrashingFunction(DumpWithoutCrashing);
 
-  base::debug::SetCrashKeyReportingFunctions(SetCrashKeyValue, ClearCrashKey);
-  crash_reporter_client->RegisterCrashKeys();
-
   if (browser_process) {
     g_database =
         crashpad::CrashReportDatabase::Initialize(database_path).release();
 
     bool enable_uploads = false;
     if (!crash_reporter_client->ReportingIsEnforcedByPolicy(&enable_uploads)) {
       enable_uploads = crash_reporter_client->GetCollectStatsConsent() ||
                        crash_reporter_client->IsRunningUnattended();
       // Breakpad provided a --disable-breakpad switch here. Should this?
     }
@@ skipping 49 matching lines @@
 
   struct {
     bool operator()(const UploadedReport& a, const UploadedReport& b) {
       return a.creation_time >= b.creation_time;
     }
   } sort_by_time;
   std::sort(uploaded_reports->begin(), uploaded_reports->end(), sort_by_time);
 }
 
 }  // namespace crash_reporter