[Webkit-unassigned] [Bug 16447] onload called too many times for <svg:image>

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=16447


rwlbuis at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #20363|0                           |1
        is obsolete|                            |
  Attachment #20490|                            |review?
               Flag|                            |




------- Comment #5 from rwlbuis at gmail.com  2008-04-12 06:41 PDT -------
Created an attachment (id=20490)
 --> (http://bugs.webkit.org/attachment.cgi?id=20490&action=view)
Better approach

This patch keeps the intend of the current code, delaying parent load
dispatching until the image is loaded for externalResourcesRequired=true. Also
it makes sure the event is dispatched to the parent node, the current code used
"this"! With the new checks there are no multiple load dispatches.


Comparing with other implementations is a bit hard; Opera does not seem to
support externalResourcesRequired, saying:

Failed attribute on image element: externalResourcesRequired="true".

Both and FireFox and Opera just seem to dispatch the load for the image right
after parsing the tag, but I don't think this is what the spec indicates.

The malloc avoiding code is just an idea I have, and could be dealt with
separately, though while messing with the load code I think it is convenient
to attack that problem too.
Cheers,

Rob.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.