1113083 - SVG/SMIL: using svg as background-image pauses animation on pageload.

Status: RESOLVED DUPLICATE of bug 935888

(Core :: Graphics: ImageLib, defect)

Description

[pepijnfens]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:34.0) Gecko/20100101 Firefox/34.0
Build ID: 20141126041045

Steps to reproduce:

Create a file with a <div> with an svg file (animated in smil) as a background-image.

To create a bottomline, I used sam herberts SVG loaders and changed the html file to include the loaders as background-image.
http://samherbert.net/svg-loaders/



Actual results:

When opening the page in firefox this animation will appear to be paused.
After a refresh, the animation does start.

I assume this has to do with caching, only triggering the animation to start when it is already loaded?


Expected results:

The animation should start right away, or at least at the moment the .svg resource file is completely loaded. Now, it needs a refresh before it actually starts.

Comment 1

[Loic]

Could you attach a testcase, please.

Comment 2

[pepijnfens]

Attachment: Test case showing the problem with SVG/SMIL in a background-image

The svg file is taken from sam herberts excellent svg loaders, I've copied it to my dropbox to make sure it's available when you open the html locally.

Comment 3

[Loic]

I think it's a dupe of bug 908634.

Comment 4

[Loic]

Not the same regression range in fact.

good=2013-09-04
bad=2013-09-05
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=e3785e299ab6&tochange=77ed46bf4c1a

Suspected bug:
Jonathan Watt — Bug 907503 - Fix invalidation for SMIL animation in SVG-as-an-image. r=roc

Comment 5

[Jonathan Watt [:jwatt]]

( So caused by the first half of http://hg.mozilla.org/mozilla-central/rev/8d8c2ce395a1 )

Comment 6

[Jonathan Watt [:jwatt]]

Two things I'd note after looking at this a bit:

 * backing out bug 907503 doesn't fix this (maybe because of bug 908634?)
 * the animation only fails on initial load or hard reloads (soft reloads get it working)

Comment 7

[Alice0775 White]

In local build
Last Good: e1976a0ce384
First Bad: f12a3d9e4cd6

Triggered by:	f12a3d9e4cd6	Jonathan Watt — Bug 911862 - Make VectorImage use the refresh driver to send out frame change notifications. r=dholbert

Comment 8

[Ciprian Muresan [:cmuresan], Formerly of Ecosystem QA - please needinfo @mcoman for things regarding the team]

Hi,

User Agent 	Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:49.0) Gecko/20100101 Firefox/49.0

The issue is still reproducible in the latest Firefox release (46.0, Build ID 20160421124000) and the latest Nightly (49.0a1, Build ID 20160427030215). After running mozregression I found the same range as Loic in comment 4.

Thanks,
Cipri

Comment 9

[Robert Longson [:longsonr]]

pretty sure the landing of bug 935888 has fixed this.

Comment 10

[Loic]

Yes, I verified, we can even dupe it.