Closed Bug 539290 Opened 11 years ago Closed 11 years ago

Test harness should look for minidumps from child (plugin) processes


(Core :: IPC, defect)

Windows 7
Not set





(Reporter: benjamin, Assigned: benjamin)




(2 files)

The test harness should look for minidumps from child (plugin) processes even if the main process exits with a success code. In this case, it should not print TEST-UNEXPECTED-FAIL (because the crash may be expected from a test), but should simply print the minidump. The leak harness will catch the unexpected crashes with "failed to print leak summary".
I suggested that we make the parent process log something to the PID log so the harness would know to look for minidumps. Currently it only checks if the main process exits with a non-zero return code.
Unfortunately that would require some way to match up minidumps with PIDs. Going for the simplest solution, I'd make "application crashed (minidump found)" be "INFO" instead of "TEST-UNEXPECTED-FAIL"
This dumps the crash data for all crashes, whether or not the crash was "expected" or not.
Attachment #421312 - Flags: review?(ted.mielczarek)
Well, no, I just meant we could leave a marker in the PID log that some child process crashed, to know to look for minidumps. But I guess it doesn't matter. I'm not wild about making the "crashed (minidump found)" bit not TEST-UNEXPECTED-FAIL, because then it won't show up in the brief tinderbox log.
This patch has the alternate problem where crashes will show up in the brief log even if they are "expected"... it needs to be combined with the tinderbox change in bug 483061 to look for the PROCESS-CRASH string.
Attachment #421325 - Flags: review?(ted.mielczarek)
Blocks: 516759
Comment on attachment 421325 [details] [diff] [review]
Dump minidumps for all child crashes, rev. 2

This is fine for now, I just know (from experience) that people will see the crash in the brief log and blame the orange on that no matter what.
Attachment #421325 - Flags: review?(ted.mielczarek) → review+
Attachment #421312 - Flags: review?(ted.mielczarek) → review-
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.