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)

All
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: cathleennscp, Assigned: darin.moz)

References

Details

(Whiteboard: 3 days)

Attachments

(1 file)

Summary: not storing large images twice in cache → [minimo] not storing large images twice in cache
QA Contact: tever → cacheqa
Blocks: 215636
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...
Summary: [minimo] not storing large images twice in cache → [minimo] not store large images in http cache and decoded image cache
Whiteboard: 3 days
Assignee: gordon → darin
moving minimo bugs to the new bugzilla product.
Component: Networking: Cache → General
Product: Browser → Minimo
Version: Trunk → unspecified
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
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.
Attached patch v1 patch (deleted) — Splinter Review
here's the trivial patch i mentioned.
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.

Attachment

General

Creator:
Created:
Updated:
Size: