Closed Bug 386799 Opened 17 years ago Closed 17 years ago

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

Categories

(Core :: Graphics: Canvas2D, 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 (deleted) —
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: 17 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.

Attachment

General

Creator:
Created:
Updated:
Size: