Attachment #454350 for bug #237711

View Details Raw Unified Return to Bug 237711

| Differences between



2022-03-10  Nikolas Zimmermann  <nzimmermann@igalia.com>

        [LBSE] Activate SVG transform support through layers
        https://bugs.webkit.org/show_bug.cgi?id=237711

        Reviewed by NOBODY (OOPS!).

        Activate transforms for SVG layers.

        Use the flag to keep track of the presence of SVG transformations
        on a renderer. CSS/HTML renderers can rely only on the RenderStyle
        to determine if transformations are applied, whereas SVG has to
        consult additional sources, such as the SVG 'transform' attribute.

        A SVG renderer with the 'HasSVGTransform' flag set should behave
        like any SVG/HTML renderer that has CSS transformations applied.

        TODO: Write more about this.

        Covered by existing tests, no change in behaviour.

        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::updateLayerTransform): Deleted.
        * rendering/RenderBox.h:
        * rendering/RenderLayer.cpp:
        (WebCore::canCreateStackingContext):
        (WebCore::updateTransformFromStyle):
        (WebCore::RenderLayer::currentTransform const):
        * rendering/RenderLayerModelObject.cpp:
        (WebCore::RenderLayerModelObject::updateLayerTransform):
        (WebCore::RenderLayerModelObject::applySVGTransform const):
        * rendering/RenderLayerModelObject.h:
        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::setHasSVGTransform):
        * rendering/RenderObject.h:
        (WebCore::RenderObject::hasSVGTransform const):
        (WebCore::RenderObject::hasTransform const):
        * rendering/svg/RenderSVGContainer.cpp:
        (WebCore::SVGLayerTransformUpdater::SVGLayerTransformUpdater):
        (WebCore::SVGLayerTransformUpdater::~SVGLayerTransformUpdater):
        (WebCore::RenderSVGContainer::layout):
        * rendering/svg/RenderSVGContainer.h:
        (WebCore::RenderSVGContainer::updateLayerInformation): Deleted.
        * rendering/svg/RenderSVGModelObject.cpp:
        (WebCore::RenderSVGModelObject::updateFromStyle):
        * rendering/svg/RenderSVGRoot.cpp:
        (WebCore::RenderSVGRoot::updateFromStyle):
        * rendering/svg/RenderSVGShape.cpp:
        (WebCore::RenderSVGShape::layout):
        (WebCore::RenderSVGShape::applyTransform const):
        * rendering/svg/RenderSVGShape.h:
        * rendering/svg/RenderSVGTransformableContainer.cpp:
        (WebCore::RenderSVGTransformableContainer::updateFromStyle):
        (WebCore::RenderSVGTransformableContainer::applyTransform const):
        * svg/SVGGraphicsElement.cpp:
        (WebCore::SVGGraphicsElement::animatedLocalTransform const):

2022-03-10  Nikolas Zimmermann  <nzimmermann@igalia.com>

        Unify reference box / CTM computation in RenderLayer



    quads.append(localToAbsoluteQuad(localRect, UseTransforms, wasFixed));
}

void RenderBox::updateLayerTransform()
{
    // Transform-origin depends on box size, so we need to update the layer transform after layout.
    if (hasLayer())
        layer()->updateTransform();
}

