1042393 - Allow expiration and unloading of background-images

Status: NEW

(Core :: CSS Parsing and Computation, defect)

Description

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

Right now once a selector matches the image is loaded and remains in memory until the style sheet is destroyed.  This leads to wasting a lot of memory when background-image is used for seldom-used images in long-lived things, such as the b2g system app.

Comment 1

[Gregor Wagner [:gwagner]]

[Blocking Requested - why for this release]:
Blocking a blocker

Comment 2

[Lucas Adamski [:ladamski]]

Lets get an owner for this ASAP.  Is this a fix we can make in Gaia or do we need Gecko (DOM?) help here?

Comment 3

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

It needs style system work.  I'm on it.

Comment 4

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

I think we should unblock on this for two reasons.

1. The patch is relatively risky.  I have a proof of concept locally but it still has crashes that I don't understand yet.  I worry that this is going to be too risky for this point in the 2.0 cycle.
2. The upside of this is limited.  After using the phone for several hours the amount of compressed image data stored in memory is less than 1 MB.  We got rid of the biggest offenders in Bug 1039631.  Everything else is pretty tiny.

I don't think the risk we would be adding with this is worth less than 1 MB.

Comment 5

[Jason Smith [:jsmith]]

Unblocking per comment 4.

Comment 6

[Gregor Wagner [:gwagner]]

Kyle, are you still working on this?

Comment 7

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

Yes.  I have a patch but it has some issues, and I keep getting sidetracked by higher priority stuff.