835041 - SMIL animation is jerky on Firefox OS browser at play.esviji.com, when a ball is destroyed

Status: RESOLVED INCOMPLETE

(Core :: SVG, defect)

Description

[Julien Wajsberg [:julienw]]

STR:
* go to http://play.esviji.com and start a game
* trigger the "ball destroying" animation

Expected:
* animation is smooth

Actual:
* animation is jerky

There is at the same time the following animations:
* rotation
* opacity change

This is done on a element with gradient on top of another element with a gradient;

Asking for a smaller testcase from Nicolas.

Comment 1

[Daniel Holbert [:dholbert]]

The animation looks smooth to me, FWIW. (though my machine's pretty fast)
Mozilla/5.0 (X11; Linux x86_64; rv:21.0) Gecko/20130129 Firefox/21.0

Comment 2

[Daniel Holbert [:dholbert]]

Oh, sorry -- didn't notice that this was filed for Firefox OS.

I can repro the jankiness on my unagi, w/ latest dogfood build.

Comment 3

[Nicolas Hoizey]

Gradients both in the background and in the balls have been removed now on http://play.esviji.com/ because of performance issues almost everywhere on mobile phones.

Comment 4

[Jonathan Watt [:jwatt]]

For B2G builds made from gecko v23 or greater, you should be able to move the balls using a transform (instead of x/y) and the rendering will be cached in an offscreen surface allowing it to be moved around much more easily. That should allow you to add the gradients back.

Can a version of the game with gradients restored be made available somewhere so that I can confirm this and, if necessary, look into any remaining issues?

Comment 5

[Jonathan Watt [:jwatt]]

Oh, and changed to use transform instead of x/y.

Comment 6

[Jonathan Watt [:jwatt]]

Julien, can you provide us with a version with gradients again?

Comment 7

[Anthony Ricaud (:rik)]

Let's ask the creator of the game directly.

Comment 8

[Nicolas Hoizey]

This commit removed the gradients, so you should be able to try the game just before it:
https://github.com/nhoizey/esviji/commit/c756e172d842f386a5ea1201f14ebdffb430f2cf

Comment 9

[Julien Wajsberg [:julienw]]

Here is a git clone at the commit just before c756e172d842f386a5ea1201f14ebdffb430f2cf:
http://everlong.org/mozilla/esviji-gradients/

Nicolas, can you please confirm this is a version with gradients ? :) (I think it is)

Comment 10

[Nicolas Hoizey]

Julien, it is indeed.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

Does this still reproduce?

Comment 12

[Julien Wajsberg [:julienw]]

I think so. Not this is not the only jerky thing but I'd like to focus on this issue only. A smaller testcase could be useful...

Comment 13

[Jonathan Watt [:jwatt]]

Closing this given B2G is no more, but I can still repro perf issues on Mac so I spun off bug 1348787 to investigate further.