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

Unified Diff: base/metrics/statistics_recorder.h

Issue 1726873002: Report histogram creation results. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: added message for why not using macro Created 4 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: base/metrics/statistics_recorder.h
diff --git a/base/metrics/statistics_recorder.h b/base/metrics/statistics_recorder.h
index 36b2f308af30bf8bc52d23a7ddfd9af69d22ccc7..6eaf07959f127f9821d3f077360bf22db20bcec0 100644
--- a/base/metrics/statistics_recorder.h
+++ b/base/metrics/statistics_recorder.h
@@ -63,6 +63,8 @@ class BASE_EXPORT StatisticsRecorder {
const bool include_persistent_;
};
+ ~StatisticsRecorder();
+
// Initializes the StatisticsRecorder system. Safe to call multiple times.
static void Initialize();
@@ -130,6 +132,9 @@ class BASE_EXPORT StatisticsRecorder {
// histogram. This method is thread safe.
static OnSampleCallback FindCallback(const std::string& histogram_name);
+ // Returns the number of known histograms.
+ static size_t GetHistogramCount();
+
// Clears all of the known histograms and resets static variables to a
// state that allows a new initialization.
static void ResetForTesting();
@@ -162,9 +167,8 @@ class BASE_EXPORT StatisticsRecorder {
// The constructor just initializes static members. Usually client code should
// use Initialize to do this. But in test code, you can friend this class and
- // call destructor/constructor to get a clean StatisticsRecorder.
+ // call the constructor to get a clean StatisticsRecorder.
StatisticsRecorder();
- ~StatisticsRecorder();
static void Reset();
static void DumpHistogramsToVlog(void* instance);

Powered by Google App Engine
This is Rietveld 408576698