Log metrics for the Location Settings Dialog prompt for geolocation.

base/metrics/histogram_functions.h

Index: base/metrics/histogram_functions.h
diff --git a/base/metrics/histogram_functions.h b/base/metrics/histogram_functions.h
index 58fc30173c6ea17480175440202c384b96661261..851171e4a5db3a6709e6261c3f1d5727d20ae4a1 100644
--- a/base/metrics/histogram_functions.h
+++ b/base/metrics/histogram_functions.h
@@ -43,7 +43,8 @@ template <typename T>
 void UmaHistogramEnumeration(const std::string& name, T sample, T max) {
   static_assert(std::is_enum<T>::value,
                 "Non enum passed to UmaHistogramEnumeration");
-  return UmaHistogramExactLinear(name, static_cast<int>(sample), max);
+  return UmaHistogramExactLinear(name, static_cast<int>(sample),
+                                 static_cast<int>(max));

[Ilya Sherman, 2017/04/04 16:37:55]

I'd prefer that you make this change in a separate CL, and check in with dcheng@
about his plans, as he's been making similar improvements to the
UMA_HISTOGRAM_ENUMERATION macro.  I'm a bit worried that adding static_casts
without any additional compile-time checks might result in some confusion and
potentially lead to bugs.  However, I really have no explanation for why the
sample was being typecast whereas the max wasn't...

 }
 
 // For adding boolean sample to histogram.