Title: [184662] trunk/Source/WebCore
Revision
184662
Author
[email protected]
Date
2015-05-20 14:53:07 -0700 (Wed, 20 May 2015)

Log Message

[Mac] inform media session of all state changes
https://bugs.webkit.org/show_bug.cgi?id=145222

Reviewed by Dean Jackson.

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::~HTMLMediaElement): Call updateMediaState.
(WebCore::HTMLMediaElement::mediaPlayerEngineUpdated): Ditto.
(WebCore::HTMLMediaElement::mediaPlayerFirstVideoFrameAvailable): Ditto.
(WebCore::HTMLMediaElement::removeEventListener): Ditto.
(WebCore::HTMLMediaElement::enqueuePlaybackTargetAvailabilityChangedEvent): Ditto.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (184661 => 184662)


--- trunk/Source/WebCore/ChangeLog	2015-05-20 21:52:35 UTC (rev 184661)
+++ trunk/Source/WebCore/ChangeLog	2015-05-20 21:53:07 UTC (rev 184662)
@@ -1,3 +1,17 @@
+2015-05-20  Eric Carlson  <[email protected]>
+
+        [Mac] inform media session of all state changes
+        https://bugs.webkit.org/show_bug.cgi?id=145222
+
+        Reviewed by Dean Jackson.
+
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::~HTMLMediaElement): Call updateMediaState.
+        (WebCore::HTMLMediaElement::mediaPlayerEngineUpdated): Ditto.
+        (WebCore::HTMLMediaElement::mediaPlayerFirstVideoFrameAvailable): Ditto.
+        (WebCore::HTMLMediaElement::removeEventListener): Ditto.
+        (WebCore::HTMLMediaElement::enqueuePlaybackTargetAvailabilityChangedEvent): Ditto.
+
 2015-05-20  Daniel Bates  <[email protected]>
 
         AX: AutoFill button is not accessible with VoiceOver

Modified: trunk/Source/WebCore/html/HTMLMediaElement.cpp (184661 => 184662)


--- trunk/Source/WebCore/html/HTMLMediaElement.cpp	2015-05-20 21:52:35 UTC (rev 184661)
+++ trunk/Source/WebCore/html/HTMLMediaElement.cpp	2015-05-20 21:53:07 UTC (rev 184662)
@@ -412,6 +412,7 @@
     if (hasEventListeners(eventNames().webkitplaybacktargetavailabilitychangedEvent)) {
         m_hasPlaybackTargetAvailabilityListeners = false;
         m_mediaSession->setHasPlaybackTargetAvailabilityListeners(*this, false);
+        updateMediaState();
     }
 #endif
 
@@ -4350,6 +4351,9 @@
     m_player->setVideoFullscreenGravity(m_videoFullscreenGravity);
     m_player->setVideoFullscreenLayer(m_videoFullscreenLayer.get());
 #endif
+#if ENABLE(WIRELESS_PLAYBACK_TARGET)
+    updateMediaState();
+#endif
 }
 
 void HTMLMediaElement::mediaPlayerFirstVideoFrameAvailable(MediaPlayer*)
@@ -4953,6 +4957,7 @@
     if (didRemoveLastAvailabilityChangedListener) {
         m_hasPlaybackTargetAvailabilityListeners = false;
         m_mediaSession->setHasPlaybackTargetAvailabilityListeners(*this, false);
+        updateMediaState();
     }
 
     return true;
@@ -4965,6 +4970,7 @@
     RefPtr<Event> event = WebKitPlaybackTargetAvailabilityEvent::create(eventNames().webkitplaybacktargetavailabilitychangedEvent, hasTargets);
     event->setTarget(this);
     m_asyncEventQueue.enqueueEvent(event.release());
+    updateMediaState();
 }
 
 void HTMLMediaElement::setWirelessPlaybackTarget(Ref<MediaPlaybackTarget>&& device)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to