2011-02-28 Tony Gentilcore <tonyg@chromium.org>
authortonyg@chromium.org <tonyg@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Mar 2011 01:31:35 +0000 (01:31 +0000)
committertonyg@chromium.org <tonyg@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Mar 2011 01:31:35 +0000 (01:31 +0000)
commit47c8e0f3ce7149199bb90138c1d20f5028219410
tree1f399e73321520105ae4cef20686a495590c7859
parent5a8c95903265f93ed86c1da32612efaf9d30fde4
2011-02-28  Tony Gentilcore  <tonyg@chromium.org>

        Reviewed by Darin Adler.

        Fix race condition in 4 media tests
        https://bugs.webkit.org/show_bug.cgi?id=55414

        The parser may yield and give events a chance to dispatch between any two tokens. So
        it is possible that the loadstart event fires immediately after the video tag is
        inserted into the document but before the subsequent script is parsed. This is very
        rare in the layout test environment since these small files are loaded from disk.

        This patch eliminates that race condition by allowing waitForEvent to register for
        events that bubble up to the document from any element. Then the registration can
        happen before before the element is inserted into the document.

        * media/video-source-media.html:
        * media/video-source-none-supported.html:
        * media/video-source-type-params.html:
        * media/video-source-type.html:
        * media/video-test.js:
        (waitForEvent):
        (waitForEventAndTest):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@79949 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/media/video-source-media.html
LayoutTests/media/video-source-none-supported.html
LayoutTests/media/video-source-type-params.html
LayoutTests/media/video-source-type.html
LayoutTests/media/video-test.js