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

Unified Diff: base/metrics/user_metrics.h

Issue 2384323002: Metrics - Add actions/README.md (Closed)
Patch Set: add owners section 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
« no previous file with comments | « no previous file | tools/metrics/actions/README.md » ('j') | tools/metrics/actions/README.md » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/metrics/user_metrics.h
diff --git a/base/metrics/user_metrics.h b/base/metrics/user_metrics.h
index 93701e8fd209897a144af07dcab0f197cc8c6ff9..eb73964bc03c6eefcbc4d5a22b916bc78aa0d9ea 100644
--- a/base/metrics/user_metrics.h
+++ b/base/metrics/user_metrics.h
@@ -17,6 +17,9 @@ namespace base {
// This module provides some helper functions for logging actions tracked by
// the user metrics system.
+// For best practices on deciding when to emit a user action, see
+// tools/metrics/actions/README.md
rkaplow 2016/10/06 22:03:07 btw I think we should consider linking to the git
Mark P 2016/10/06 22:21:12 Done here and actions.xml. Also did for the histo
+
// Record that the user performed an action.
// This function must be called after the task runner has been set with
// SetRecordActionTaskRunner().
« no previous file with comments | « no previous file | tools/metrics/actions/README.md » ('j') | tools/metrics/actions/README.md » ('J')

Powered by Google App Engine
This is Rietveld 408576698