682638 - Firefox uses 3-4x times as much memory as Chrome on animated image test case

Status: RESOLVED INCOMPLETE

(Core :: Graphics: ImageLib, defect)

Description

[Joe Wilson]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0a1) Gecko/20110827 Firefox/9.0a1
Build ID: 20110827030801

Steps to reproduce:

1. Download the archive http://dl.dropbox.com/u/11652751/1.rar - it containts a pure html page with a ton of gifs.
It's about 135mb in the packed state
2. Unrar the archive
3. Open the page inside of it
4. Watch memory consumption by firefox process


Actual results:

+650mb instantly increased RAM consumption


Expected results:

Chrome eats +170mb in that case

Comment 1

[Boris Zbarsky [:bzbarsky]]

Presumably a duplicate of the bug on us decoding all frames of an animated gif....

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Yeah, that's probably part of it.  Chrome is also using less CPU than us, which makes me think that they're only animating the GIFs in view, but I didn't investigate much.

Comment 3

[Joe Wilson]

I've added bug 666446 as a blocking of this one, I don't understand if they are the same or not.

Comment 4

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

That bug will help with the CPU usage here.  It may or may not help with the memory usage (I didn't read the patches in detail).

Comment 5

[Jeff Muizelaar [:jrmuizel]]

Apparently bug 595671 will allow us to discard images.

Comment 6

[Daniel Holbert [:dholbert]]

[Clearing dependency on bug 666446, since that bug's not intended to help with memory usage as khuey suggested in comment 4]

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

The severity field for this bug is relatively low, S3. However, the bug has 17 votes.
:aosmond, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Comment 8

[BugBot (nomail) [:suhaib / :marco/ :calixte]]

The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.

Comment 9

[Daniel Holbert [:dholbert]]

URL in comment 0 is 404 (and reporter's bugzilla account seems to be disabled/inactive), so this isn't directly reproducible with the original STR at this point.

Given comment 5 had a theory about a helper bug which would address this ("Apparently bug 595671 will allow us to discard images"), and also given the decade that has passed, there's reason to believe this may no longer be an issue.

Let's call this INCOMPLETE; we can file a new bug if there are still measurable/concerning memory usage differences with this sort of comparison.