Revert of Crashpad Windows: Use the Crashpad client instead of Breakpad on Windows

components/crash/core/common/crash_keys.cc

Index: components/crash/core/common/crash_keys.cc
diff --git a/components/crash/core/common/crash_keys.cc b/components/crash/core/common/crash_keys.cc
index 3c3f2fba4f9939844bad5ac9d6bce79aab96549f..c0afc6a14bde0ef66ddd94777434472fa6ebdd56 100644
--- a/components/crash/core/common/crash_keys.cc
+++ b/components/crash/core/common/crash_keys.cc
@@ -16,14 +16,6 @@
 // Crashpad owns the "guid" key. Chrome's metrics client ID is a separate ID
 // carried in a distinct "metrics_client_id" field.
 const char kMetricsClientId[] = "metrics_client_id";
-#elif defined(OS_WIN)
-// TODO(scottmg): While transitioning to Crashpad, there are some executables
-// that use Crashpad (which use kMetricsClientId), and some that use Breakpad
-// (kClientId), and they both use this file. For now we define both, but once
-// Breakpad is no longer used on Windows, we will no longer need kClientId, and
-// this can be combined with the OS_MACOSX block above.
-const char kMetricsClientId[] = "metrics_client_id";
-const char kClientId[] = "guid";
 #else
 const char kClientId[] = "guid";
 #endif
@@ -52,7 +44,7 @@
   if (stripped_guid.empty())
     return;
 
-#if defined(OS_MACOSX) || defined(OS_WIN)
+#if defined(OS_MACOSX)
   // The crash client ID is maintained by Crashpad and is distinct from the
   // metrics client ID, which is carried in its own key.
   base::debug::SetCrashKeyValue(kMetricsClientId, stripped_guid);
@@ -64,7 +56,7 @@
 }
 
 void ClearMetricsClientId() {
-#if defined(OS_MACOSX) || defined(OS_WIN)
+#if defined(OS_MACOSX)
   // Crashpad always monitors for crashes, but doesn't upload them when
   // crash reporting is disabled. The preference to upload crash reports is
   // linked to the preference for metrics reporting. When metrics reporting is