Bug 38836

Confirmed with r58908.

The root cause of this bug is: the masker is not updated during animation. This problem can only reproduce with 'animateTransform' and 'animateMotion'. Other animation elements use a different mechanism: Everything is routed through 'setAttribute' calls, then the masker gets invalidated in attribute changed callback SVGStyledElement::svgAttributeChanged'. I have a quick fix for this bug. Add the following code to the end of SVGAnimateTransformElement::applyResultsToTarget() if (targetElement->isStyled()) static_cast<SVGStyledElement*>(targetElement)->invalidateResourcesInAncestorChain(); It works, but i'm not sure if this is the correct solution. Dirk, do you have any suggestions?

(In reply to comment #2) > The root cause of this bug is: the masker is not updated during animation. > This problem can only reproduce with 'animateTransform' and 'animateMotion'. Other animation elements use a different mechanism: > Everything is routed through 'setAttribute' calls, then the masker gets invalidated in attribute changed callback SVGStyledElement::svgAttributeChanged'. > > I have a quick fix for this bug. > Add the following code to the end of SVGAnimateTransformElement::applyResultsToTarget() > > if (targetElement->isStyled()) > static_cast<SVGStyledElement*>(targetElement)->invalidateResourcesInAncestorChain(); > > It works, but i'm not sure if this is the correct solution. > > Dirk, do you have any suggestions? Well, isn't it better to do it like other elements (SVGRectElement, SVGGElement, ...)? It would look like this: RenderObject* renderer = targetElement->renderer(); if (targetElement->isStyled()) { renderer->setNeedsTransformUpdate(); renderer->setNeedsLayout(true); } else renderer->setNeedsLayout(true); This is untested, but telling the renderer to just update the transform is maybe faster. And it's my opinion, that the code shouldn't handle resources manually here. Niko may know it better, since he changed layouting lately.

(In reply to comment #3) > (In reply to comment #2) > > The root cause of this bug is: the masker is not updated during animation. > > This problem can only reproduce with 'animateTransform' and 'animateMotion'. Other animation elements use a different mechanism: > > Everything is routed through 'setAttribute' calls, then the masker gets invalidated in attribute changed callback SVGStyledElement::svgAttributeChanged'. > > > > I have a quick fix for this bug. > > Add the following code to the end of SVGAnimateTransformElement::applyResultsToTarget() > > > > if (targetElement->isStyled()) > > static_cast<SVGStyledElement*>(targetElement)->invalidateResourcesInAncestorChain(); > > > > It works, but i'm not sure if this is the correct solution. > > > > Dirk, do you have any suggestions? > > Well, isn't it better to do it like other elements (SVGRectElement, SVGGElement, ...)? It would look like this: > > RenderObject* renderer = targetElement->renderer(); > if (targetElement->isStyled()) { > renderer->setNeedsTransformUpdate(); > renderer->setNeedsLayout(true); > } else > renderer->setNeedsLayout(true); > > This is untested, but telling the renderer to just update the transform is maybe faster. And it's my opinion, that the code shouldn't handle resources manually here. > > Niko may know it better, since he changed layouting lately. Sorry, looked at SVGAnimateTransformElement again. We're already calling relayouting. It looks like you're right with invalidateResourcesInAncestorChain(), but you also need to call registerFromResources(renderer), see SVGStyledElement::svgAttributeChanged. I still believe that this should be done in layout() on the renderer, but it's ok to handle it here for the moment.

This has been fixed. Possibly with the recent SMIL changes.