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

Unified Diff: components/autofill/core/browser/autofill_metrics.cc

Issue 2434543003: [Autofill] Separate Autofill.CreditCardInfoBar into .Local and .Server (Closed)
Patch Set: Add base=true value to histogram Created 4 years, 2 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/autofill/core/browser/autofill_metrics.cc
diff --git a/components/autofill/core/browser/autofill_metrics.cc b/components/autofill/core/browser/autofill_metrics.cc
index 54a927a430b658723733917ee7bf854919b024ae..b4ea42e27c484d53f169bf582018fc7a4d081e46 100644
--- a/components/autofill/core/browser/autofill_metrics.cc
+++ b/components/autofill/core/browser/autofill_metrics.cc
@@ -258,10 +258,16 @@ void AutofillMetrics::LogCardUploadDecisionMetric(
}
// static
-void AutofillMetrics::LogCreditCardInfoBarMetric(InfoBarMetric metric) {
+void AutofillMetrics::LogCreditCardInfoBarMetric(InfoBarMetric metric,
+ bool is_uploading) {
DCHECK_LT(metric, NUM_INFO_BAR_METRICS);
- UMA_HISTOGRAM_ENUMERATION("Autofill.CreditCardInfoBar", metric,
- NUM_INFO_BAR_METRICS);
+ if (is_uploading) {
+ UMA_HISTOGRAM_ENUMERATION("Autofill.CreditCardInfoBar.Server", metric,
+ NUM_INFO_BAR_METRICS);
+ } else {
+ UMA_HISTOGRAM_ENUMERATION("Autofill.CreditCardInfoBar.Local", metric,
+ NUM_INFO_BAR_METRICS);
+ }
}
// static

Powered by Google App Engine
This is Rietveld 408576698