126529 - ASSERTION: You can't draw an image with a 0 width or height!

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Image: Painting, defect, P3)

Description

[Akkana Peck]

I see this all the time when I browse with a debug build:

###!!! ASSERTION: You can't draw an image with a 0 width or height!: 'aSWidth >
0 && aDWidth > 0 && aSHeight > 0 && aDHeight > 0', file nsImageGTK.cpp, line 623
###!!! Break: at file nsImageGTK.cpp, line 623

One way to get it reproducibly: Load Tinderbox when something is flaming (not
sure if it works when everything is green).  Scroll down so that you don't see
the flames.  Hit reload (it reloads then scrolls down to where you were).  Now
use the mousewheel to scroll up again, and you get the assertion.

Can we turn this off if it's not important, or fix it if it is?  It happens a
lot in general browsing.

Comment 1

[Peter A Jonsson]

Attachment: Testcase that triggers the assert.

This is one of the things that trigger the assert. gray_bar.gif is a GIF89a,
1x22 pixels. Changing that picture to a bigger one makes the assert go away.
Removing or line breaking the code makes the assert go away.

Comment 2

[Peter A Jonsson]

Attachment: Stacktrace when the assert occurs

By calling TransformCoord() in nsRenderingContextImpl.cpp |sr.width| becomes 0.
This goes through to img->Draw() a few lines down. The source for |sr.width| to
become 0 is TransformCoord() in nsTransform2D.cpp. |aWidth| is 1 at entrance,
|m00| is 0.062500 and |ex| is 0. At line 577 |aWidth| gets set to 0.

Someone else should look at this.

Comment 3

[Stuart Parmenter]

Moving bugs to new Image: GFX component

Comment 4

[Vedran Miletic]

retargeting