Title: [87703] trunk/Source/WebCore
Revision
87703
Author
[email protected]
Date
2011-05-30 17:05:02 -0700 (Mon, 30 May 2011)

Log Message

2011-05-30  Jer Noble  <[email protected]>

        Reviewed by Dan Bernstein.

        Play/Pause button in wrong state when entering full screen on Vimeo
        https://bugs.webkit.org/show_bug.cgi?id=61754

        Update the play/pause button inside reset().  This ensures the button is in
        the correct state when the controls are created after playback has already
        started.

        * html/shadow/MediaControlRootElement.cpp:
        (WebCore::MediaControlRootElement::reset):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (87702 => 87703)


--- trunk/Source/WebCore/ChangeLog	2011-05-30 23:49:51 UTC (rev 87702)
+++ trunk/Source/WebCore/ChangeLog	2011-05-31 00:05:02 UTC (rev 87703)
@@ -1,3 +1,17 @@
+2011-05-30  Jer Noble  <[email protected]>
+
+        Reviewed by Dan Bernstein.
+
+        Play/Pause button in wrong state when entering full screen on Vimeo
+        https://bugs.webkit.org/show_bug.cgi?id=61754
+
+        Update the play/pause button inside reset().  This ensures the button is in
+        the correct state when the controls are created after playback has already
+        started.
+
+        * html/shadow/MediaControlRootElement.cpp:
+        (WebCore::MediaControlRootElement::reset):
+
 2011-05-30  No'am Rosenthal  <[email protected]>
 
         Reviewed by Simon Hausmann.

Modified: trunk/Source/WebCore/html/shadow/MediaControlRootElement.cpp (87702 => 87703)


--- trunk/Source/WebCore/html/shadow/MediaControlRootElement.cpp	2011-05-30 23:49:51 UTC (rev 87702)
+++ trunk/Source/WebCore/html/shadow/MediaControlRootElement.cpp	2011-05-31 00:05:02 UTC (rev 87703)
@@ -310,6 +310,8 @@
             m_toggleClosedCaptionsButton->hide();
     }
 
+    m_playButton->updateDisplayType();
+
 #if ENABLE(FULLSCREEN_API)
     if (document()->webkitIsFullScreen() && document()->webkitCurrentFullScreenElement() == m_mediaElement) {
         if (m_mediaElement->movieLoadType() == MediaPlayer::LiveStream) {
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to