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

Unified Diff: media/blink/resource_multibuffer_data_provider.cc

Issue 2776853002: Make UMA_HISTOGRAM_ENUMERATION work with scoped enums. (Closed)
Patch Set: rebase Created 3 years, 9 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
Index: media/blink/resource_multibuffer_data_provider.cc
diff --git a/media/blink/resource_multibuffer_data_provider.cc b/media/blink/resource_multibuffer_data_provider.cc
index f1f80f07823f60f9f15875d87437c18f44b13d8c..a4a4ac84567cdf179fe3148d73b3959547b5a2e5 100644
--- a/media/blink/resource_multibuffer_data_provider.cc
+++ b/media/blink/resource_multibuffer_data_provider.cc
@@ -263,8 +263,10 @@ void ResourceMultiBufferDataProvider::didReceiveResponse(
while (reasons) {
DCHECK_LT(shift, max_enum); // Sanity check.
if (reasons & 0x1) {
- UMA_HISTOGRAM_ENUMERATION("Media.UncacheableReason", shift,
- max_enum); // PRESUBMIT_IGNORE_UMA_MAX
+ // Note: this uses an exact linear UMA to fake an enum UMA, as the actual
+ // enum is a bitmask.
+ UMA_HISTOGRAM_EXACT_LINEAR("Media.UncacheableReason", shift,
+ max_enum); // PRESUBMIT_IGNORE_UMA_MAX
}
reasons >>= 1;

Powered by Google App Engine
This is Rietveld 408576698