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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: base/metrics/histogram_macros.h

Issue 2381233002: Metrics - Add histograms/README.md (Closed)
Patch Set: fix another typo 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/histograms/README.md » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/metrics/histogram_macros.h
diff --git a/base/metrics/histogram_macros.h b/base/metrics/histogram_macros.h
index 9cb21ec563a2ed54e45cdd885b4d2501a84ad985..657613250f8d4b7a921859c6d1447513038a482e 100644
--- a/base/metrics/histogram_macros.h
+++ b/base/metrics/histogram_macros.h
@@ -10,7 +10,13 @@
#include "base/metrics/histogram_macros_local.h"
#include "base/time/time.h"
+// Macros for efficient use of histograms.
+//
+// For best practices on deciding when to emit to a histogram and what form
+// the histogram should take, see tools/metrics/histograms/README.md
+//
// TODO(nikunjb): Move sparse macros to this file.
+//
// UMA_HISTOGRAM_SPARSE_SLOWLY is defined in sparse_histogram.h as it has
// different #include dependencies.
« no previous file with comments | « no previous file | tools/metrics/histograms/README.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698