Closed Bug 1598258 Opened 10 months ago Closed 10 months ago

Add the list of additional minidumps to the crashed plug-in events

Categories

(Toolkit :: Crash Reporting, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: gsvelto, Assigned: gsvelto)

References

Details

Attachments

(1 file)

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: nobody → gsvelto
Status: NEW → ASSIGNED

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.

Pushed by gsvelto@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/86dc639426da
Add a list of additional minidumps to the event delivered when a plug-in crashes r=froydnj
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.