Bug 5965 – feImage and image have Image loading race condition

• Component: SVG
• Status: RESOLVED FIXED

[Eric Seidel (no email)]

feImage and image have Image loading race condition

This should probably be solved by using a refactored version of HTMLImageLoader from the khtml side.  
<feImage> has an aditional issue in that it has no way currently to notify it's parent <filter> element that 
it's changed (once the image load is done) so as to allow the filter to notify all clients of that resource to 
tell them to repaint themselves.

Right now, if the images happen to load quickly enough, things display propertly... otherwise they don't.

[Eric Seidel (no email)]

Bumping this priority, since now everyone will be running into these, now that svg tests are turned on by 
default.  I may have to disable svg tests by default until we fix this.

[Eric Seidel (no email)]

These tests have been disabled until this bug is fixed:

svg/W3C-SVG-1.1/filters-light-01-f.svg-disabled
svg/W3C-SVG-1.1/filters-specular-01-f.svg-disabled
svg/W3C-SVG-1.1/render-groups-03-t.svg-disabled

[Alexander Kellett]

Created attachment 5612 [details]
refactor <image> to use a RenderImage subclass for display and a HTMLImageLoader subclass for loading

[Eric Seidel (no email)]

Comment on attachment 5612 [details]
refactor <image> to use a RenderImage subclass for display and a HTMLImageLoader subclass for loading

I had to make a couple tweaks when landing (mostly removing uncessary header
includes, a slight build fix, and a trivial copy/paste change from RenderPath
to fix filter and mask support with image rendering.)

Looks great though.  r=me.