Title: [150454] trunk/Source/WebCore
Revision
150454
Author
[email protected]
Date
2013-05-21 10:20:17 -0700 (Tue, 21 May 2013)

Log Message

REGRESSION(r101810): Media controls status text missing for live-stream videos.
https://bugs.webkit.org/show_bug.cgi?id=116547

Reviewed by Eric Carlson.

Un-reverse the logic in loadedMetadata() so that the status text isn't
hidden (when it should be shown) for live-stream videos.

* html/shadow/MediaControlsApple.cpp:
(WebCore::MediaControlsApple::loadedMetadata):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (150453 => 150454)


--- trunk/Source/WebCore/ChangeLog	2013-05-21 17:16:34 UTC (rev 150453)
+++ trunk/Source/WebCore/ChangeLog	2013-05-21 17:20:17 UTC (rev 150454)
@@ -1,5 +1,18 @@
 2013-05-21  Jer Noble  <[email protected]>
 
+        REGRESSION(r101810): Media controls status text missing for live-stream videos.
+        https://bugs.webkit.org/show_bug.cgi?id=116547
+
+        Reviewed by Eric Carlson.
+
+        Un-reverse the logic in loadedMetadata() so that the status text isn't
+        hidden (when it should be shown) for live-stream videos.
+
+        * html/shadow/MediaControlsApple.cpp:
+        (WebCore::MediaControlsApple::loadedMetadata):
+
+2013-05-21  Jer Noble  <[email protected]>
+
         Implement overlap-avoidance for in-band text track cues.
         https://bugs.webkit.org/show_bug.cgi?id=116540
 

Modified: trunk/Source/WebCore/html/shadow/MediaControlsApple.cpp (150453 => 150454)


--- trunk/Source/WebCore/html/shadow/MediaControlsApple.cpp	2013-05-21 17:16:34 UTC (rev 150453)
+++ trunk/Source/WebCore/html/shadow/MediaControlsApple.cpp	2013-05-21 17:20:17 UTC (rev 150454)
@@ -426,7 +426,7 @@
 
 void MediaControlsApple::loadedMetadata()
 {
-    if (m_statusDisplay && m_mediaController->isLiveStream())
+    if (m_statusDisplay && !m_mediaController->isLiveStream())
         m_statusDisplay->hide();
 
     MediaControls::loadedMetadata();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to