Changeset 202455 in webkit

Timestamp:

Jun 24, 2016 3:25:24 PM (8 years ago)

Author:

jer.noble@apple.com

Message:

Vimeo.com videos do not get playback controls
​https://bugs.webkit.org/show_bug.cgi?id=159082
<rdar://problem/26586630>

Reviewed by Eric Carlson.

When the HasAudioOrVideo media state changes, let the video controls recalculate
by calling videoControlsManagerDidChange().

• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::isPlayingMediaDidChange):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/WebPageProxy.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2016-06-24  Jer Noble  <jer.noble@apple.com>
+
+        Vimeo.com videos do not get playback controls
+        https://bugs.webkit.org/show_bug.cgi?id=159082
+        <rdar://problem/26586630>
+
+        Reviewed by Eric Carlson.
+
+        When the HasAudioOrVideo media state changes, let the video controls recalculate
+        by calling videoControlsManagerDidChange().
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::isPlayingMediaDidChange):
+
 2016-06-24  Beth Dakin  <bdakin@apple.com>
 

trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp

@@ -6150 +6150 @@
     if ((oldState & playingMediaMask) != (m_mediaState & playingMediaMask))
         m_uiClient->isPlayingAudioDidChange(*this);
+#if PLATFORM(MAC)
+    if ((oldState & MediaProducer::HasAudioOrVideo) != (m_mediaState & MediaProducer::HasAudioOrVideo))
+        videoControlsManagerDidChange();
+#endif
 }