Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 6ee5bd84aa85a20079fbf8e63c42439c7a9b5cec
https://github.com/WebKit/WebKit/commit/6ee5bd84aa85a20079fbf8e63c42439c7a9b5cec
Author: Matt Woodrow <[email protected]>
Date: 2025-04-01 (Tue, 01 Apr 2025)
Changed paths:
A
LayoutTests/fast/repaint/content-visibility-hidden-container-with-abs-pos-c-v-auto-child-reftest-expected.html
A
LayoutTests/fast/repaint/content-visibility-hidden-container-with-abs-pos-c-v-auto-child-reftest.html
M
LayoutTests/fast/repaint/incorrect-repaint-when-child-layer-overflows-expected.txt
M Source/WebCore/rendering/RenderElement.cpp
M Source/WebCore/rendering/RenderLayer.cpp
Log Message:
-----------
REGRESSION (281882@main):
LayoutTests/fast/repaint/content-visibility-hidden-container-with-abs-pos-c-v-auto-child.html
fails.
https://bugs.webkit.org/show_bug.cgi?id=290723
<rdar://148183112>
Reviewed by Alan Baradlay.
Changing the value of content-visibility requires that child layers recompute
their visible content status (RenderLayer::computeHasVisibleContent() checks
isSkippedContent(), not hasSkippedContent()).
We also can't use the fast path of setHasVisibleContent() (instead of setting
the dirty bit) when setting visibility:visible, since the layer can still be
hidden (by content-visibility, or overflow truncation).
Duplicate the existing repaint rect test to have a reftest equivalent.
*
LayoutTests/fast/repaint/content-visibility-hidden-container-with-abs-pos-c-v-auto-child-reftest-expected.html:
Added.
*
LayoutTests/fast/repaint/content-visibility-hidden-container-with-abs-pos-c-v-auto-child-reftest.html:
Added.
* Source/WebCore/rendering/RenderElement.cpp:
(WebCore::RenderElement::styleWillChange):
Canonical link: https://commits.webkit.org/293085@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
