908634 - CSS-animated SVG is not animated anymore as background-image, regression

Status: RESOLVED DUPLICATE of bug 1190881

(Core :: SVG, defect)

Description

[Stanislas Signoud]

Attachment: Example: animated spinner.svg

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.

Comment 1

[Stanislas Signoud]

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

Comment 2

[Alice0775 White]

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

Comment 3

[Daniel Holbert [:dholbert]]

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.

Comment 4

[Jonathan Watt [:jwatt]]

I believe this was fixed by longsonr's patch in bug 1067375. Robert?

Comment 5

[Daniel Holbert [:dholbert]]

Doesn't look like it. That bug is fixed in Nightly (I verified w/ its testcase), whereas this bug still affects Nightly.

Comment 6

[Robert Longson [:longsonr]]

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.

Comment 7

[Robert Longson [:longsonr]]

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.

Comment 8

[Robert Longson [:longsonr]]

Either bug 975757 (or possibly bug 1090936) does fix this now.

Comment 10

[Philip Rogers]

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.

Comment 11

[tomati]

This Bug is still there.

Comment 12

[Ghetolay]

This is totally my case, I had svg uanimation sing SMIL but since webkit started complaining about SMIL I change my animations to use CSS. And now I ended up on this bug :)
I Have to use 2 svg (one using SMIL the other CSS) coupled with special rule to detect browser and apply the good svg.

Comment 13

[u459114]

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