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

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

Issue 2740633002: [Autofill] Add upstreaming UKM (Closed)
Patch Set: Addressed comments + feature flag 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: components/autofill/core/browser/autofill_metrics.h
diff --git a/components/autofill/core/browser/autofill_metrics.h b/components/autofill/core/browser/autofill_metrics.h
index 4127e9cd09d853a6b87b5e3fe77c4a8fb65c2ca2..26f3a38fade3ca30e3558be86f7ce48ebf411512 100644
--- a/components/autofill/core/browser/autofill_metrics.h
+++ b/components/autofill/core/browser/autofill_metrics.h
@@ -17,7 +17,11 @@
namespace base {
class TimeDelta;
-}
+} // namespace base
+
+namespace ukm {
+class UkmService;
+} // namespace ukm
namespace autofill {
@@ -680,6 +684,13 @@ class AutofillMetrics {
// suggestion to show an explanation of the warning.
static void LogShowedHttpNotSecureExplanation();
+ // Logs the the |ukm| with the specified |url| and the specified |metrics|.
+ // Returns whether the ukm was sucessfully logged.
+ static bool LogUkm(ukm::UkmService* ukm_service,
+ const GURL& url,
+ const std::string& ukm,
+ const std::map<std::string, int>& metrics);
+
// Utility to autofill form events in the relevant histograms depending on
// the presence of server and/or local data.
class FormEventLogger {

Powered by Google App Engine
This is Rietveld 408576698