New issue
Advanced search Search tips

Issue 664257 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

SMIL animations on descendants of a display:none SVG should not cause style recalculations

Project Member Reported by erikc...@chromium.org, Nov 10 2016

Issue description

In fact, no box should be emitted at all, so the animation should have no effect.

See https://docs.google.com/document/d/1YVO9FGRBF1RUsedU28H6P8lt9bDpOci0vi0CLSkFih8/edit# for a more details of the investigation. See section on slideshare.net/patrickmeenan.
 
This is not the same issue as https://bugs.chromium.org/p/chromium/issues/detail?id=650433, since this is a display: None block inside of an iframe.
 
Components: Blink>Animation
Labels: -OS-Mac Performance Update-Quarterly OS-All

Comment 2 by benhenry@google.com, Jul 14 2017

Labels: -Performance Performance-Responsiveness
Cc: smcgruer@chromium.org erikc...@chromium.org
Components: -Blink>Animation Blink>SVG
Owner: ----
Status: Available (was: Assigned)
Summary: SMIL animations on descendants of a display:none SVG should not cause style recalculations (was: Infinite CSS animations on descendants of a display:None SVG in an iframe should not cause stye recalculations)
Ok so after trying a few things I don't think this relates to iframes, but appears more to be about SVGs, SVG animations, and display: none. The animations in the repro page are SMIL, *not* CSS animations.

Taking devtools performance captures from:

https://oasis-poppy.glitch.me/with-animation.html
https://oasis-poppy.glitch.me/without-animation.html
https://oasis-poppy.glitch.me/css-with-animation.html
https://oasis-poppy.glitch.me/svg-with-css-animation.html

The latter three all produce no main frames during the capture, but the with-animation case regularly runs style+layout. So it seems specific to SMIL animations.

Removing from erikchen's queue as he is not active here. Also based on a quick crbug search SMIL animations are generally owned by Blink>SVG, so changing component
It might be that this is deliberate. SVG content behind display:none is still available for use with <use> and <pattern> and other content.
I don't the <use> necessarily has anything to do with it - what you see is the timeline running and updating the 'transform' attribute on the <rect> (which marks style dirty because it's a presentation attribute).

Sign in to add a comment