|
Add MinidumpCrashpadInfo::report_id.
Now that Chrome’s about:crashes displays the crash report UUID, I wanted
to add it to the minidump. In the future, we may be able to index these
on the server. This will also help identify dumps that correspond to the
same event once we’re equipped to convert between different formats.
Ideally, this new field is populated with the same UUID used locally in
the crash report database. To make this work,
CrashReportDatabase::NewReport must carry the UUID. This was actually
part of CrashReportDatabaseWin’s private extension to NewReport, so that
extension subclass can now be cleaned up.
TEST=crashpad_minidump_test MinidumpCrashpadInfoWriter.*,
crashpad_client_test CrashReportDatabaseTest.NewCrashReport
R=rsesek@chromium.org
Committed: https://chromium.googlesource.com/crashpad/crashpad/+/6bf80c3e48594d98f77a01bae8ec4bcc7bfc116d
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+110 lines, -25 lines) |
Patch |
|
M |
client/crash_report_database.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
client/crash_report_database_mac.mm
|
View
|
|
3 chunks |
+11 lines, -5 lines |
0 comments
|
Download
|
|
M |
client/crash_report_database_test.cc
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
client/crash_report_database_win.cc
|
View
|
|
4 chunks |
+10 lines, -18 lines |
0 comments
|
Download
|
|
M |
handler/mac/crash_report_exception_handler.cc
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
minidump/minidump_crashpad_info_writer.h
|
View
|
|
1 chunk |
+3 lines, -0 lines |
0 comments
|
Download
|
|
M |
minidump/minidump_crashpad_info_writer.cc
|
View
|
|
3 chunks |
+12 lines, -1 line |
0 comments
|
Download
|
|
M |
minidump/minidump_crashpad_info_writer_test.cc
|
View
|
|
5 chunks |
+14 lines, -1 line |
0 comments
|
Download
|
|
M |
minidump/minidump_extensions.h
|
View
|
|
2 chunks |
+12 lines, -0 lines |
0 comments
|
Download
|
|
M |
snapshot/mac/process_snapshot_mac.h
|
View
|
|
3 chunks |
+9 lines, -0 lines |
0 comments
|
Download
|
|
M |
snapshot/mac/process_snapshot_mac.cc
|
View
|
|
2 chunks |
+6 lines, -0 lines |
0 comments
|
Download
|
|
M |
snapshot/minidump/process_snapshot_minidump.h
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
snapshot/minidump/process_snapshot_minidump.cc
|
View
|
|
1 chunk |
+5 lines, -0 lines |
0 comments
|
Download
|
|
M |
snapshot/process_snapshot.h
|
View
|
|
1 chunk |
+11 lines, -0 lines |
0 comments
|
Download
|
|
M |
snapshot/test/test_process_snapshot.h
|
View
|
|
3 chunks |
+3 lines, -0 lines |
0 comments
|
Download
|
|
M |
snapshot/test/test_process_snapshot.cc
|
View
|
|
2 chunks |
+5 lines, -0 lines |
0 comments
|
Download
|
Total messages: 6 (1 generated)
|