Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1236)

Unified Diff: base/metrics/histogram_functions.h

Issue 2801613002: Allow UmaHistogramEnumeration to be used with enum classes. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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),
dcheng 2017/04/05 17:12:13 Nit: it might be nice to DCHECK that sample and ma
benwells 2017/04/06 07:30:01 Hmmm .... OK. Not the simplest nit I've seen, or m
+ static_cast<int>(max));
}
// For adding boolean sample to histogram.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698