Closed Bug 908634 Opened 7 years ago Closed 4 years ago
CSS-animated SVG is not animated anymore as background-image, regression
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:23.0) Gecko/20100101 Firefox/23.0 (Beta/Release) Build ID: 20130817191046 Steps to reproduce: We replaced our GIF spinners with SVG animated ones, using embedded CSS animation; they are rendered using background-image. 1. Create an animated SVG file 2. Save it an asset (spinner.svg, here) 3. Link it in a CSS file as a background-image JSFiddle: http://jsfiddle.net/zBudu/ (SVG file is attached) Actual results: Since Firefox Aurora (Gecko 25), only the first animation frame is rendered. Expected results: In Firefox Stable (and in Chrome Stable), all frames are correctly rendered and animation occured.
Please note that it doesn't happen with Firefox 23 but with Firefox 25: User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:25.0) Gecko/20100101 Firefox/25.0
Regression window(m-i) Good: http://hg.mozilla.org/integration/mozilla-inbound/rev/243533ca7769 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20130524 Firefox/24.0 ID:20130524211144 Bad: http://hg.mozilla.org/integration/mozilla-inbound/rev/d1b9087dd689 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20130525 Firefox/24.0 ID:20130525045945 Pushlog: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=243533ca7769&tochange=d1b9087dd689 Regressed by: 297eccee61b5 Jonathan Watt — Bug 854765 - Get rid of the DidSetStyleContext overrides in SVG frame code to avoid some major perf hits. r=roc
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
Component: Layout: Images → SVG
OS: Linux → All
Resolution: --- → DUPLICATE
Version: 23 Branch → 24 Branch
Duplicate of bug: 907503
As pointed out in bug 907503 comment 49, this bug's original testcase ( http://jsfiddle.net/zBudu/ ) is not fixed, so let's un-dupe & reopen this bug.
Status: RESOLVED → REOPENED
Ever confirmed: true
Resolution: DUPLICATE → ---
I believe this was fixed by longsonr's patch in bug 1067375. Robert?
Doesn't look like it. That bug is fixed in Nightly (I verified w/ its testcase), whereas this bug still affects Nightly.
No it's not fixed by anything I've landed yet. I think it might be fixed by bug 975757 though depending on how that turns out.
bug 975757 won't fix this. Presumably some style changes need an nsSVGEffects:InvalidateRenderingObservers to be called. bug 975757 will add a change hint for this.
Would it be possible to re-evaluate the priority of this bug? I just discovered this when preparing a demo for the SVGWG. Blink recently announced an intent to move away from SMIL in favor of CSS animations for svg-in-image. If it pans out (even best-case this wouldn't land for several months), this leaves developers in a tough spot: Blink/Edge/WebKit would support CSS animations in SVG images. Gecko/WebKit would support SMIL animations in SVG images.
The root cause of this bug is the same with bug 1190881. We did not give any nsDOMNavigationTiming to the SVGDocument inside SVGDocumentWrapper. I will dup this one after fix bug 1190881
Status: REOPENED → RESOLVED
Closed: 7 years ago → 4 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1190881
You need to log in before you can comment on or make changes to this bug.