121927 - animated MNG images do not honor animated image pref

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Carl-Daniel Hailfinger]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.7+) Gecko/20020125
BuildID:    2002012505

If you go to above URL, you'll notice you can't stop the animated MNG with the
stop button. Preferences->Privacy&Security->Images-> Animated_Images_should_loop
settings don't have any effect either.
This applies to every animated MNG you can find.

Reproducible: Always
Steps to Reproduce:
1.Go to above URL
2.Watch the big image at the top loop forever
3.Try to stop it - you can't

Actual Results:  The image still loops.

Expected Results:  Looping should stop and also honor the Preferences for image
looping

This may be a regression as we now can't stop all animated images anymore.

Comment 1

[Jesse Ruderman]

The stop button also doesn't affect animated gifs (bug 70030).

Comment 2

[Carl-Daniel Hailfinger]

Yes, but MNG images also do *not* honor the preferences in Images->
Animated_Images_should_loop which is ok for GIFs.

Comment 3

[joel]

Is this bug EVER going to get fixed?  (should've been fixed a year ago)

Comment 4

[Henrik Gemal]

http://www.libpng.org/pub/mng/img_mng/pnglogo-grr-anim-lc.mng
setting the pref to "loop once" still loops the mng image forever. Either the 
code should be fixed or the pref should be changed to "gif animation"

Comment 5

[Stuart Parmenter]

-> tor

Comment 6

[Alfonso Martinez]

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

Comment 7

[jsoffer]

May it be related to Bug 103166 ?

Comment 8

[Alexander Opitz]

Attachment: patch against last patch of 18574

This patch is against patch 'Full MNG-1.0.6+ restoration to Mozilla-1.7a
(gzip)' from 2004-02-18 18:41 PST of bug 18574 . And will be included in the
next patch on 18574.

With the patch the animation pref will be honored and also some clean-ups or
'looks like gif container' included.

Will make a new binary tomorrow on mngzilla.

Comment 9

[Alexander Opitz]

Attachment: updated patch

The old patch had sometimes problems viewing all 'animated' MNG's. So here
comes a update. mngzilla packages are on compile.

Comment 10

[Alexander Opitz]

patch is in bug #18574 and will be checked in if MNG comes back :-)