Add the list of additional minidumps to the crashed plug-in events
Categories
(Toolkit :: Crash Reporting, enhancement)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox72 | --- | fixed |
People
(Reporter: gsvelto, Assigned: gsvelto)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
Tests that need to cleanup crashes are currently forced to parse the .extra file to find out if more than one minidump was included in a crash. This is annoying because they sometimes race with the minidump-analyzer and there's no simple way of preventing it. Also parsing the file requires blocking I/O which makes the tests longer.
This issue can easily be solved by adding a field with the list of additional minidumps to the plugin-crashed event. This would both fix the races and simplify the test infrastructure dealing with them.
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 1•5 years ago
|
||
This patch adds a new field to the plugin-crashed event that holds the list
of additional minidumps associated with a crash report. The test
infrastructure is modified to use it which also fixes a race when processing
the .extra file. The reftest machinery has also been modified to take the new
field into account.
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Description
•