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

NEW
Unassigned

Status

()

defect
8 years ago
a year ago

People

(Reporter: b1437400, Unassigned)

Tracking

(Blocks 1 bug)

9 Branch
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [MemShrink:P2] DUPEME)

(Reporter)

Description

8 years ago
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
(Reporter)

Updated

8 years ago
Summary: Too much memory consumption on the page with a lot of images → On the pages with a lot of images firefox consumes trice more memory then expected
Status: UNCONFIRMED → NEW
Component: General → ImageLib
Ever confirmed: true
Product: Firefox → Core
QA Contact: general → imagelib
Summary: On the pages with a lot of images firefox consumes trice more memory then expected → Firefox uses 3-4x times as much memory as Chrome on animated image test case
Presumably a duplicate of the bug on us decoding all frames of an animated gif....
Whiteboard: [MemShrink] → [MemShrink] DUPEME
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.
(Reporter)

Comment 3

8 years ago
I've added bug 666446 as a blocking of this one, I don't understand if they are the same or not.
Depends on: 666446
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).
Whiteboard: [MemShrink] DUPEME → [MemShrink:P2] DUPEME
Apparently bug 595671 will allow us to discard images.
[Clearing dependency on bug 666446, since that bug's not intended to help with memory usage as khuey suggested in comment 4]
No longer depends on: 666446
You need to log in before you can comment on or make changes to this bug.