2010-11-23 Abhishek Arya <inferno@chromium.org>
authorinferno@chromium.org <inferno@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Nov 2010 21:34:29 +0000 (21:34 +0000)
committerinferno@chromium.org <inferno@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Nov 2010 21:34:29 +0000 (21:34 +0000)
commit4f638de4d4edd662700b64898809c77042bbfec2
tree8846030d8efaf6166dd48e0886250013614a8f9d
parent9363fb02cace9eae2e697b988cdca04a2c3761cb
2010-11-23  Abhishek Arya  <inferno@chromium.org>

        Reviewed by Adam Barth.

        dispatchEvent call in EventSource endRequest can lead to calling endRequest
        again which frees up the pending activity. Make sure we have request in flight
        to prevent that from happening.
        https://bugs.webkit.org/show_bug.cgi?id=49448

        Test: http/tests/eventsource/eventsource-status-error-iframe-crash.html

        * page/EventSource.cpp:
        (WebCore::EventSource::endRequest):
        (WebCore::EventSource::close):
2010-11-23  Abhishek Arya  <inferno@chromium.org>

        Reviewed by Adam Barth.

        Tests that we do not crash on EventSource status errors inside an iframe.
        https://bugs.webkit.org/show_bug.cgi?id=49448

        * http/tests/eventsource/eventsource-status-error-iframe-crash-expected.txt: Added.
        * http/tests/eventsource/eventsource-status-error-iframe-crash.html: Added.
        * http/tests/eventsource/resources/request-status-error.htm: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@72625 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/http/tests/eventsource/eventsource-status-error-iframe-crash-expected.txt [new file with mode: 0644]
LayoutTests/http/tests/eventsource/eventsource-status-error-iframe-crash.html [new file with mode: 0644]
LayoutTests/http/tests/eventsource/resources/request-status-error.htm [new file with mode: 0644]
WebCore/ChangeLog
WebCore/page/EventSource.cpp