Bug 7079 – various event cleanup, including fixing of the load event for iframes

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Maciej Stachowiak]

I made a few improvements to event dispatch:

        - don't bother to nil-check the document, a node can never have a null document now
        - move temp event forgetting from dispatchEvent to dispatchGenericEvent
        - pass event down using RefPtr::release() to avoid ref thrashing
        - support default handlers even for non-bubbling events (only on
          target node) and skip calling default event handler explicitly in
          callers
        - dispatch a whole separate load event to a frame document's containing frame
        - don't let propagationStopped prevent this new event
        - remove bogus security check for iframe onload

[Maciej Stachowiak]

Created attachment 6264 [details]
the fix (including test case)

[Darin Adler]

Comment on attachment 6264 [details]
the fix (including test case)

Looks good. Gave Maciej some comments on IRC. r=me

[Darin Adler]

Had to roll it out -- I think Maciej had a fix last night but the commit failed.

[Adele Peterson]

Maciej checked in a fixed version of this.  Setting state to resolved/fixed.

[Lucas Forschler]

Mass moving XML DOM bugs to the "DOM" Component.