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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Apr 12 12:48:34 PDT 2008


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


eric at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #20491|review?                     |review-
               Flag|                            |




------- Comment #8 from eric at webkit.org  2008-04-12 12:48 PDT -------
(From update of attachment 20491)
Two comments:

1.  I think SVG Load events might support capture, in which case you need to
dispatch the event anyway (unless you walk up the parent chain and check to see
if any parent has a capturing load event listener (which is what the comment is
about).

Also, 
hasListenerType(const AtomicString &eventType)
we put the & next to AtomicString, per our style guidelines.

Since you've pointed out yet another case that our testing missed, you might
want to write a test case to verify that we correctly support capturing load
event listeners (since this patch would have broken that).

r- for the broken event capture support.


-- 
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.



More information about the webkit-unassigned mailing list