Title: [126158] trunk/LayoutTests
- Revision
- 126158
- Author
- [email protected]
- Date
- 2012-08-21 08:15:32 -0700 (Tue, 21 Aug 2012)
Log Message
media/media-document-audio-repaint.html is racy
https://bugs.webkit.org/show_bug.cgi?id=94454
Reviewed by Eric Carlson.
* media/media-document-audio-repaint.html: Don't rely on the play
event. Trigger the display and seek upon receipt of the pause event.
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (126157 => 126158)
--- trunk/LayoutTests/ChangeLog 2012-08-21 15:12:43 UTC (rev 126157)
+++ trunk/LayoutTests/ChangeLog 2012-08-21 15:15:32 UTC (rev 126158)
@@ -1,3 +1,13 @@
+2012-08-20 Philippe Normand <[email protected]>
+
+ media/media-document-audio-repaint.html is racy
+ https://bugs.webkit.org/show_bug.cgi?id=94454
+
+ Reviewed by Eric Carlson.
+
+ * media/media-document-audio-repaint.html: Don't rely on the play
+ event. Trigger the display and seek upon receipt of the pause event.
+
2012-08-21 Keishi Hattori <[email protected]>
Fix range-hit-test-with-padding.html by removing unnecessary part.
Modified: trunk/LayoutTests/media/media-document-audio-repaint.html (126157 => 126158)
--- trunk/LayoutTests/media/media-document-audio-repaint.html 2012-08-21 15:12:43 UTC (rev 126157)
+++ trunk/LayoutTests/media/media-document-audio-repaint.html 2012-08-21 15:15:32 UTC (rev 126158)
@@ -16,21 +16,13 @@
testRunner.notifyDone();
}
- function onPlay()
- {
- var videoElement = document.getElementById("fr").contentDocument.querySelector("video");
- videoElement.pause();
- }
-
function frameLoaded()
{
var videoElement = document.getElementById("fr").contentDocument.querySelector("video");
- videoElement.addEventListener("play", onPlay, false);
videoElement.addEventListener("pause", onPause, false);
videoElement.addEventListener("seeked", onSeeked, false);
- // Make sure we're playing, even though the MediaDocument has AUTOPLAY set,
- // to avoid races between the frame's loaded event and the video's play event.
- videoElement.play();
+ // The MediaDocument has AUTOPLAY set, make sure we're paused.
+ videoElement.pause();
}
</script>
<p>
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
