Title: [87035] branches/safari-534-branch
- Revision
- 87035
- Author
- lforsch...@apple.com
- Date
- 2011-05-22 10:26:11 -0700 (Sun, 22 May 2011)
Log Message
Merge r86904.
Modified Paths
Added Paths
Diff
Modified: branches/safari-534-branch/LayoutTests/ChangeLog (87034 => 87035)
--- branches/safari-534-branch/LayoutTests/ChangeLog 2011-05-22 17:23:19 UTC (rev 87034)
+++ branches/safari-534-branch/LayoutTests/ChangeLog 2011-05-22 17:26:11 UTC (rev 87035)
@@ -1,3 +1,17 @@
+2011-05-22 Lucas Forschler <lforsch...@apple.com>
+
+ Merge r86904.
+
+ 2011-05-18 Jeremy Noble <jer.no...@apple.com>
+
+ Reviewed by Darin Adler.
+
+ Poster is not shown in Safari for video element with no playable source elements.
+ https://bugs.webkit.org/show_bug.cgi?id=61109
+
+ * media/video-src-invalid-poster-expected.txt: Added.
+ * media/video-src-invalid-poster.html: Added.
+
2011-05-19 Lucas Forschler <lforsch...@apple.com
Merged r86852.
Copied: branches/safari-534-branch/LayoutTests/media/video-src-invalid-poster-expected.txt (from rev 86904, trunk/LayoutTests/media/video-src-invalid-poster-expected.txt) (0 => 87035)
--- branches/safari-534-branch/LayoutTests/media/video-src-invalid-poster-expected.txt (rev 0)
+++ branches/safari-534-branch/LayoutTests/media/video-src-invalid-poster-expected.txt 2011-05-22 17:26:11 UTC (rev 87035)
@@ -0,0 +1,4 @@
+EXPECTED (video.clientWidth == '76') OK
+EXPECTED (video.clientHeight == '103') OK
+END OF TEST
+
Copied: branches/safari-534-branch/LayoutTests/media/video-src-invalid-poster.html (from rev 86904, trunk/LayoutTests/media/video-src-invalid-poster.html) (0 => 87035)
--- branches/safari-534-branch/LayoutTests/media/video-src-invalid-poster.html (rev 0)
+++ branches/safari-534-branch/LayoutTests/media/video-src-invalid-poster.html 2011-05-22 17:26:11 UTC (rev 87035)
@@ -0,0 +1,26 @@
+<script src=""
+<video poster="content/abe.png">
+ <source src="" type="bogus">
+</video>
+<script>
+ findMediaElement();
+
+ function listenForWidthAndHeight(expectedWidth, expectedHeight, callback) {
+ if (video.clientWidth == expectedWidth && video.clientHeight == expectedHeight) {
+ callback();
+ } else {
+ // This uses a 20ms sleep loop to accomplish the wait, since the
+ // standard specifies no events that fire on poster load or error.
+ window.setTimeout(listenForWidthAndHeight, 20, expectedWidth, expectedHeight, callback);
+ }
+ }
+
+ function expected() {
+ testExpected("video.clientWidth", 76);
+ testExpected("video.clientHeight", 103);
+ endTest();
+ }
+
+ listenForWidthAndHeight(76, 103, expected);
+ failTestIn(1000);
+</script>
Modified: branches/safari-534-branch/Source/WebCore/ChangeLog (87034 => 87035)
--- branches/safari-534-branch/Source/WebCore/ChangeLog 2011-05-22 17:23:19 UTC (rev 87034)
+++ branches/safari-534-branch/Source/WebCore/ChangeLog 2011-05-22 17:26:11 UTC (rev 87035)
@@ -1,3 +1,22 @@
+2011-05-22 Lucas Forschler <lforsch...@apple.com>
+
+ Merge r86904.
+
+ 2011-05-18 Jeremy Noble <jer.no...@apple.com>
+
+ Reviewed by Darin Adler.
+
+ Poster is not shown in Safari for video element with no playable source elements.
+ https://bugs.webkit.org/show_bug.cgi?id=61109
+
+ Test: media/video-src-invalid-poster.html
+
+ In the case where no video sources are playable, update the display state and
+ renderer, allowing the poster image to display.
+
+ * html/HTMLMediaElement.cpp:
+ (WebCore::HTMLMediaElement::waitForSourceChange):
+
2011-05-19 Lucas Forschler <lforsch...@apple.com
Merged r86852.
Modified: branches/safari-534-branch/Source/WebCore/html/HTMLMediaElement.cpp (87034 => 87035)
--- branches/safari-534-branch/Source/WebCore/html/HTMLMediaElement.cpp 2011-05-22 17:23:19 UTC (rev 87034)
+++ branches/safari-534-branch/Source/WebCore/html/HTMLMediaElement.cpp 2011-05-22 17:26:11 UTC (rev 87035)
@@ -772,6 +772,11 @@
// 6.18 - Set the element's delaying-the-load-event flag to false. This stops delaying the load event.
setShouldDelayLoadEvent(false);
+
+ updateDisplayState();
+
+ if (renderer())
+ renderer()->updateFromElement();
}
void HTMLMediaElement::noneSupported()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes