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

Unified Diff: chrome/browser/autofill/autofill_metrics.cc

Issue 7747009: Add metrics to track Autofill "user happiness" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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: chrome/browser/autofill/autofill_metrics.cc
diff --git a/chrome/browser/autofill/autofill_metrics.cc b/chrome/browser/autofill/autofill_metrics.cc
index 3670f05ba214c4d870ffdcc79145f1357226a46e..4b1d846461a6f32347cc87557ca4bd570c14ce60 100644
--- a/chrome/browser/autofill/autofill_metrics.cc
+++ b/chrome/browser/autofill/autofill_metrics.cc
@@ -288,6 +288,50 @@ void AutofillMetrics::LogServerQueryMetric(ServerQueryMetric metric) const {
NUM_SERVER_QUERY_METRICS);
}
+void AutofillMetrics::LogUserHappinessMetric(UserHappinessMetric metric) const {
+ DCHECK(metric < NUM_USER_HAPPINESS_METRICS);
+
+ // TODO(isherman): Useful for manual testing, but should be removed prior to
+ // committing.
+ switch (metric) {
+ case FORMS_LOADED:
+ DLOG(WARNING) << "LogUserHappinessMetric(FORMS_LOADED)";
+ break;
+ case FORM_SUBMITTED_WITHOUT_AUTOFILL:
+ DLOG(WARNING) << "LogUserHappinessMetric(FORM_SUBMITTED_WITHOUT_AUTOFILL)";
+ break;
+ case FORM_SUBMITTED_AFTER_AUTOFILL:
+ DLOG(WARNING) << "LogUserHappinessMetric(FORM_SUBMITTED_AFTER_AUTOFILL)";
+ break;
+ case USER_DID_TYPE:
+ DLOG(WARNING) << "LogUserHappinessMetric(USER_DID_TYPE)";
+ break;
+ case SUBMITTED_FORM_COULD_HAVE_BEEN_AUTOFILLED:
+ DLOG(WARNING) << "LogUserHappinessMetric(SUBMITTED_FORM_COULD_HAVE_BEEN_AUTOFILLED)";
+ break;
+ case SUGGESTIONS_SHOWN:
+ DLOG(WARNING) << "LogUserHappinessMetric(SUGGESTIONS_SHOWN)";
+ break;
+ case SUGGESTIONS_SHOWN_FIRST_TIME:
+ DLOG(WARNING) << "LogUserHappinessMetric(SUGGESTIONS_SHOWN_FIRST_TIME)";
+ break;
+ case USER_DID_AUTOFILL:
+ DLOG(WARNING) << "LogUserHappinessMetric(USER_DID_AUTOFILL)";
+ break;
+ case USER_DID_AUTOFILL_FIRST_TIME:
+ DLOG(WARNING) << "LogUserHappinessMetric(USER_DID_AUTOFILL_FIRST_TIME)";
+ break;
+ case USER_DID_EDIT_AUTOFILLED_FIELD:
+ DLOG(WARNING) << "LogUserHappinessMetric(USER_DID_EDIT_AUTOFILLED_FIELD)";
+ break;
+ default:
+ DLOG(WARNING) << "I have no idea what you're talking about.";
+ }
+
+ UMA_HISTOGRAM_ENUMERATION("Autofill.UserHappiness", metric,
+ NUM_USER_HAPPINESS_METRICS);
+}
+
void AutofillMetrics::LogIsAutofillEnabledAtStartup(bool enabled) const {
UMA_HISTOGRAM_BOOLEAN("Autofill.IsEnabled.Startup", enabled);
}

Powered by Google App Engine
This is Rietveld 408576698