Attachment 360141 Details for Bug 193837 – Patch

[patch] Patch

bug-193837-20190125120203.patch (text/plain), 5.39 KB, created by Jer Noble on 2019-01-25 12:02:03 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Jer Noble

Created: 2019-01-25 12:02:03 PST

Size: 5.39 KB

patch

obsolete

>Subversion Revision: 240299
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 76676af91c59171c2afeee433a7e88564655180b..2612c721036054d6580d0ed410b232d8d786c5bd 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,24 @@
>+2019-01-25  Jer Noble  <jer.noble@apple.com>
>+
>+        <video> elements not in the DOM should be allowed to AirPlay
>+        https://bugs.webkit.org/show_bug.cgi?id=193837
>+        <rdar://42559491>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Test: media/airplay-allows-buffering.html
>+
>+        Some websites will switch between <video> elements backed by MSE to one backed by
>+        a media file in order to implement an AirPlay control. But when a <video> element is
>+        removed from the DOM and paused, further buffering is blocked. For some ports (namely
>+        Cocoa ones), this keeps AirPlay from engaging. Relax this buffering restriction for
>+        elements who have been asked to play wirelessly, but whose wireless playback has not
>+        started yet.
>+
>+        * html/MediaElementSession.cpp:
>+        (WebCore::MediaElementSession::dataBufferingPermitted const):
>+        (WebCore::MediaElementSession::setShouldPlayToPlaybackTarget):
>+
> 2019-01-24  Jer Noble  <jer.noble@apple.com>
> 
>         Fix leak of AVSampleBufferRenderSynchronizer boundaryObserver object.
>diff --git a/Source/WebCore/html/MediaElementSession.cpp b/Source/WebCore/html/MediaElementSession.cpp
>index be6203dd7dc0c5a642167598ab009557503f4b16..71ac9956883102752def4d877501226402645a6f 100644
>--- a/Source/WebCore/html/MediaElementSession.cpp
>+++ b/Source/WebCore/html/MediaElementSession.cpp
>@@ -385,6 +385,11 @@ bool MediaElementSession::dataBufferingPermitted() const
>     if (shouldOverrideBackgroundLoadingRestriction())
>         return true;
> 
>+#if ENABLE(WIRELESS_PLAYBACK_TARGET)
>+    if (m_shouldPlayToPlaybackTarget)
>+        return true;
>+#endif
>+
>     if (m_elementIsHiddenUntilVisibleInViewport || m_elementIsHiddenBecauseItWasRemovedFromDOM || m_element.elementIsHidden())
>         return false;
> 
>@@ -683,6 +688,7 @@ void MediaElementSession::setShouldPlayToPlaybackTarget(bool shouldPlay)
> {
>     INFO_LOG(LOGIDENTIFIER, shouldPlay);
>     m_shouldPlayToPlaybackTarget = shouldPlay;
>+    updateClientDataBuffering();
>     client().setShouldPlayToPlaybackTarget(shouldPlay);
> }
> 
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index c90ff2522694442acb58d914bdfd086cff8dc8ec..2e7b7a9704f87910c5086df0a772a33a6af9a251 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,13 @@
>+2019-01-25  Jer Noble  <jer.noble@apple.com>
>+
>+        <video> elements not in the DOM should be allowed to AirPlay
>+        https://bugs.webkit.org/show_bug.cgi?id=193837
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * media/airplay-allows-buffering-expected.txt: Added.
>+        * media/airplay-allows-buffering.html: Added.
>+
> 2019-01-22  Michael Catanzaro  <mcatanzaro@igalia.com>
> 
>         Unreviewed, skip all resource load statistics tests on GTK
>diff --git a/LayoutTests/media/airplay-allows-buffering-expected.txt b/LayoutTests/media/airplay-allows-buffering-expected.txt
>new file mode 100644
>index 0000000000000000000000000000000000000000..d6030e1c1d72d8f919b20a250aab24c5a0d0344d
>--- /dev/null
>+++ b/LayoutTests/media/airplay-allows-buffering-expected.txt
>@@ -0,0 +1,13 @@
>+EVENT(load)
>+RUN(internals.setMockMediaPlaybackTargetPickerEnabled(true))
>+RUN(video.src = findMediaFile('video', 'content/test'))
>+EVENT(loadedmetadata)
>+RUN(video.parentNode.removeChild(video))
>+EXPECTED (internals.elementShouldBufferData(video) == 'false') OK
>+RUN(video.webkitShowPlaybackTargetPicker())
>+RUN(internals.setMockMediaPlaybackTargetPickerState('Sleepy TV', 'DeviceAvailable'))
>+EVENT(webkitcurrentplaybacktargetiswirelesschanged)
>+EXPECTED (video.webkitCurrentPlaybackTargetIsWireless == 'true') OK
>+EXPECTED (internals.elementShouldBufferData(video) == 'true') OK
>+END OF TEST
>+
>diff --git a/LayoutTests/media/airplay-allows-buffering.html b/LayoutTests/media/airplay-allows-buffering.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..75f5803cc24200b497983b716aa5a1327d0d2c0f
>--- /dev/null
>+++ b/LayoutTests/media/airplay-allows-buffering.html
>@@ -0,0 +1,37 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+    <title>airplay-allows-buffering</title>
>+    <script src="media-file.js"></script>
>+    <script src="video-test.js"></script>
>+    <script>
>+
>+    waitFor(window, 'load').then(async event => { 
>+        findMediaElement();
>+
>+        run('internals.setMockMediaPlaybackTargetPickerEnabled(true)');
>+
>+        run(`video.src = findMediaFile('video', 'content/test')`);
>+        await waitFor(video, 'loadedmetadata');
>+
>+        run(`video.parentNode.removeChild(video)`);
>+        await testExpectedEventually('internals.elementShouldBufferData(video)', false);
>+
>+        run(`video.webkitShowPlaybackTargetPicker()`);
>+        await sleepFor(100);
>+
>+        run(`internals.setMockMediaPlaybackTargetPickerState('Sleepy TV', 'DeviceAvailable')`);
>+        await waitFor(video, 'webkitcurrentplaybacktargetiswirelesschanged');
>+        testExpected('video.webkitCurrentPlaybackTargetIsWireless', true);
>+
>+        await testExpectedEventually('internals.elementShouldBufferData(video)', true);
>+
>+        endTest();
>+    });
>+
>+    </script>
>+</head>
>+<body>
>+    <video controls preload='metadata'></video>
>+</body>
>+</html>

Actions: View | Formatted Diff | Diff

Attachments on bug 193837: 360140 | 360141 | 360162 | 360195