2008-05-07 Stephanie Lewis <slewis@apple.com>
authorslewis@apple.com <slewis@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 May 2008 01:05:19 +0000 (01:05 +0000)
committerslewis@apple.com <slewis@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 May 2008 01:05:19 +0000 (01:05 +0000)
commitbb297d30c69e4a58ef123e08d0bfe76289444837
tree1a846ca7c7154405b8a1b8694c56ed92b67240c9
parent02523b687252bcb29d4dd5be382b5d4b85c3aa71
2008-05-07  Stephanie Lewis <slewis@apple.com>

        Reviewed by Maciej.

        remove non-functioning code.  If onunload should be fired from these nodes then
        a WindowEventListener needs to be set.  However, Firefox and IE also do not
        fire onunload events for these cases.

        Test: fast/events/onunload-not-on-body.html

        * html/HTMLFrameElementBase.cpp:
        (WebCore::HTMLFrameElementBase::parseMappedAttribute):
        * html/HTMLObjectElement.cpp
        (WebCore::HTMLObjectElement::parseMappedAttribute):

        Test onunload attr on a frame does not call onunload, but that an onunload attr on the body of
        a frame does.  Also test that onunloads on object elements are not called.

        * fast/events/onunload-not-on-body-expected.txt: Added.
        * fast/events/onunload-not-on-body.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@32967 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/fast/events/onunload-not-on-body-expected.txt [new file with mode: 0644]
LayoutTests/fast/events/onunload-not-on-body.html [new file with mode: 0644]
WebCore/ChangeLog
WebCore/html/HTMLFrameElementBase.cpp
WebCore/html/HTMLObjectElement.cpp