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

Unified Diff: components/metrics/metrics_log.h

Issue 2657083003: Populate a basic SystemProfileProto in UKM. (Closed)
Patch Set: remove stale TODO Created 3 years, 11 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 | components/metrics/metrics_log.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/metrics/metrics_log.h
diff --git a/components/metrics/metrics_log.h b/components/metrics/metrics_log.h
index 2150ea4d111a2ee7113cf472547399a7b2b28740..fb4bef2b9d7dcafe61dce481adb60c6f58c1b9e5 100644
--- a/components/metrics/metrics_log.h
+++ b/components/metrics/metrics_log.h
@@ -74,6 +74,10 @@ class MetricsLog {
// always incrementing for use in measuring time durations.
static int64_t GetCurrentTime();
+ // Record core profile settings into the SystemProfileProto.
+ static void RecordCoreSystemProfile(MetricsServiceClient* client,
+ SystemProfileProto* system_profile);
+
// Records a user-initiated action.
void RecordUserAction(const std::string& key);
@@ -81,6 +85,9 @@ class MetricsLog {
void RecordHistogramDelta(const std::string& histogram_name,
const base::HistogramSamples& snapshot);
+
+ // TODO(rkaplow): I think this can be a little refactored as it currently
+ // records a pretty arbitrary set of things.
// Records the current operating environment, including metrics provided by
// the specified set of |metrics_providers|. Takes the list of synthetic
// trial IDs as a parameter. A synthetic trial is one that is set up
« no previous file with comments | « no previous file | components/metrics/metrics_log.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698