Closed
Bug 199323
Opened 22 years ago
Closed 21 years ago
[minimo] not store large images in http cache and decoded image cache
Categories
(Minimo Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cathleennscp, Assigned: darin.moz)
References
Details
(Whiteboard: 3 days)
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Summary: not storing large images twice in cache → [minimo] not storing large images twice in cache
Comment 1•21 years ago
|
||
right now this has been sort of fixed for minimo by turning off http caching.
so fixing this now would allow turning back on http caching, but not for
images... the images would be stored in the image cache only...
Updated•21 years ago
|
Summary: [minimo] not storing large images twice in cache → [minimo] not store large images in http cache and decoded image cache
Updated•21 years ago
|
Whiteboard: 3 days
Updated•21 years ago
|
Assignee: gordon → darin
Comment 2•21 years ago
|
||
moving minimo bugs to the new bugzilla product.
Component: Networking: Cache → General
Product: Browser → Minimo
Version: Trunk → unspecified
Assignee | ||
Comment 3•21 years ago
|
||
solving this bug correctly depends on being able to use the INHIBIT_CACHING load
flag. once bug 232385 is fixed, we should be able to solve this bug by simply
making imgLoader::LoadImage set the INHIBIT_CACHING load flag on the nsIChannel
it creates.
Depends on: 232385
Assignee | ||
Comment 4•21 years ago
|
||
hmm... my proposed solution would actually cause images to be cached
indefinitely (i.e., until displaced by other content in the cache _or_ until
replaced by an explicit page reload). this means that my proposed change could
break pages that depend on being able to limit the amount of time a browser
caches images.
to solve this problem properly, we'd probably still need to create a HTTP cache
entry containing just the meta data for the images. (the reason is that only
the HTTP code knows how to properly calculate the "freshness lifetime" of a
cached document.)
this same problem (obviously) exists with the solution of disabling all HTTP
caching.
Assignee | ||
Comment 5•21 years ago
|
||
here's the trivial patch i mentioned.
Comment 6•21 years ago
|
||
Minimo milestone 3 uses the http caching turned off solution. In milestone 4 we
could explore more opitmizations.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•