RESOLVED FIXED 221863
MobileSafari crashes at WebCore: WebCore::VideoFullscreenInterfaceAVKit::doEnterFullscreen 
https://bugs.webkit.org/show_bug.cgi?id=221863
Summary MobileSafari crashes at WebCore: WebCore::VideoFullscreenInterfaceAVKit::doEn...
Jean-Yves Avenard [:jya]
Reported 2021-02-12 17:09:36 PST
UI process crashes at WebCore::VideoFullscreenInterfaceAVKit::doEnterFullscreen due to infinite recursion leading to stack exhaustion.
Attachments
Patch (2.81 KB, patch)
2021-02-12 18:01 PST, Jean-Yves Avenard [:jya] 		no flags
DetailsFormatted DiffDiff
Patch (2.23 KB, patch)
2021-02-13 02:37 PST, Jean-Yves Avenard [:jya] 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Jean-Yves Avenard [:jya]
Comment 1 2021-02-12 17:12:49 PST
<rdar://problem/73861140>
Jean-Yves Avenard [:jya]
Comment 2 2021-02-12 18:01:43 PST
Created attachment 420203 [details] Patch
Jean-Yves Avenard [:jya]
Comment 3 2021-02-13 02:32:04 PST
Comment on attachment 420203 [details] Patch I could make things go awry with this patch. start playing a video (not YouTube), go to full screen. Now put the video in PiP and press the X button in the top right corner. The video would exit PiP mode and go back to non-full screen but the video has now disappeared. Tapping play can only allow you to hear the video, but you can't see anything
Jean-Yves Avenard [:jya]
Comment 4 2021-02-13 02:37:11 PST
Created attachment 420218 [details] Patch
Eric Carlson
Comment 5 2021-02-14 12:48:09 PST
Comment on attachment 420218 [details] Patch The Windows bot failure is unrelated, only iOS code is changed by this patch.
EWS
Comment 6 2021-02-14 12:51:05 PST
Committed r272841: <https://commits.webkit.org/r272841> All reviewed patches have been landed. Closing bug and clearing flags on attachment 420218 [details].
Note You need to log in before you can comment on or make changes to this bug.