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

Unified Diff: components/browser_watcher/stability_report.proto

Issue 2927653003: Stability instrumentation: add a Vectored Exception Handler (Closed)
Patch Set: merge Created 3 years, 6 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 b1ca52e785cb6e04ceabace3781f3a88d26ed11d..af0dae3c9503b7aee81b8b0f642faeded3ef09f6 100644
--- a/components/browser_watcher/stability_report.proto
+++ b/components/browser_watcher/stability_report.proto
@@ -142,8 +142,17 @@ message Activity {
map<string, TypedValue> user_data = 9;
}
-// The state of a thread.
+// Details about an exception.
// Next id: 5
+message Exception {
+ optional uint32 code = 1;
+ optional uint64 program_counter = 2;
+ optional uint64 exception_address = 3;
+ optional int64 time = 4;
+}
+
+// The state of a thread.
+// Next id: 6
message ThreadState {
// The name of the thread, up to a maxiumum length.
optional string thread_name = 1;
@@ -155,6 +164,10 @@ message ThreadState {
// stack and |activities| holds the base of the stack (up to a maximum size).
optional int32 activity_count = 3;
repeated Activity activities = 4;
+
+ // The last exception to be successfully captured. Note this exception may
+ // have been recovered from.
+ optional Exception exception = 5;
}
// The state of a process.
« no previous file with comments | « components/browser_watcher/stability_paths_unittest.cc ('k') | components/browser_watcher/stability_report_extractor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698