void RenderBox::applyTransform(TransformationMatrix& t, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const
{
    style.applyTransform(t, boundingBox, options);

Lines 218-224 public: a/Source/WebCore/rendering/RenderBox.h_sec1
- a/Source/WebCore/rendering/RenderBox.h -1 lines
218	void addOverflowFromChild(const RenderBox* child) { addOverflowFromChild(child, child->locationOffset()); }	218	void addOverflowFromChild(const RenderBox* child) { addOverflowFromChild(child, child->locationOffset()); }
219	void addOverflowFromChild(const RenderBox* child, const LayoutSize& delta);	219	void addOverflowFromChild(const RenderBox* child, const LayoutSize& delta);
220		220
221	void updateLayerTransform();
222	void applyTransform(TransformationMatrix&, const RenderStyle&, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> = RenderStyle::allTransformOperations) const override;	221	void applyTransform(TransformationMatrix&, const RenderStyle&, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> = RenderStyle::allTransformOperations) const override;
223		222
224	LayoutSize contentSize() const { return { contentWidth(), contentHeight() }; }	223	LayoutSize contentSize() const { return { contentWidth(), contentHeight() }; }

Lines 552-557 static bool canCreateStackingContext(const RenderLayer& layer) a/Source/WebCore/rendering/RenderLayer.cpp_sec1
- a/Source/WebCore/rendering/RenderLayer.cpp -14 / +17 lines
552	{	552	{
553	auto& renderer = layer.renderer();	553	auto& renderer = layer.renderer();
554	return renderer.hasTransformRelatedProperty()	554	return renderer.hasTransformRelatedProperty()
		555	#if ENABLE(LAYER_BASED_SVG_ENGINE)
		556	\|\| renderer.hasSVGTransform()
		557	#endif
555	\|\| renderer.hasClipPath()	558	\|\| renderer.hasClipPath()
556	\|\| renderer.hasFilter()	559	\|\| renderer.hasFilter()
557	\|\| renderer.hasMask()	560	\|\| renderer.hasMask()
Lines 1310-1322 static inline FloatRect computeReferenceBox(const RenderLayerModelObject& render a/Source/WebCore/rendering/RenderLayer.cpp_sec2
1310		1313
1311	static inline void updateTransformFromStyle(TransformationMatrix& transform, const RenderLayerModelObject& renderer, const RenderStyle& style, OptionSet<RenderStyle::TransformOperationOption> options)	1314	static inline void updateTransformFromStyle(TransformationMatrix& transform, const RenderLayerModelObject& renderer, const RenderStyle& style, OptionSet<RenderStyle::TransformOperationOption> options)
1312	{	1315	{
1313	FloatRect referenceBox;	1316	auto referenceBox = renderer.transformReferenceBoxRect();
1314		1317
1315	// FIXME: [LBSE] Upstream reference box computation for RenderSVGModelObject derived renderers	1318	#if ENABLE(LAYER_BASED_SVG_ENGINE)
1316	if (is<RenderBox>(renderer))	1319	// No pixel snapping within SVG subtree.
1317	referenceBox = snapRectToDevicePixels(LayoutRect(renderer.transformReferenceBoxRect()), renderer.document().deviceScaleFactor());	1320	if (renderer.isSVGLayerAwareRenderer() && renderer.document().settings().layerBasedSVGEngineEnabled()) {
		1321	renderer.applyTransform(transform, style, referenceBox, options);
		1322	makeMatrixRenderable(transform, renderer.view().compositor().canRender3DTransforms());
		1323	return;
		1324	}
		1325	#endif
1318		1326
1319	renderer.applyTransform(transform, style, referenceBox, options);	1327	auto pixelSnappedReferenceBox = snapRectToDevicePixels(LayoutRect(referenceBox), renderer.document().deviceScaleFactor());
		1328	renderer.applyTransform(transform, style, pixelSnappedReferenceBox, options);
1320	makeMatrixRenderable(transform, renderer.view().compositor().canRender3DTransforms());	1329	makeMatrixRenderable(transform, renderer.view().compositor().canRender3DTransforms());
1321	}	1330	}
1322		1331
Lines 1353-1374 TransformationMatrix RenderLayer::currentTransform(OptionSet<RenderStyle::Transf a/Source/WebCore/rendering/RenderLayer.cpp_sec3
1353	if (!m_transform)	1362	if (!m_transform)
1354	return { };	1363	return { };
1355		1364
1356	// FIXME: [LBSE] Upstream transform support for RenderSVGModelObject derived renderers
1357	if (!is<RenderBox>(renderer()))
1358	return { };
1359
1360	auto& renderBox = downcast<RenderBox>(renderer());
1361
1362	// m_transform includes transform-origin and is affected by the choice of the transform-box.	1365	// m_transform includes transform-origin and is affected by the choice of the transform-box.
1363	// Therefore we can only use the cached m_transform, if the animation doesn't alter transform-box or excludes transform-origin.	1366	// Therefore we can only use the cached m_transform, if the animation doesn't alter transform-box or excludes transform-origin.
1364		1367
1365	// Query the animatedStyle() to obtain the current transformation, when accelerated transform animations are running.	1368	// Query the animatedStyle() to obtain the current transformation, when accelerated transform animations are running.
1366	auto styleable = Styleable::fromRenderer(renderBox);	1369	auto styleable = Styleable::fromRenderer(renderer());
1367	if ((styleable && styleable->isRunningAcceleratedTransformAnimation()) \|\| !options.contains(RenderStyle::TransformOperationOption::TransformOrigin)) {	1370	if ((styleable && styleable->isRunningAcceleratedTransformAnimation()) \|\| !options.contains(RenderStyle::TransformOperationOption::TransformOrigin)) {
1368	std::unique_ptr<RenderStyle> animatedStyle = renderBox.animatedStyle();	1371	std::unique_ptr<RenderStyle> animatedStyle = renderer().animatedStyle();
1369		1372
1370	TransformationMatrix transform;	1373	TransformationMatrix transform;
1371	updateTransformFromStyle(transform, renderBox, *animatedStyle, options);	1374	updateTransformFromStyle(transform, renderer(), *animatedStyle, options);
1372	return transform;	1375	return transform;
1373	}	1376	}
1374		1377

Lines 31-36 a/Source/WebCore/rendering/RenderLayerModelObject.cpp_sec1
- a/Source/WebCore/rendering/RenderLayerModelObject.cpp +40 lines
31	#include "RenderLayerScrollableArea.h"	31	#include "RenderLayerScrollableArea.h"
32	#include "RenderSVGModelObject.h"	32	#include "RenderSVGModelObject.h"
33	#include "RenderView.h"	33	#include "RenderView.h"
		34	#include "SVGGraphicsElement.h"
34	#include "Settings.h"	35	#include "Settings.h"
35	#include "StyleScrollSnapPoints.h"	36	#include "StyleScrollSnapPoints.h"
36	#include "TransformState.h"	37	#include "TransformState.h"
Lines 231-236 void RenderLayerModelObject::suspendAnimations(MonotonicTime time) a/Source/WebCore/rendering/RenderLayerModelObject.cpp_sec2
231	layer()->backing()->suspendAnimations(time);	232	layer()->backing()->suspendAnimations(time);
232	}	233	}
233		234
		235	void RenderLayerModelObject::updateLayerTransform()
		236	{
		237	// Transform-origin depends on box size, so we need to update the layer transform after layout.
		238	if (hasLayer())
		239	layer()->updateTransform();
		240	}
		241
234	#if ENABLE(LAYER_BASED_SVG_ENGINE)	242	#if ENABLE(LAYER_BASED_SVG_ENGINE)
235	std::optional<LayoutRect> RenderLayerModelObject::computeVisibleRectInSVGContainer(const LayoutRect& rect, const RenderLayerModelObject* container, RenderObject::VisibleRectContext context) const	243	std::optional<LayoutRect> RenderLayerModelObject::computeVisibleRectInSVGContainer(const LayoutRect& rect, const RenderLayerModelObject* container, RenderObject::VisibleRectContext context) const
236	{	244	{
Lines 316-321 void RenderLayerModelObject::mapLocalToSVGContainer(const RenderLayerModelObject a/Source/WebCore/rendering/RenderLayerModelObject.cpp_sec3
316		324
317	container->mapLocalToContainer(ancestorContainer, transformState, mode, wasFixed);	325	container->mapLocalToContainer(ancestorContainer, transformState, mode, wasFixed);
318	}	326	}
		327
		328	void RenderLayerModelObject::applySVGTransform(TransformationMatrix& transform, SVGGraphicsElement& graphicsElement, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const
		329	{
		330	// This check does not use style.hasTransformRelatedProperty() on purpose -- we only want to know if either the 'transform' property, an
		331	// offset path, or the individual transform operations are set (perspective / transform-style: preserve-3d are not relevant here).
		332	bool hasCSSTransform = style.hasTransform() \|\| style.rotate() \|\| style.translate() \|\| style.scale();
		333
		334	bool affectedByTransformOrigin = false;
		335	std::optional<AffineTransform> svgTransform;
		336
		337	if (hasCSSTransform)
		338	affectedByTransformOrigin = style.affectedByTransformOrigin();
		339	else if (auto affineTransform = graphicsElement.animatedLocalTransform(); !affineTransform.isIdentity()) {
		340	svgTransform = affineTransform;
		341	affectedByTransformOrigin = affineTransform.a() != 1 \|\| affineTransform.b() \|\| affineTransform.c() \|\| affineTransform.d() != 1;
		342	}
		343
		344	if (!hasCSSTransform && !svgTransform.has_value())
		345	return;
		346
		347	FloatPoint3D originTranslate;
		348	if (options.contains(RenderStyle::TransformOperationOption::TransformOrigin) && affectedByTransformOrigin)
		349	originTranslate = style.applyTransformOrigin(transform, boundingBox);
		350
		351	// CSS transforms take precedence over SVG transforms.
		352	if (hasCSSTransform)
		353	style.applyCSSTransform(transform, boundingBox, options);
		354	else
		355	transform.multiplyAffineTransform(svgTransform.value());
		356
		357	style.unapplyTransformOrigin(transform, originTranslate);
		358	}
319	#endif	359	#endif
320		360
321	} // namespace WebCore	361	} // namespace WebCore

Lines 28-33 namespace WebCore { a/Source/WebCore/rendering/RenderLayerModelObject.h_sec1
- a/Source/WebCore/rendering/RenderLayerModelObject.h +5 lines
28		28
29	class KeyframeList;	29	class KeyframeList;
30	class RenderLayer;	30	class RenderLayer;
		31	class SVGGraphicsElement;
31		32
32	struct LayerRepaintRects {	33	struct LayerRepaintRects {
33	LayoutRect clippedOverflowRect;	34	LayoutRect clippedOverflowRect;
Lines 75-82 public: a/Source/WebCore/rendering/RenderLayerModelObject.h_sec2
75	// Provides the SVG implementation for mapLocalToContainer().	76	// Provides the SVG implementation for mapLocalToContainer().
76	// This lives in RenderLayerModelObject, which is the common base-class for all SVG renderers.	77	// This lives in RenderLayerModelObject, which is the common base-class for all SVG renderers.
77	void mapLocalToSVGContainer(const RenderLayerModelObject* ancestorContainer, TransformState&, OptionSet<MapCoordinatesMode>, bool* wasFixed) const;	78	void mapLocalToSVGContainer(const RenderLayerModelObject* ancestorContainer, TransformState&, OptionSet<MapCoordinatesMode>, bool* wasFixed) const;
		79
		80	void applySVGTransform(TransformationMatrix&, SVGGraphicsElement&, const RenderStyle&, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption>) const;
78	#endif	81	#endif
79		82
		83	void updateLayerTransform();
		84
80	virtual void applyTransform(TransformationMatrix&, const RenderStyle&, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> = RenderStyle::allTransformOperations) const = 0;	85	virtual void applyTransform(TransformationMatrix&, const RenderStyle&, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> = RenderStyle::allTransformOperations) const = 0;
81		86
82	protected:	87	protected:

Lines 1975-1980 void RenderObject::setHasOutlineAutoAncestor(bool hasOutlineAutoAncestor) a/Source/WebCore/rendering/RenderObject.cpp_sec1
- a/Source/WebCore/rendering/RenderObject.cpp +8 lines
1975	ensureRareData().setHasOutlineAutoAncestor(hasOutlineAutoAncestor);	1975	ensureRareData().setHasOutlineAutoAncestor(hasOutlineAutoAncestor);
1976	}	1976	}
1977		1977
		1978	#if ENABLE(LAYER_BASED_SVG_ENGINE)
		1979	void RenderObject::setHasSVGTransform(bool hasSVGTransform)
		1980	{
		1981	if (hasSVGTransform \|\| hasRareData())
		1982	ensureRareData().setHasSVGTransform(hasSVGTransform);
		1983	}
		1984	#endif
		1985
1978	void RenderObject::setPaintContainmentApplies(bool paintContainmentApplies)	1986	void RenderObject::setPaintContainmentApplies(bool paintContainmentApplies)
1979	{	1987	{
1980	if (paintContainmentApplies \|\| hasRareData())	1988	if (paintContainmentApplies \|\| hasRareData())

Lines 415-420 public: a/Source/WebCore/rendering/RenderObject.h_sec1
- a/Source/WebCore/rendering/RenderObject.h -1 / +13 lines
415	bool hasOutlineAutoAncestor() const { return m_bitfields.hasRareData() && rareData().hasOutlineAutoAncestor(); }	415	bool hasOutlineAutoAncestor() const { return m_bitfields.hasRareData() && rareData().hasOutlineAutoAncestor(); }
416	bool paintContainmentApplies() const { return m_bitfields.hasRareData() && rareData().paintContainmentApplies(); }	416	bool paintContainmentApplies() const { return m_bitfields.hasRareData() && rareData().paintContainmentApplies(); }
417		417
		418	#if ENABLE(LAYER_BASED_SVG_ENGINE)
		419	bool hasSVGTransform() const { return m_bitfields.hasRareData() && rareData().hasSVGTransform(); }
		420	#else
		421	bool hasSVGTransform() const { return false; }
		422	#endif
		423
418	bool isExcludedFromNormalLayout() const { return m_bitfields.isExcludedFromNormalLayout(); }	424	bool isExcludedFromNormalLayout() const { return m_bitfields.isExcludedFromNormalLayout(); }
419	void setIsExcludedFromNormalLayout(bool excluded) { m_bitfields.setIsExcludedFromNormalLayout(excluded); }	425	void setIsExcludedFromNormalLayout(bool excluded) { m_bitfields.setIsExcludedFromNormalLayout(excluded); }
420	bool isExcludedAndPlacedInBorder() const { return isExcludedFromNormalLayout() && isLegend(); }	426	bool isExcludedAndPlacedInBorder() const { return isExcludedFromNormalLayout() && isLegend(); }
Lines 449-455 public: a/Source/WebCore/rendering/RenderObject.h_sec2
449	bool hasPotentiallyScrollableOverflow() const;	455	bool hasPotentiallyScrollableOverflow() const;
450		456
451	bool hasTransformRelatedProperty() const { return m_bitfields.hasTransformRelatedProperty(); } // Transform, perspective or transform-style: preserve-3d.	457	bool hasTransformRelatedProperty() const { return m_bitfields.hasTransformRelatedProperty(); } // Transform, perspective or transform-style: preserve-3d.
452	bool hasTransform() const { return hasTransformRelatedProperty() && (style().hasTransform() \|\| style().translate() \|\| style().scale() \|\| style().rotate()); }	458	bool hasTransform() const { return hasTransformRelatedProperty() && (style().hasTransform() \|\| style().translate() \|\| style().scale() \|\| style().rotate() \|\| hasSVGTransform()); }
453	bool hasTransformOrPespective() const { return hasTransformRelatedProperty() && (hasTransform() \|\| style().hasPerspective()); }	459	bool hasTransformOrPespective() const { return hasTransformRelatedProperty() && (hasTransform() \|\| style().hasPerspective()); }
454		460
455	inline bool preservesNewline() const;	461	inline bool preservesNewline() const;
Lines 514-519 public: a/Source/WebCore/rendering/RenderObject.h_sec3
514	void setIsRenderFragmentedFlow(bool = true);	520	void setIsRenderFragmentedFlow(bool = true);
515	void setHasOutlineAutoAncestor(bool = true);	521	void setHasOutlineAutoAncestor(bool = true);
516	void setPaintContainmentApplies(bool = true);	522	void setPaintContainmentApplies(bool = true);
		523	#if ENABLE(LAYER_BASED_SVG_ENGINE)
		524	void setHasSVGTransform(bool = true);
		525	#endif
517		526
518	// Hook so that RenderTextControl can return the line height of its inner renderer.	527	// Hook so that RenderTextControl can return the line height of its inner renderer.
519	// For other renderers, the value is the same as lineHeight(false).	528	// For other renderers, the value is the same as lineHeight(false).
Lines 943-948 private: a/Source/WebCore/rendering/RenderObject.h_sec4
943	ADD_BOOLEAN_BITFIELD(isRenderFragmentedFlow, IsRenderFragmentedFlow);	952	ADD_BOOLEAN_BITFIELD(isRenderFragmentedFlow, IsRenderFragmentedFlow);
944	ADD_BOOLEAN_BITFIELD(hasOutlineAutoAncestor, HasOutlineAutoAncestor);	953	ADD_BOOLEAN_BITFIELD(hasOutlineAutoAncestor, HasOutlineAutoAncestor);
945	ADD_BOOLEAN_BITFIELD(paintContainmentApplies, PaintContainmentApplies);	954	ADD_BOOLEAN_BITFIELD(paintContainmentApplies, PaintContainmentApplies);
		955	#if ENABLE(LAYER_BASED_SVG_ENGINE)
		956	ADD_BOOLEAN_BITFIELD(hasSVGTransform, HasSVGTransform);
		957	#endif
946		958
947	// From RenderElement	959	// From RenderElement
948	std::unique_ptr<ReferencedSVGResources> referencedSVGResources;	960	std::unique_ptr<ReferencedSVGResources> referencedSVGResources;




RenderSVGContainer::~RenderSVGContainer() = default;

// Helper class, move to its own file once utilized in more than one place.
class SVGLayerTransformUpdater {
    WTF_MAKE_NONCOPYABLE(SVGLayerTransformUpdater);
public:
    SVGLayerTransformUpdater(RenderLayerModelObject& renderer)
        : m_renderer(renderer)
    {
        if (!m_renderer.hasLayer())
            return;

        m_transformReferenceBox = m_renderer.transformReferenceBoxRect();
        m_renderer.updateLayerTransform();
    }


    ~SVGLayerTransformUpdater()
    {
        if (!m_renderer.hasLayer())
            return;
        if (m_renderer.transformReferenceBoxRect() == m_transformReferenceBox)
            return;

        m_renderer.updateLayerTransform();
    }

private:
    RenderLayerModelObject& m_renderer;
    FloatRect m_transformReferenceBox;
};

void RenderSVGContainer::layout()
{
    StackStats::LayoutCheckPoint layoutCheckPoint;

    // Update layer transform before laying out children (SVG needs access to the transform matrices during layout for on-screen text font-size calculations).
    // Eventually re-update if the transform reference box, relevant for transform-origin, has changed during layout.
    {
        SVGLayerTransformUpdater updateTransform(*this);
        // SVGLayerTransformUpdater transformUpdater(*this);
        layoutChildren();
    }

    updateLayerInformation();

    // Invalidate all resources of this client if our layout changed.
    if (everHadLayout() && needsLayout())
        SVGResourcesCache::clientLayoutChanged(*this);

Lines 55-61 protected: a/Source/WebCore/rendering/svg/RenderSVGContainer.h_sec1
- a/Source/WebCore/rendering/svg/RenderSVGContainer.h -1 lines
55	virtual void layoutChildren();	55	virtual void layoutChildren();
56	bool nodeAtPoint(const HitTestRequest&, HitTestResult&, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, HitTestAction) override;	56	bool nodeAtPoint(const HitTestRequest&, HitTestResult&, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, HitTestAction) override;
57		57
58	virtual void updateLayerInformation() { }
59	virtual void calculateViewport();	58	virtual void calculateViewport();
60	virtual bool pointIsInsideViewportClip(const FloatPoint&) { return true; }	59	virtual bool pointIsInsideViewportClip(const FloatPoint&) { return true; }
61		60



#include "RenderGeometryMap.h"
#include "RenderLayer.h"
#include "RenderLayerModelObject.h"
#include "RenderSVGModelObjectInlines.h"
#include "RenderSVGResource.h"
#include "RenderView.h"
#include "SVGElementInlines.h"

void RenderSVGModelObject::updateFromStyle()
{
    RenderLayerModelObject::updateFromStyle();
    setHasTransformRelatedProperty(style().hasTransformRelatedProperty());

    bool hasSVGTransform = false;
    if (is<SVGGraphicsElement>(element()))
        hasSVGTransform = !downcast<SVGGraphicsElement>(element()).animatedLocalTransform().isIdentity();

    setHasTransformRelatedProperty(style().hasTransformRelatedProperty() || hasSVGTransform);
    setHasSVGTransform(hasSVGTransform);
    //     setHasSVGTransform();
}

FloatRect RenderSVGModelObject::borderBoxRectInFragmentEquivalent(RenderFragmentContainer*, RenderBox::RenderBoxFragmentInfoFlags) const



{
    RenderReplaced::updateFromStyle();

    setHasSVGTransform();
    // setHasSVGTransform();

    if (shouldApplyViewportClip())
        setHasNonVisibleOverflow();



        setLayoutRect(enclosingLayoutRect(m_fillBoundingBox));
    }

    updateLayerTransform();
    // SVGRenderSupport::updateLayerTransform(*this);

    // Invalidate all resources of this client if our layout changed.
    if (everHadLayout() && selfNeedsLayout())


void RenderSVGShape::applyTransform(TransformationMatrix& transform, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const
{
    applySVGTransform(transform, graphicsElement(), style, boundingBox, options);
    style.applyTransform(transform, boundingBox, options);
}

}

Lines 82-88 public: a/Source/WebCore/rendering/svg/RenderSVGShape.h_sec1
- a/Source/WebCore/rendering/svg/RenderSVGShape.h -1 / +1 lines
82		82
83	FloatRect computeMarkerBoundingBox(const SVGBoundingBoxComputation::DecorationOptions&) const;	83	FloatRect computeMarkerBoundingBox(const SVGBoundingBoxComputation::DecorationOptions&) const;
84		84
85	void applyTransform(TransformationMatrix&, const RenderStyle&, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> = RenderStyle::allTransformOperations) const override;	85	void applyTransform(TransformationMatrix&, const RenderStyle&, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> = RenderStyle::allTransformOperations) const final;
86		86
87	protected:	87	protected:
88	void element() const = delete;	88	void element() const = delete;



{
    RenderSVGContainer::updateFromStyle();

    if (associatedUseElement(graphicsElement()))
        setHasSVGTransform();
    //    setHasSVGTransform();
}

void RenderSVGTransformableContainer::applyTransform(TransformationMatrix& transform, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const
{
    applySVGTransform(transform, graphicsElement(), style, boundingBox, options);
    style.applyTransform(transform, boundingBox, options);
}

void RenderSVGTransformableContainer::styleWillChange(StyleDifference diff, const RenderStyle& newStyle)

Lines 73-78 AffineTransform SVGGraphicsElement::getScreenCTM(StyleUpdateStrategy styleUpdate a/Source/WebCore/svg/SVGGraphicsElement.cpp_sec1
- a/Source/WebCore/svg/SVGGraphicsElement.cpp +9 lines
73		73
74	AffineTransform SVGGraphicsElement::animatedLocalTransform() const	74	AffineTransform SVGGraphicsElement::animatedLocalTransform() const
75	{	75	{
		76	#if ENABLE(LAYER_BASED_SVG_ENGINE)
		77	// LBSE handles transforms via RenderLayer, no need to handle CSS transforms here.
		78	if (document().settings().layerBasedSVGEngineEnabled()) {
		79	if (m_supplementalTransform)
		80	return m_supplementalTransform transform().concatenate();
		81	return transform().concatenate();
		82	}
		83	#endif
		84
76	AffineTransform matrix;	85	AffineTransform matrix;
77		86
78	auto* style = renderer() ? &renderer()->style() : nullptr;	87	auto* style = renderer() ? &renderer()->style() : nullptr;

Return to Bug 237711

Lines 662-674 void RenderBox::absoluteQuads(Vector<FloatQuad>& quads, bool* wasFixed) const a/Source/WebCore/rendering/RenderBox.cpp_sec1
- a/Source/WebCore/rendering/RenderBox.cpp -7 lines
662	quads.append(localToAbsoluteQuad(localRect, UseTransforms, wasFixed));	662	quads.append(localToAbsoluteQuad(localRect, UseTransforms, wasFixed));
663	}	663	}
664		664
665	void RenderBox::updateLayerTransform()
666	{
667	// Transform-origin depends on box size, so we need to update the layer transform after layout.
668	if (hasLayer())
669	layer()->updateTransform();
670	}
671
672	void RenderBox::applyTransform(TransformationMatrix& t, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const	665	void RenderBox::applyTransform(TransformationMatrix& t, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const
673	{	666	{
674	style.applyTransform(t, boundingBox, options);	667	style.applyTransform(t, boundingBox, options);

Lines 396-403 void RenderSVGRoot::updateFromStyle() a/Source/WebCore/rendering/svg/RenderSVGRoot.cpp_sec1
- a/Source/WebCore/rendering/svg/RenderSVGRoot.cpp -2 / +1 lines
396	{	396	{
397	RenderReplaced::updateFromStyle();	397	RenderReplaced::updateFromStyle();
398		398
399	// FIXME: [LBSE] Upstream RenderObject changes	399	setHasSVGTransform();
400	// setHasSVGTransform();
401		400
402	if (shouldApplyViewportClip())	401	if (shouldApplyViewportClip())
403	setHasNonVisibleOverflow();	402	setHasNonVisibleOverflow();

Lines 154-161 void RenderSVGShape::layout() a/Source/WebCore/rendering/svg/RenderSVGShape.cpp_sec1
- a/Source/WebCore/rendering/svg/RenderSVGShape.cpp -4 / +2 lines
154	setLayoutRect(enclosingLayoutRect(m_fillBoundingBox));	154	setLayoutRect(enclosingLayoutRect(m_fillBoundingBox));
155	}	155	}
156		156
157	// FIXME: [LBSE] Upstream SVGLayerTransformUpdater	157	updateLayerTransform();
158	// SVGRenderSupport::updateLayerTransform(*this);
159		158
160	// Invalidate all resources of this client if our layout changed.	159	// Invalidate all resources of this client if our layout changed.
161	if (everHadLayout() && selfNeedsLayout())	160	if (everHadLayout() && selfNeedsLayout())
Lines 536-543 void RenderSVGShape::styleWillChange(StyleDifference diff, const RenderStyle& ne a/Source/WebCore/rendering/svg/RenderSVGShape.cpp_sec2
536		535
537	void RenderSVGShape::applyTransform(TransformationMatrix& transform, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const	536	void RenderSVGShape::applyTransform(TransformationMatrix& transform, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const
538	{	537	{
539	// FIXME: [LBSE] Upstream CSS/SVG transform support	538	applySVGTransform(transform, graphicsElement(), style, boundingBox, options);
540	style.applyTransform(transform, boundingBox, options);
541	}	539	}
542		540
543	}	541	}

Lines 97-111 void RenderSVGTransformableContainer::updateFromStyle() a/Source/WebCore/rendering/svg/RenderSVGTransformableContainer.cpp_sec1
- a/Source/WebCore/rendering/svg/RenderSVGTransformableContainer.cpp -5 / +3 lines
97	{	97	{
98	RenderSVGContainer::updateFromStyle();	98	RenderSVGContainer::updateFromStyle();
99		99
100	// FIXME: [LBSE] Upstream CSS/SVG transform support	100	if (associatedUseElement(graphicsElement()))
101	// if (associatedUseElement(graphicsElement()))	101	setHasSVGTransform();
102	// setHasSVGTransform();
103	}	102	}
104		103
105	void RenderSVGTransformableContainer::applyTransform(TransformationMatrix& transform, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const	104	void RenderSVGTransformableContainer::applyTransform(TransformationMatrix& transform, const RenderStyle& style, const FloatRect& boundingBox, OptionSet<RenderStyle::TransformOperationOption> options) const
106	{	105	{
107	// FIXME: [LBSE] Upstream CSS/SVG transform support	106	applySVGTransform(transform, graphicsElement(), style, boundingBox, options);
108	style.applyTransform(transform, boundingBox, options);
109	}	107	}
110		108
111	void RenderSVGTransformableContainer::styleWillChange(StyleDifference diff, const RenderStyle& newStyle)	109	void RenderSVGTransformableContainer::styleWillChange(StyleDifference diff, const RenderStyle& newStyle)

Lines 1-3 a/Source/WebCore/ChangeLog_sec1
- a/Source/WebCore/ChangeLog +57 lines
		1	2022-03-10 Nikolas Zimmermann <nzimmermann@igalia.com>
		2
		3	[LBSE] Activate SVG transform support through layers
		4	https://bugs.webkit.org/show_bug.cgi?id=237711
		5
		6	Reviewed by NOBODY (OOPS!).
		7
		8	Activate transforms for SVG layers.
		9
		10	Use the flag to keep track of the presence of SVG transformations
		11	on a renderer. CSS/HTML renderers can rely only on the RenderStyle
		12	to determine if transformations are applied, whereas SVG has to
		13	consult additional sources, such as the SVG 'transform' attribute.
		14
		15	A SVG renderer with the 'HasSVGTransform' flag set should behave
		16	like any SVG/HTML renderer that has CSS transformations applied.
		17
		18	TODO: Write more about this.
		19
		20	Covered by existing tests, no change in behaviour.
		21
		22	* rendering/RenderBox.cpp:
		23	(WebCore::RenderBox::updateLayerTransform): Deleted.
		24	* rendering/RenderBox.h:
		25	* rendering/RenderLayer.cpp:
		26	(WebCore::canCreateStackingContext):
		27	(WebCore::updateTransformFromStyle):
		28	(WebCore::RenderLayer::currentTransform const):
		29	* rendering/RenderLayerModelObject.cpp:
		30	(WebCore::RenderLayerModelObject::updateLayerTransform):
		31	(WebCore::RenderLayerModelObject::applySVGTransform const):
		32	* rendering/RenderLayerModelObject.h:
		33	* rendering/RenderObject.cpp:
		34	(WebCore::RenderObject::setHasSVGTransform):
		35	* rendering/RenderObject.h:
		36	(WebCore::RenderObject::hasSVGTransform const):
		37	(WebCore::RenderObject::hasTransform const):
		38	* rendering/svg/RenderSVGContainer.cpp:
		39	(WebCore::SVGLayerTransformUpdater::SVGLayerTransformUpdater):
		40	(WebCore::SVGLayerTransformUpdater::~SVGLayerTransformUpdater):
		41	(WebCore::RenderSVGContainer::layout):
		42	* rendering/svg/RenderSVGContainer.h:
		43	(WebCore::RenderSVGContainer::updateLayerInformation): Deleted.
		44	* rendering/svg/RenderSVGModelObject.cpp:
		45	(WebCore::RenderSVGModelObject::updateFromStyle):
		46	* rendering/svg/RenderSVGRoot.cpp:
		47	(WebCore::RenderSVGRoot::updateFromStyle):
		48	* rendering/svg/RenderSVGShape.cpp:
		49	(WebCore::RenderSVGShape::layout):
		50	(WebCore::RenderSVGShape::applyTransform const):
		51	* rendering/svg/RenderSVGShape.h:
		52	* rendering/svg/RenderSVGTransformableContainer.cpp:
		53	(WebCore::RenderSVGTransformableContainer::updateFromStyle):
		54	(WebCore::RenderSVGTransformableContainer::applyTransform const):
		55	* svg/SVGGraphicsElement.cpp:
		56	(WebCore::SVGGraphicsElement::animatedLocalTransform const):
		57
1	2022-03-10 Nikolas Zimmermann <nzimmermann@igalia.com>	58	2022-03-10 Nikolas Zimmermann <nzimmermann@igalia.com>
2		59
3	Unify reference box / CTM computation in RenderLayer	60	Unify reference box / CTM computation in RenderLayer

Lines 52-57 RenderSVGContainer::RenderSVGContainer(SVGElement& element, RenderStyle&& style) a/Source/WebCore/rendering/svg/RenderSVGContainer.cpp_sec1
- a/Source/WebCore/rendering/svg/RenderSVGContainer.cpp -4 / +31 lines
52		52
53	RenderSVGContainer::~RenderSVGContainer() = default;	53	RenderSVGContainer::~RenderSVGContainer() = default;
54		54
		55	// Helper class, move to its own file once utilized in more than one place.
		56	class SVGLayerTransformUpdater {
		57	WTF_MAKE_NONCOPYABLE(SVGLayerTransformUpdater);
		58	public:
		59	SVGLayerTransformUpdater(RenderLayerModelObject& renderer)
		60	: m_renderer(renderer)
		61	{
		62	if (!m_renderer.hasLayer())
		63	return;
		64
		65	m_transformReferenceBox = m_renderer.transformReferenceBoxRect();
		66	m_renderer.updateLayerTransform();
		67	}
		68
		69
		70	~SVGLayerTransformUpdater()
		71	{
		72	if (!m_renderer.hasLayer())
		73	return;
		74	if (m_renderer.transformReferenceBoxRect() == m_transformReferenceBox)
		75	return;
		76
		77	m_renderer.updateLayerTransform();
		78	}
		79
		80	private:
		81	RenderLayerModelObject& m_renderer;
		82	FloatRect m_transformReferenceBox;
		83	};
		84
55	void RenderSVGContainer::layout()	85	void RenderSVGContainer::layout()
56	{	86	{
57	StackStats::LayoutCheckPoint layoutCheckPoint;	87	StackStats::LayoutCheckPoint layoutCheckPoint;
Lines 64-76 void RenderSVGContainer::layout() a/Source/WebCore/rendering/svg/RenderSVGContainer.cpp_sec2
64	// Update layer transform before laying out children (SVG needs access to the transform matrices during layout for on-screen text font-size calculations).	94	// Update layer transform before laying out children (SVG needs access to the transform matrices during layout for on-screen text font-size calculations).
65	// Eventually re-update if the transform reference box, relevant for transform-origin, has changed during layout.	95	// Eventually re-update if the transform reference box, relevant for transform-origin, has changed during layout.
66	{	96	{
67	// FIXME: [LBSE] Upstream SVGLayerTransformUpdater	97	SVGLayerTransformUpdater updateTransform(*this);
68	// SVGLayerTransformUpdater transformUpdater(*this);
69	layoutChildren();	98	layoutChildren();
70	}	99	}
71		100
72	updateLayerInformation();
73
74	// Invalidate all resources of this client if our layout changed.	101	// Invalidate all resources of this client if our layout changed.
75	if (everHadLayout() && needsLayout())	102	if (everHadLayout() && needsLayout())
76	SVGResourcesCache::clientLayoutChanged(*this);	103	SVGResourcesCache::clientLayoutChanged(*this);

Lines 36-41 a/Source/WebCore/rendering/svg/RenderSVGModelObject.cpp_sec1
- a/Source/WebCore/rendering/svg/RenderSVGModelObject.cpp -7 / +6 lines
36	#include "RenderGeometryMap.h"	36	#include "RenderGeometryMap.h"
37	#include "RenderLayer.h"	37	#include "RenderLayer.h"
38	#include "RenderLayerModelObject.h"	38	#include "RenderLayerModelObject.h"
		39	#include "RenderSVGModelObjectInlines.h"
39	#include "RenderSVGResource.h"	40	#include "RenderSVGResource.h"
40	#include "RenderView.h"	41	#include "RenderView.h"
41	#include "SVGElementInlines.h"	42	#include "SVGElementInlines.h"
Lines 59-73 RenderSVGModelObject::RenderSVGModelObject(SVGElement& element, RenderStyle&& st a/Source/WebCore/rendering/svg/RenderSVGModelObject.cpp_sec2
59	void RenderSVGModelObject::updateFromStyle()	60	void RenderSVGModelObject::updateFromStyle()
60	{	61	{
61	RenderLayerModelObject::updateFromStyle();	62	RenderLayerModelObject::updateFromStyle();
62	setHasTransformRelatedProperty(style().hasTransformRelatedProperty());
63		63
64	AffineTransform transform;	64	bool hasSVGTransform = false;
65	if (is<SVGGraphicsElement>(nodeForNonAnonymous()))	65	if (is<SVGGraphicsElement>(element()))
66	transform = downcast<SVGGraphicsElement>(nodeForNonAnonymous()).animatedLocalTransform();	66	hasSVGTransform = !downcast<SVGGraphicsElement>(element()).animatedLocalTransform().isIdentity();
67		67
68	// FIXME: [LBSE] Upstream RenderObject changes	68	setHasTransformRelatedProperty(style().hasTransformRelatedProperty() \|\| hasSVGTransform);
69	// if (!transform.isIdentity())	69	setHasSVGTransform(hasSVGTransform);
70	// setHasSVGTransform();
71	}	70	}
72		71
73	FloatRect RenderSVGModelObject::borderBoxRectInFragmentEquivalent(RenderFragmentContainer*, RenderBox::RenderBoxFragmentInfoFlags) const	72	FloatRect RenderSVGModelObject::borderBoxRectInFragmentEquivalent(RenderFragmentContainer*, RenderBox::RenderBoxFragmentInfoFlags) const