StabilityReport proto changes for multi-process support

components/browser_watcher/stability_report.proto

Index: components/browser_watcher/stability_report.proto
diff --git a/components/browser_watcher/stability_report.proto b/components/browser_watcher/stability_report.proto
index af0dae3c9503b7aee81b8b0f642faeded3ef09f6..a56e582927f4764c264ce24cef85b123367d9f19 100644
--- a/components/browser_watcher/stability_report.proto
+++ b/components/browser_watcher/stability_report.proto
@@ -87,7 +87,7 @@ message TypedValue {
 }
 
 // An activity represents information about something of interest on a thread.
-// Next id: 14
+// Next id: 18
 message Activity {
   enum Type {
     UNKNOWN = 0;
@@ -136,7 +136,7 @@ message Activity {
   // An arbitrary value used for information purposes.
   optional int32 generic_data = 13;
 
-  // Tag id 10 is reserved for server side augmentation.
+  // Tag ids 10 and 14-17 are reserved for server side augmentation.
 
   // A key-value store.
   map<string, TypedValue> user_data = 9;
@@ -171,16 +171,24 @@ message ThreadState {
 }
 
 // The state of a process.
-// Next id: 4
+// Next id: 6
 message ProcessState {
+  enum Type {
+    UNKNOWN_PROCESS = 0;
+    BROWSER_PROCESS = 1;
+    WATCHER_PROCESS = 2;
+  }
+
   // The identifier of the process.
   optional int64 process_id = 3;
+  optional Type process_type = 4;
 
   // Note: likely only a subset of modules of interest (e.g. Chromium's own
   // modules).
   repeated CodeModule modules = 1;
   repeated ThreadState threads = 2;
-  // TODO(manzagop): add experiment state.
+
+  map<string, TypedValue> data = 5;

[bcwhite, 2017/06/30 16:40:54]

Can the name be more descriptive?  Or at least a comment?  Relationship to
deprecated "global_data"?

[manzagop (departed), 2017/06/30 20:22:47]

Revised comments, here and below.

 }
 
 // Description of a field trial or experiment that the user is currently
@@ -200,12 +208,15 @@ message FieldTrial {
 // 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: 7
+// Next id: 8
 message StabilityReport {
   // Whether the report is complete. Reports can be incomplete when the
   // recording size quota is hit.
   optional bool is_complete = 6;
 
+  // The process identifier of the crashed process.
+  optional int64 crashed_process_id = 7;
+
   optional SystemState system_state = 1;
   // TODO(manzagop): revisit whether a single repeated field should contain all
   //     processes, or whether it's preferable to have separate fields per type.
@@ -213,10 +224,13 @@ message StabilityReport {
   //     times (e.g. start time), hierarchical relationships (e.g. parent pid),
   //     command line, etc.
   repeated ProcessState process_states = 2;
+
   // TODO(manzagop): if/when reports contain multiple processes, attribute and
   //     relocate these to their process (and perhaps thread).
   repeated string log_messages = 3;
-  map<string, TypedValue> global_data = 4;
+
+  // A global key-value store.
+  map<string, TypedValue> global_data = 4 [deprecated = true];
 
   // The field trials the user is currently enrolled in.
   repeated FieldTrial field_trials = 5;