Open
Bug 919113
Opened 11 years ago
Updated 2 years ago
Per-document in-memory image cache should really be per-document
Categories
(Core :: Graphics: ImageLib, defect)
Tracking
()
NEW
People
(Reporter: bzbarsky, Unassigned)
References
(Depends on 1 open bug)
Details
Right now if two documents alternate loading the same no-cache image URI, we'll keep revalidating it, because the aCX passed in won't match the key stored in the image cache entry. This apparently actually bites gmail composition when the user attaches an image: for some reason they have two documents in play that alternate trying to load the image. We should consider including the aCX in the cache lookup key instead or something.... Or storing in the cache a list or hashset of aCx values for which the image has been validated?
![]() |
Reporter | |
Comment 1•11 years ago
|
||
Seth, do you have time to deal with this, or know who does? I realize imagelib is a bit undermanned right now. :(
Flags: needinfo?(seth)
Comment 2•11 years ago
|
||
I do not expect to have time to deal with this for the next few weeks, although if some free time appears I'll keep it in mind. Timothy, would you be able to take a look at this more quickly?
Flags: needinfo?(seth) → needinfo?(tnikkel)
Comment 3•11 years ago
|
||
I don't think I'll be able to get to this in the short term either. :(
Flags: needinfo?(tnikkel)
![]() |
Reporter | |
Comment 4•11 years ago
|
||
Oh, one thing I forgot in comment 0: gmail is hitting this. If you're composing a message with an image in it, we hit the situation from comment 0 every time they auto-save the message, causing us to download the image several times per autosave. :(
Updated•2 years ago
|
Severity: normal → S3
Updated•2 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•