175509 - Animated GIF stops working after reload

Status: RESOLVED WORKSFORME

(Core :: Graphics: ImageLib, defect)

Description

[Frank Wein [:mcsmurf]]

First, I think this worked some time ago, so this must be an regression bug.

When I go to
http://forum.counter-strike.de/cgi-bin/bbs/forumdisplay.cgi?action=topics&number=8&SUBMIT=Go
(notice some animated gifs on the left), then click on any link on this site,
press the back button and then click on the reload button. The animated gifs
stopped working (they dont animate anymore). But this Bug could also be related
to the cache manager in some way, because if you now reload the page again (but
with shift-key holded for a complete reload) the animated gifs work again.

Comment 1

[mwehner]

WFM on Linux 2002101701

Comment 2

[Frank Wein [:mcsmurf]]

Ah, I forgot something important: My buildid is Mozilla/5.0 (Windows; U; Windows
NT 5.0; en-US; rv:1.2b) Gecko/20021016

Comment 3

[Christian :Biesinger (don't email me, ping me on IRC)]

I'd think that bug 152756 should've fixed this.
cc'ing paper.

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

hm, can reproduce - cvs build from yesterday or so, linux.

Comment 5

[Kevin Chen]

Also see bug 156826, which is duplicate of bug 69405...

*** This bug has been marked as a duplicate of 69405 ***

Comment 6

[Frank Wein [:mcsmurf]]

This is _not_ a dupe of 69405, this isn't a feature request or enhancement, this
is a real bug.

Comment 7

[Frank Wein [:mcsmurf]]

Paper: When i reload the page with the shift-key and proceed the steps again
(clicking on a link, going back,...) and the animated GIFs stoped again working.

Comment 8

[R.K.Aa.]

*** Bug 177253 has been marked as a duplicate of this bug. ***

Comment 9

[Frank Wein [:mcsmurf]]

Not really a dupe, but thats right (probably in Bug  119597 there would be a
much better bug to dupe it to)

Comment 10

[Frank Wein [:mcsmurf]]

Update: Now it seems even more broken with Mozilla/5.0 (Windows; U; Windows NT
5.0; en-US; rv:1.3a) Gecko/20021113
When you now press the back button, the animated GIF loops once and then stops.
Reloading doesn't change anything (as in Comment 0).

Comment 11

[Frank Wein [:mcsmurf]]

Resolving Wfm again, because
1. the testcase is gone
2. with my own testcase i can't reproduce this anymore (look at URL)

Comment 12

[Frank Wein [:mcsmurf]]

Reopening, seeing this again, see URL and try steps from Comment 0 (look at
animated gifs, select an article, click back, reload or not=>animated GIFs
stopped working)

Comment 13

[R.K.Aa.]

*** Bug 198050 has been marked as a duplicate of this bug. ***

Comment 14

[Frank Wein [:mcsmurf]]

Also see http://tinderbox.mozilla.org/showbuilds.cgi?tree=SeaMonkey when the
tree is burning.

Comment 15

[Frank Wein [:mcsmurf]]

Also see http://tinderbox.mozilla.org/showbuilds.cgi?tree=SeaMonkey;when the
tree is burning you can take this as a testcase.

Comment 16

[Simon Fraser [no longer active]]

Seems pretty serious. Regression from the image/content landing?

Comment 17

[Boris Zbarsky [:bzbarsky]]

The first comment on this bug that is _after_ the landing is comment 14.  So no.
 And I can't reproduce this in a current-ish build in any case; could someone
who was seeing this please retest?

Comment 18

[Simon Fraser [no longer active]]

I see it in a 20030421.

Comment 19

[Boris Zbarsky [:bzbarsky]]

Hmm.. So I don't see it in 2003-04-12-22, but I do see it in my current CVS build.

I can't run any nightlies newer than 2003-04-12-22, so if someone could
determine when this broke in the last week and a half, that would be nice.

Comment 20

[Frank Wein [:mcsmurf]]

Bug is still present in cvs trunk builds from today and various other builds
(1.4b on Mac and 1.4b on Linux)

Comment 21

[Christian Mattar]

*** This bug has been marked as a duplicate of 129986 ***

Comment 22

[Frank Wein [:mcsmurf]]

no dupe, the testcases here loops the animated gif , so no need for a reset.

Comment 23

[Boris Zbarsky [:bzbarsky]]

What's a URL to reproduce the bug at?  I could narrow down the regression window
now, but the urls in comment 0 and the URL field don't seem to have animated
images, and relying on a red tinderbox makes for difficulty reproducing...

Comment 24

[Frank Wein [:mcsmurf]]

why have the testcases change so often :/
so see URLs
http://drload.de/forum/viewforum.php?forum=1&19929
http://forums.mozillazine.org/viewforum.php?f=22
http://yog71c59ef9c3963f645c8965937911efbb.web2.onlineglueck.de/forum/thread-10741.html
(click some link on the right side)

anyway i can't reproduce this problem anymore, biesi, sfraser how about you
(biesi fell free to remove you from this bug again)?

Comment 25

[Darryl C.]

Here's a sample of this bug using javascript for rollover/out swapping (script
forced reloading)

http://flatlinesystems.net/~dclarke/tests/ani-gif.html

confirmed w/
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5b) Gecko/20030916

Comment 26

[Nick]

this is not fixed on win2k either. there seem to be a lot of versions of this
bug but this one seems covers all os's. anyway, it has slightly improved. In
Firefox [Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6) Gecko/20040206
Firefox/0.8], the image still hangs on reload but if you request (GET) another
page and then follow a link back to the original page (not the back button), the
image will reset.

The following page demonstrates this. Look in the right-hand box titled "Recent
Cytek News". If the headline is bold then the bullet next to it should be
animated. If you reload, the gif's current frame will be permanent when the page
loads. This page validates XHTML Strict and CSS2: http://www.cytekdev.com/cd/

The reason I say it's improved is because in [Mozilla/5.0 (Windows; U; Windows
NT 5.0; en-US; rv:1.4) Gecko/20030624] the only way to reset the image was
shift+reload.

Comment 27

[Boris Zbarsky [:bzbarsky]]

Testing this bug with any builds prior to 1.7a is pointless at this point, since
the patch to bug 210001 changed around how image animations are handled.

The latest testcase works for me with a current build as far as I can tell.

Comment 28

[Frank Wein [:mcsmurf]]

yes for me too so resolving again (i hope i don't have to reopen it again
anytime soon) :)