[webkit-changes] [WebKit/WebKit] f82dc7: REGRESSION(265043 at main): Fullscreen mode only show...
Alan Baradlay
noreply at github.com
Tue Jul 25 20:32:44 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: f82dc74981d8b65a808c9c9fa7f9af7496e51074
https://github.com/WebKit/WebKit/commit/f82dc74981d8b65a808c9c9fa7f9af7496e51074
Author: Alan Baradlay <zalan at apple.com>
Date: 2023-07-25 (Tue, 25 Jul 2023)
Changed paths:
A LayoutTests/fullscreen/fullscreen-containing-block-change-expected.txt
A LayoutTests/fullscreen/fullscreen-containing-block-change-with-continuation-expected.txt
A LayoutTests/fullscreen/fullscreen-containing-block-change-with-continuation.html
A LayoutTests/fullscreen/fullscreen-containing-block-change.html
M Source/WebCore/dom/FullscreenManager.cpp
Log Message:
-----------
REGRESSION(265043 at main): Fullscreen mode only shows video in some part of the screen on Twitter
https://bugs.webkit.org/show_bug.cgi?id=258907
rdar://111095697>
Reviewed by Simon Fraser and Tim Nguyen.
Add-to-top-layer changes containing block state which may affect geometry and requires layout to run.
This patch ensures that all the relevant pieces are marked for layout when transitioning to fullscreen including cases when a child frame initiates it.
It is essentially mimicking what out-of-flow type of containing block change triggers through styleDid/WillChange
e.g. when going from absolute to fix with a relative positioned ancestor.
(Test case credit goes to Tim Nguyen)
* LayoutTests/fullscreen/fullscreen-containing-block-change-expected.txt: Added.
* LayoutTests/fullscreen/fullscreen-containing-block-change-with-continuation-expected.txt: Added.
* LayoutTests/fullscreen/fullscreen-containing-block-change-with-continuation.html: Added.
* LayoutTests/fullscreen/fullscreen-containing-block-change.html: Added.
* Source/WebCore/dom/FullscreenManager.cpp:
(WebCore::markRendererDirtyAfterTopLayerChange):
(WebCore::FullscreenManager::willEnterFullscreen):
Canonical link: https://commits.webkit.org/266309@main
More information about the webkit-changes
mailing list