Closed Bug 386799 Opened 15 years ago Closed 15 years ago

Crash [@ _moz_cairo_surface_get_type] with zero-width <canvas>

Categories

(Core :: Canvas: 2D, defect)

x86
macOS
defect
Not set
critical

Tracking

()

RESOLVED FIXED

People

(Reporter: jruderman, Assigned: vlad)

References

Details

(Keywords: assertion, crash, testcase)

Crash Data

Attachments

(2 files)

###!!! ASSERTION: gfxASurface::AddRef without mSurface: 'mSurface != nsnull', file /Users/jruderman/trunk/mozilla/gfx/thebes/src/gfxASurface.cpp, line 66

Thread 0 Crashed:
0   _moz_cairo_surface_get_type
1   gfxASurface::GetType() const
2   nsNativeThemeCocoa::DrawWidgetBackground
3   nsCSSRendering::PaintBackgroundWithSC
4   nsCSSRendering::PaintBackground
5   nsDisplayBackground::Paint
6   nsDisplayList::Paint
7   nsDisplayWrapList::Paint
8   nsDisplayClip::Paint
9   nsDisplayList::Paint
...
Attached file testcase
Might want to retest after Mats patch in bug 303901 lands.
Setting a 0,0 scale matrix is a no-no; this avoids that situation, and also bulletproofs the cocoa native theme drawing code somewhat.
Assignee: nobody → vladimir
Status: NEW → ASSIGNED
Attachment #271213 - Flags: superreview?(roc)
Attachment #271213 - Flags: review?(roc)
Comment on attachment 271213 [details] [diff] [review]
avoid setting bogus cairo matrix

Er, meant to ask stuart for review.. but either one, whoever gets to it first please r/sr :)
Attachment #271213 - Flags: review?(roc) → review?(pavlov)
Attachment #271213 - Flags: review?(pavlov) → review+
Attachment #271213 - Flags: superreview?(roc) → superreview+
r+/sr+ == checkin needed? ;)
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
Crashtest checked in.
Flags: in-testsuite? → in-testsuite+
Crash Signature: [@ _moz_cairo_surface_get_type]
You need to log in before you can comment on or make changes to this bug.