[webkit-reviews] review granted: [Bug 220887] Twitter PiP video pauses when scrolling : [Attachment 418309] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jan 25 11:49:18 PST 2021
Eric Carlson <eric.carlson at apple.com> has granted Peng Liu
<peng.liu6 at apple.com>'s request for review:
Bug 220887: Twitter PiP video pauses when scrolling
https://bugs.webkit.org/show_bug.cgi?id=220887
Attachment 418309: Patch
https://bugs.webkit.org/attachment.cgi?id=418309&action=review
--- Comment #6 from Eric Carlson <eric.carlson at apple.com> ---
Comment on attachment 418309
--> https://bugs.webkit.org/attachment.cgi?id=418309
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=418309&action=review
> Source/WebCore/html/MediaElementSession.cpp:279
> +SuccessOr<MediaPlaybackDenialReason>
MediaElementSession::playbackPermitted(MediaPlaybackOperation operation) const
As we discussed, I'm not happy with this enum. Please add a FIXME here and note
the url of the bug you filed to fix this.
> Source/WebCore/page/Quirks.cpp:1269
> + // Facebook and twitter will naively pause a <video> element that has
scrolled out of the viewport,
s/twitter/Twitter/
> Source/WebCore/page/Quirks.cpp:1276
> auto domain = RegistrableDomain(m_document->topDocument().url());
auto domain = RegistrableDomain(m_document->topDocument().url()).string();
m_requiresUserGestureToPauseInPictureInPicture = domain == "facebook.com"_s ||
domain == "twitter.com"_s;
More information about the webkit-reviews
mailing list