Title: [188196] trunk/Source/WebCore
Revision
188196
Author
[email protected]
Date
2015-08-09 13:39:56 -0700 (Sun, 09 Aug 2015)

Log Message

[Mac] Always require ExternalDeviceAutoPlayCandidate flag to AirPlay automatically
https://bugs.webkit.org/show_bug.cgi?id=147801

Reviewed by Dean Jackson.

Test: http/tests/media/video-media-document-disposition-download.html

* Modules/mediasession/WebMediaSessionManager.cpp:
(WebCore::WebMediaSessionManager::configurePlaybackTargetClients): Don't tell the last element
  to begin playing to the target unless the ExternalDeviceAutoPlayCandidate flag is set and
  it is not currently playing.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (188195 => 188196)


--- trunk/Source/WebCore/ChangeLog	2015-08-09 20:01:15 UTC (rev 188195)
+++ trunk/Source/WebCore/ChangeLog	2015-08-09 20:39:56 UTC (rev 188196)
@@ -1,3 +1,17 @@
+2015-08-09  Eric Carlson  <[email protected]>
+
+        [Mac] Always require ExternalDeviceAutoPlayCandidate flag to AirPlay automatically
+        https://bugs.webkit.org/show_bug.cgi?id=147801
+
+        Reviewed by Dean Jackson.
+
+        Test: http/tests/media/video-media-document-disposition-download.html
+
+        * Modules/mediasession/WebMediaSessionManager.cpp:
+        (WebCore::WebMediaSessionManager::configurePlaybackTargetClients): Don't tell the last element
+          to begin playing to the target unless the ExternalDeviceAutoPlayCandidate flag is set and 
+          it is not currently playing.
+
 2015-08-09  Myles C. Maxfield  <[email protected]>
 
         Crash in ComplexTextController when laying out obscure text

Modified: trunk/Source/WebCore/Modules/mediasession/WebMediaSessionManager.cpp (188195 => 188196)


--- trunk/Source/WebCore/Modules/mediasession/WebMediaSessionManager.cpp	2015-08-09 20:01:15 UTC (rev 188195)
+++ trunk/Source/WebCore/Modules/mediasession/WebMediaSessionManager.cpp	2015-08-09 20:39:56 UTC (rev 188196)
@@ -272,7 +272,7 @@
         indexOfClientWillPlayToTarget = indexOfClientThatRequestedPicker;
     if (indexOfClientWillPlayToTarget == notFound && indexOfLastClientToRequestPicker != notFound)
         indexOfClientWillPlayToTarget = indexOfLastClientToRequestPicker;
-    if (indexOfClientWillPlayToTarget == notFound && haveActiveRoute)
+    if (indexOfClientWillPlayToTarget == notFound && haveActiveRoute && flagsAreSet(m_clientState[0]->flags, MediaProducer::ExternalDeviceAutoPlayCandidate) && !flagsAreSet(m_clientState[0]->flags, MediaProducer::IsPlayingVideo))
         indexOfClientWillPlayToTarget = 0;
 
     LOG(Media, "WebMediaSessionManager::configurePlaybackTargetClients - indexOfClientWillPlayToTarget = %zu", indexOfClientWillPlayToTarget);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to