Title: [187776] trunk/Source/WebCore
Revision
187776
Author
[email protected]
Date
2015-08-03 15:20:11 -0700 (Mon, 03 Aug 2015)

Log Message

[Mac] Always require user gesture to begin playing to AppleTV automatically
https://bugs.webkit.org/show_bug.cgi?id=147591

Reviewed by Jer Noble.

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::mediaState): Don't set the ExternalDeviceAutoPlayCandidate
  flag until the user has explicitly triggered playback.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (187775 => 187776)


--- trunk/Source/WebCore/ChangeLog	2015-08-03 22:05:38 UTC (rev 187775)
+++ trunk/Source/WebCore/ChangeLog	2015-08-03 22:20:11 UTC (rev 187776)
@@ -1,3 +1,14 @@
+2015-08-03  Eric Carlson  <[email protected]>
+
+        [Mac] Always require user gesture to begin playing to AppleTV automatically
+        https://bugs.webkit.org/show_bug.cgi?id=147591
+
+        Reviewed by Jer Noble.
+
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::mediaState): Don't set the ExternalDeviceAutoPlayCandidate 
+          flag until the user has explicitly triggered playback.
+
 2015-08-03  Dean Jackson  <[email protected]>
 
         Inline video controls overlap each other while playing video in slideover mode

Modified: trunk/Source/WebCore/html/HTMLMediaElement.cpp (187775 => 187776)


--- trunk/Source/WebCore/html/HTMLMediaElement.cpp	2015-08-03 22:05:38 UTC (rev 187775)
+++ trunk/Source/WebCore/html/HTMLMediaElement.cpp	2015-08-03 22:20:11 UTC (rev 187776)
@@ -6466,7 +6466,7 @@
         state |= RequiresPlaybackTargetMonitoring;
 
     bool requireUserGesture = m_mediaSession->hasBehaviorRestriction(MediaElementSession::RequireUserGestureToAutoplayToExternalDevice);
-    if (hasActiveVideo && (!requireUserGesture || (hasAudio && !m_initiallyMuted && !loop())) && !m_failedToPlayToWirelessTarget)
+    if (hasActiveVideo && !requireUserGesture && !m_failedToPlayToWirelessTarget)
         state |= ExternalDeviceAutoPlayCandidate;
 
     if (hasActiveVideo && endedPlayback())
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to