Removing fullscreen element in rAF() callback after requestFullscreen() can leave fullscreen in inconsistent state.
<rdar://problem/54164587>
Created attachment 377249 [details] Patch
Created attachment 377251 [details] Patch
Comment on attachment 377251 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=377251&action=review > Source/WebCore/dom/FullscreenManager.cpp:245 > + if (m_pendingFullscreenElement) Nit: this test is unnecessary > Source/WebCore/dom/FullscreenManager.cpp:271 > + if (m_pendingFullscreenElement) Ditto > LayoutTests/fullscreen/full-screen-request-removed-with-raf.html:9 > + if (document.webkitCurrentFullScreenElement && document.webkitCurrentFullScreenElement.ownerDocument) Shouldn’t this be “! document.webkitCurrentFullScreenElement.ownerDocument”?
Comment on attachment 377251 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=377251&action=review >> LayoutTests/fullscreen/full-screen-request-removed-with-raf.html:9 >> + if (document.webkitCurrentFullScreenElement && document.webkitCurrentFullScreenElement.ownerDocument) > > Shouldn’t this be “! document.webkitCurrentFullScreenElement.ownerDocument”? Yep.
Committed r249147: <https://trac.webkit.org/changeset/249147>