[webkit-reviews] review denied: [Bug 56049] Fix some potentially flaky tests : [Attachment 85225] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 9 14:56:59 PST 2011


Mihai Parparita <mihaip at chromium.org> has denied Tony Gentilcore
<tonyg at chromium.org>'s request for review:
Bug 56049: Fix some potentially flaky tests
https://bugs.webkit.org/show_bug.cgi?id=56049

Attachment 85225: Patch
https://bugs.webkit.org/attachment.cgi?id=85225&action=review

------- Additional Comments from Mihai Parparita <mihaip at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=85225&action=review

> LayoutTests/media/video-src-plus-source.html:8
>	   });

I think you need the final "true" argument to addEventListener so that this
handler is registered in the capture phase. Before you call findMediaElement(),
mediaElement is set to document, and the loadedmetadata event doesn't bubble.


More information about the webkit-reviews mailing list