Printing only has some basic tests at the moment, so we still see problems that we actually have assertions for in the code, but they're just never hit in debug tests. This bug is for specifically making some of the assertions in the opening and closing of the file for recording MOZ_DIAGNOSTIC_ASSERT so we get crashes on Nightly to help diagnose bug 1418238. We might want to extend this to other assertions within printing, to try and surface other currently hidden bugs.
Created attachment 8943712 [details] [diff] [review] Make assertions guarding opening and closing print recording file streams MOZ_DIAGNOSTIC_ASSERTs
Attachment #8943712 - Flags: review?(jwatt)
Attachment #8943712 - Flags: review?(jwatt) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/877e173a9104 Make assertions guarding opening and closing print recording file streams MOZ_DIAGNOSTIC_ASSERTs. r=jwatt
Status: ASSIGNED → RESOLVED
Last Resolved: a month ago
status-firefox60: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.