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

Unified Diff: components/search_provider_logos/logo_tracker.cc

Issue 2886933003: Use stricter type checking in UMA_HISTOGRAM_ENUMERATION (Closed)
Patch Set: nocompile test Created 3 years, 7 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: components/search_provider_logos/logo_tracker.cc
diff --git a/components/search_provider_logos/logo_tracker.cc b/components/search_provider_logos/logo_tracker.cc
index a2fc7f9e9e4b9071fd6defae6e0a3e07c8b904a1..bf1c31255f0bedfc5c892ef5ee6f86419e8e0535 100644
--- a/components/search_provider_logos/logo_tracker.cc
+++ b/components/search_provider_logos/logo_tracker.cc
@@ -146,7 +146,8 @@ void LogoTracker::SetClockForTests(std::unique_ptr<base::Clock> clock) {
void LogoTracker::ReturnToIdle(int outcome) {
if (outcome != kDownloadOutcomeNotTracked) {
- UMA_HISTOGRAM_ENUMERATION("NewTabPage.LogoDownloadOutcome", outcome,
+ UMA_HISTOGRAM_ENUMERATION("NewTabPage.LogoDownloadOutcome",
+ static_cast<LogoDownloadOutcome>(outcome),
DOWNLOAD_OUTCOME_COUNT);
}
// Cancel the current asynchronous operation, if any.

Powered by Google App Engine
This is Rietveld 408576698