-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vimeo video goes black if you hit the in-website play button while the video is open in PiP #12520
Conversation
b193cf2
to
6d4718b
Compare
EWS run on current version of this PR (hash 6d4718b) |
EWS run on previous version of this PR (hash b193cf2) |
Thanks for reviewing, Simon! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me
Thanks for reviewing, Geoff :) |
β¦e video is open in PiP https://bugs.webkit.org/show_bug.cgi?id=255163 rdar://107592139 Reviewed by Simon Fraser. Vimeo enters fullscreen when starting playback from the inline play button while already in PIP. This behavior is revealing a bug in the fullscreen handling. This patch introduces a quirk to work around this issue. This patch also reverts <https://commits.webkit.org/262703@main> since it did not resolve the issue. * Source/WebCore/page/Quirks.cpp: (WebCore::Quirks::blocksEnteringStandardFullscreenFromPictureInPictureQuirk const): * Source/WebCore/page/Quirks.h: * Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm: (VideoFullscreenInterfaceAVKit::didStopPictureInPicture): (VideoFullscreenInterfaceAVKit::prepareForPictureInPictureStopWithCompletionHandler): * Source/WebKit/WebProcess/FullScreen/WebFullScreenManager.cpp: (WebKit::WebFullScreenManager::enterFullScreenForElement): * Source/WebKit/WebProcess/cocoa/VideoFullscreenManager.h: (WebKit::VideoFullscreenManager::videoElementInPictureInPicture const): Canonical link: https://commits.webkit.org/262723@main
6d4718b
to
12441d9
Compare
Committed 262723@main (12441d9): https://commits.webkit.org/262723@main Reviewed commits have been landed. Closing PR #12520 and removing active labels. |
12441d9
6d4718b
π macπ wpeπ wincairoπ§ͺ bindingsπ mac-AS-debugπ§ͺ wpe-wk2π§ͺ ios-wk2π§ͺ api-macπ gtkπ§ͺ ios-wk2-wptπ§ͺ mac-wk1π§ͺ gtk-wk2π§ͺ api-iosπ§ͺ api-gtkπ§ͺ mac-wk2π§ͺ mac-AS-debug-wk2π tv-simπ§ͺ mac-wk2-stressπ π§ͺ mergeπ watch-sim