Update Crashpad to 4a2043ea65e2641ef1a921801c0aaa15ada02fc7

third_party/crashpad/crashpad/util/misc/metrics.cc

Index: third_party/crashpad/crashpad/util/misc/metrics.cc
diff --git a/third_party/crashpad/crashpad/util/misc/metrics.cc b/third_party/crashpad/crashpad/util/misc/metrics.cc
index 48be7ecd7aed785a9822331b51c961ff19184443..e2bd9f0ba38bdc761739b02ee66f68a1c5fbf5cd 100644
--- a/third_party/crashpad/crashpad/util/misc/metrics.cc
+++ b/third_party/crashpad/crashpad/util/misc/metrics.cc
@@ -98,6 +98,14 @@ void Metrics::ExceptionEncountered() {
   ExceptionProcessing(ExceptionProcessingState::kStarted);
 }
 
+// static
+void Metrics::HandlerLifetimeMilestone(LifetimeMilestone milestone) {
+  UMA_HISTOGRAM_ENUMERATION("Crashpad.HandlerLifetimeMilestone",
+                            static_cast<int32_t>(milestone),
+                            static_cast<int32_t>(LifetimeMilestone::kMaxValue));

[Ilya Sherman, 2017/02/22 20:16:18]

Hmm, are the static_casts needed?  In theory we have some template magic that
tries to make them unnecessary... bummer if it isn't working.

[Mark Mentovai, 2017/02/22 20:27:47]

It's because we use enum class in Crashpad pretty regularly for type safety.

[Ilya Sherman, 2017/02/22 22:53:01]

Yes, I understand that.  The intent of the template magic was to make it
typesafe to use enum classes with histogram macros.  Oh well.

+}
+
+// static
 void Metrics::HandlerCrashed(uint32_t exception_code) {
   UMA_HISTOGRAM_SPARSE_SLOWLY(
       "Crashpad.HandlerCrash.ExceptionCode." METRICS_OS_NAME,