Title: [101675] trunk/Source/WebCore
Revision
101675
Author
eric.carl...@apple.com
Date
2011-12-01 08:59:32 -0800 (Thu, 01 Dec 2011)

Log Message

When playing audio in <video>, the poster is hidden on play
https://bugs.webkit.org/show_bug.cgi?id=73405

Reviewed by Darin Adler.

* html/HTMLVideoElement.cpp:
(WebCore::HTMLVideoElement::hasAvailableVideoFrame): Don't return true if the file
    doesn't have video.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (101674 => 101675)


--- trunk/Source/WebCore/ChangeLog	2011-12-01 16:53:14 UTC (rev 101674)
+++ trunk/Source/WebCore/ChangeLog	2011-12-01 16:59:32 UTC (rev 101675)
@@ -1,5 +1,16 @@
 2011-12-01  Eric Carlson  <eric.carl...@apple.com>
 
+        When playing audio in <video>, the poster is hidden on play
+        https://bugs.webkit.org/show_bug.cgi?id=73405
+
+        Reviewed by Darin Adler.
+
+        * html/HTMLVideoElement.cpp:
+        (WebCore::HTMLVideoElement::hasAvailableVideoFrame): Don't return true if the file
+            doesn't have video.
+
+2011-12-01  Eric Carlson  <eric.carl...@apple.com>
+
         Add Settings for text track types
         https://bugs.webkit.org/show_bug.cgi?id=73383
 

Modified: trunk/Source/WebCore/html/HTMLVideoElement.cpp (101674 => 101675)


--- trunk/Source/WebCore/html/HTMLVideoElement.cpp	2011-12-01 16:53:14 UTC (rev 101674)
+++ trunk/Source/WebCore/html/HTMLVideoElement.cpp	2011-12-01 16:59:32 UTC (rev 101675)
@@ -237,7 +237,7 @@
     if (!player())
         return false;
     
-    return player()->hasAvailableVideoFrame();
+    return player()->hasVideo() && player()->hasAvailableVideoFrame();
 }
 
 void HTMLVideoElement::webkitEnterFullscreen(ExceptionCode& ec)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to