Commit cf47f9ae authored by pdr's avatar pdr Committed by Commit bot

Update paint properties on animate motion transform updates

This patch updates SVGAnimateMotionElement so that changes to the
animate motion transform call setNeedsPaintPropertyUpdate. A followup
patch will unify the behavior of the setNeedsTransformUpdate() calls.

With this patch and --enable-blink-features=SlimmingPaintInvalidation,
new tests pass:
  svg/animations/animateMotion-fill-freeze.html
  svg/animations/animateMotion-fill-remove.html
  svg/animations/animateMotion_changingPath.html
+7 more

BUG=645667

Review-Url: https://codereview.chromium.org/2577993003
Cr-Commit-Position: refs/heads/master@{#438915}
parent 7ea0ee47
...@@ -299,6 +299,10 @@ void SVGAnimateMotionElement::updateAnimationMode() { ...@@ -299,6 +299,10 @@ void SVGAnimateMotionElement::updateAnimationMode() {
void SVGAnimateMotionElement::invalidateForAnimateMotionTransformChange( void SVGAnimateMotionElement::invalidateForAnimateMotionTransformChange(
LayoutObject& object) { LayoutObject& object) {
object.setNeedsTransformUpdate(); object.setNeedsTransformUpdate();
if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled()) {
// The transform paint property relies on the SVG transform value.
object.setNeedsPaintPropertyUpdate();
}
markForLayoutAndParentResourceInvalidation(&object); markForLayoutAndParentResourceInvalidation(&object);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment