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

Unified Diff: chrome/common/metrics/proto/profile_collection.proto

Issue 226273004: Add new SampledProfile protobuf definition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added perf_data.proto include; fixed protobuf field numbers Created 6 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: chrome/common/metrics/proto/profile_collection.proto
diff --git a/chrome/common/metrics/proto/profile_collection.proto b/chrome/common/metrics/proto/profile_collection.proto
new file mode 100644
index 0000000000000000000000000000000000000000..40366a40a36b793330da09f84ff245d819e4b9de
--- /dev/null
+++ b/chrome/common/metrics/proto/profile_collection.proto
@@ -0,0 +1,42 @@
+// Copyright (c) 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+syntax = "proto2";
+
+option optimize_for = LITE_RUNTIME;
+
+package metrics;
+
+import "perf_data.proto";
+
+// Protocol buffer for collected profiling data.
+// Contains the parameters and data from a single profile collection event.
+
+// Next tag: 6
+message ProfileCollection {
+ // Indicates the event that triggered this collection.
+ enum TriggerType {
+ UNKNOWN = 0;
+ INTERVAL = 1; // Collect at a random point within regular intervals.
tipp 2014/04/04 23:01:37 s/INTERVAL/PERIODIC/
Simon Que 2014/04/04 23:57:08 Done.
+ }
+ optional TriggerType trigger_type = 1;
+
+ // The type of profile data that was collected.
+ enum ProfileType {
+ PERF = 0; // Perf profile.
+ }
+ optional ProfileType profile_type = 2;
+
+ // The actual perf data that was collected, if this is a perf profile.
+ optional PerfDataProto perf_data = 3;
+
+ // Time after system boot when the collection took place.
+ optional int64 seconds_after_boot = 4;
tipp 2014/04/04 23:01:37 seconds are a long time. make all of these millis
Simon Que 2014/04/04 23:57:08 Done.
+
+ // Time after last login when the collection took place.
+ optional int64 seconds_after_login = 5;
+
+ // For interval-based collections, the length of each interval.
+ optional int64 interval_length_seconds = 6;
tipp 2014/04/04 23:01:37 s/length/period/
Simon Que 2014/04/04 23:57:08 Done.
+}

Powered by Google App Engine
This is Rietveld 408576698