Closed
Bug 1431475
Opened 7 years ago
Closed 7 years ago
Make some printing assertions diagnostic ones, so we get crashes on Nightly.
Categories
(Core :: Printing: Output, enhancement)
Core
Printing: Output
Tracking
()
RESOLVED
FIXED
mozilla60
People
(Reporter: bobowen, Assigned: bobowen)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
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.
Assignee | ||
Comment 1•7 years ago
|
||
Attachment #8943712 -
Flags: review?(jwatt)
Updated•7 years ago
|
Attachment #8943712 -
Flags: review?(jwatt) → review+
Assignee | ||
Comment 2•7 years ago
|
||
Pushed by bobowencode@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/877e173a9104
Make assertions guarding opening and closing print recording file streams MOZ_DIAGNOSTIC_ASSERTs. r=jwatt
Comment 4•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•