795438 - Regression: Animated GIF animates only after a pan (invalidation) on certain devices

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Lawrence Mandel [:lmandel] (use needinfo)]

When viewing an animated GIF the animation only progresses when I pan the screen around. Simply loading the page is not enough to view the animation.

STR:

1. Load http://i.imgur.com/dZOXE.gif on Fennec. Note how a static image loads.
2. Pan the page up and down. Note how the GIF advances frames while panning but stops advancing when you stop panning.

Expected results:
The animated GIF should be animated.

Actual results:
A static image shows.

Comment 1

[Lawrence Mandel [:lmandel] (use needinfo)]

Perhaps I should add something about my platform.

Samsung Galaxy S2, ICS 4.0.4.
Nightly 18.0a1 2012-09-28

Comment 2

[Aaron Train [:aaronmt]]

I can't reproduce this on the Galaxy Nexus/7, same build (http://www.youtube.com/watch?v=LCd7ha_WC3Q)

Comment 3

[Lawrence Mandel [:lmandel] (use needinfo)]

Here's a video of the issue on my Galaxy S2.

http://youtu.be/4nkNLVGFgos

Comment 4

[Aaron Train [:aaronmt]]

Still an issue: http://i.imgur.com/s5knR.gif

--
Nightly (11/05)

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

tracking with the assumption that this is a regression in 19, if the regression range doesn't prove that out, please renom

Comment 6

[Aaron Train [:aaronmt]]

The original issue of the GIF not animating on all devices is WFM; the new issue here is that on some devices (such as the Galaxy Note II) (and possibly the SII and similar devices) is that the animated GIF will only animate after a redraw from a pan (invalidation issue). 

This is a regression from the progressive-tile work on trunk.

Comment 7

[Cristian Nicolae (:xti)]

The last Good build: 10/25

Bad build: 10/26

The exact regression-range is: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=8a6b6b5f3e55&tochange=044f1370c2b8

Comment 8

[Aaron Train [:aaronmt]]

Makes sense; thanks!

Comment 9

[Chris Lord [:cwiiis]]

Ah, this sounds like bad cancelling and will be fixed as part of the patch series in bug 783368. Thanks for the regression range :)

Comment 10

[Aaron Train [:aaronmt]]

Marking as Aurora/Nightly affected now.

Comment 11

[Aaron Train [:aaronmt]]

WFM (Note II), fixed on Nightly via bug 783368. Marking WFM.

Comment 12

[Lawrence Mandel [:lmandel] (use needinfo)]

I confirmed the fix on Galaxy S2. I changed the resolution status to FIXED as this was confirmed to be a reproducible problem.

Comment 13

[Cristian Nicolae (:xti)]

I cannot reproduce this issue on the latest Nightly build. Closing bug as verified fixed on:

Firefox 20.0a1 (2012-11-26)
Device: Galaxy S2
OS: Android 4.0.3

Comment 14

[Marco Castelluccio [:marco]]

I can reproduce this bug on a Galaxy S2. See also bug 858137.

Comment 15

[Kartikaya Gupta (email:[e-mail])]

We should keep this bug closed because attaching new patches to this one will get messy quickly, and it was verified fixed at some point. We can track the new re-regression in bug 858137.