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

Unified Diff: components/browser_watcher/stability_report.proto

Issue 2691033002: Collect field trial information from the stability file (Closed)
Patch Set: Fix test post merge Created 3 years, 10 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: components/browser_watcher/stability_report.proto
diff --git a/components/browser_watcher/stability_report.proto b/components/browser_watcher/stability_report.proto
index 7f20f7d5841c9aba8050ca69bde7372ab51a42ee..d04d87138f9a35e6c89b8d3aa3c95b8f078a6823 100644
--- a/components/browser_watcher/stability_report.proto
+++ b/components/browser_watcher/stability_report.proto
@@ -150,10 +150,21 @@ message ProcessState {
// TODO(manzagop): add experiment state.
}
+// Description of a field trial or experiment that the user is currently
rkaplow 2017/02/16 21:27:28 can you add pointers to the uma proto which stores
manzagop (departed) 2017/02/17 22:20:05 Done.
+// enrolled in.
+// Next id: 3
+message FieldTrial {
+ // The name of the field trial, as a 32-bit identifier.
+ optional fixed32 name_id = 1;
+
+ // The user's group within the field trial, as a 32-bit identifier.
+ optional fixed32 group_id = 2;
+}
+
// A stability report contains information pertaining to the execution of a
// single logical instance of a "chrome browser". It is comprised of information
// about the system state and about the chrome browser's processes.
-// Next id: 5
+// Next id: 6
message StabilityReport {
optional SystemState system_state = 1;
// TODO(manzagop): revisit whether a single repeated field should contain all
@@ -166,4 +177,7 @@ message StabilityReport {
// relocate these to their process (and perhaps thread).
repeated string log_messages = 3;
map<string, TypedValue> global_data = 4;
+
+ // The field trials the user is currently enrolled in.
+ repeated FieldTrial field_trials = 5;
}

Powered by Google App Engine
This is Rietveld 408576698