Vimeo.com videos do not get playback controls
authorjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Jun 2016 22:25:24 +0000 (22:25 +0000)
committerjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Jun 2016 22:25:24 +0000 (22:25 +0000)
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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@202455 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/WebPageProxy.cpp

index 792d20a..b354907 100644 (file)
@@ -1,3 +1,17 @@
+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>
 
         Include enclosingListType in EditorState
index 526c716..e142969 100644 (file)
@@ -6149,6 +6149,10 @@ void WebPageProxy::isPlayingMediaDidChange(MediaProducer::MediaStateFlags state,
     playingMediaMask |= MediaProducer::HasActiveMediaCaptureDevice;
     if ((oldState & playingMediaMask) != (m_mediaState & playingMediaMask))
         m_uiClient->isPlayingAudioDidChange(*this);
+#if PLATFORM(MAC)
+    if ((oldState & MediaProducer::HasAudioOrVideo) != (m_mediaState & MediaProducer::HasAudioOrVideo))
+        videoControlsManagerDidChange();
+#endif
 }
 
 #if PLATFORM(MAC)