commit 2b42433c269e3cc4ab4b99e916e430c1d35aeaee
author Justin Cohen <justincohen@google.com> Fri Jul 16 15:23:48 2021 -0400
committer Copybara-Service <copybara-worker@google.com> Wed Aug 11 08:54:44 2021 -0700
tree ac406df484c25400aeac29c1ef62ae7cb5da5e89
parent 3f773cea0331b82ccaa1cd3290961ec45b968f83

ios: Fix MissingIntermediateDumpKey and InvalidIntermediateDumpKeySize histogram.

The actual number of enums that will be recorded is approx ~75.

Bug: crashpad: 31
Change-Id: I8f7510869bbf23a7ed48d1164c58cc789df16ccc
Reviewed-on: https://chromium-review.googlesource.com/c/crashpad/crashpad/+/3035823
Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
GitOrigin-RevId: 7451893cd80b22170c23e172a9dd0e280ed15560

util/misc/metrics.cc

diff --git a/util/misc/metrics.cc b/util/misc/metrics.cc
index d45fad6..59a392c 100644
--- a/util/misc/metrics.cc
+++ b/util/misc/metrics.cc
@@ -114,17 +114,15 @@
 // static
 void Metrics::MissingIntermediateDumpKey(
     const internal::IntermediateDumpKey& key) {
-  UMA_HISTOGRAM_ENUMERATION("Crashpad.IntermediateDump.Reader.MissingKey",
-                            key,
-                            internal::IntermediateDumpKey::kMaxValue);
+  base::UmaHistogramSparse("Crashpad.IntermediateDump.Reader.MissingKey",
+                           static_cast<uint16_t>(key));
 }
 
 // static
 void Metrics::InvalidIntermediateDumpKeySize(
     const internal::IntermediateDumpKey& key) {
-  UMA_HISTOGRAM_ENUMERATION("Crashpad.IntermediateDump.Reader.InvalidKeySize",
-                            key,
-                            internal::IntermediateDumpKey::kMaxValue);
+  base::UmaHistogramSparse("Crashpad.IntermediateDump.Reader.InvalidKeySize",
+                           static_cast<uint16_t>(key));
 }
 #endif