Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: b81ec57abd524faa522348fed9722e2b35d40ec8
https://github.com/WebKit/WebKit/commit/b81ec57abd524faa522348fed9722e2b35d40ec8
Author: Alan Baradlay <[email protected]>
Date: 2026-09-17 (Thu, 17 Sep 2026)
Changed paths:
M Source/WebCore/rendering/RenderBox.cpp
M Source/WebCore/rendering/RenderBoxModelObject.cpp
M Source/WebCore/rendering/RenderInline.cpp
Log Message:
-----------
[cleanup] The box-only conditions of requiresLayer() belong to RenderBox
https://bugs.webkit.org/show_bug.cgi?id=324461
<rdar://problem/187702297>
Reviewed by Antti Koivisto.
RenderBox::requiresLayer() was the only caller of
RenderBoxModelObject::requiresLayer(), so the document element, the transform
related properties, the hidden backface, the reflection and the view transition
capture moved there.
hasRunningAcceleratedAnimations() went the other way: both overrides asked it
and it already lives on RenderBoxModelObject.
An inline box can now ask the base instead of repeating createsGroup() and
requiresRenderingConsolidationForViewTransition(); isPositioned() answers what
isInFlowPositioned() did, since an out-of-flow position blockifies. The clip
path is gone with it: createsGroup() already asks !style().clipPath().isNone().
* Source/WebCore/rendering/RenderBox.cpp:
(WebCore::RenderBox::requiresLayer const):
* Source/WebCore/rendering/RenderBoxModelObject.cpp:
(WebCore::RenderBoxModelObject::requiresLayer const):
* Source/WebCore/rendering/RenderInline.cpp:
(WebCore::RenderInline::requiresLayer const):
Canonical link: https://commits.webkit.org/321350@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
