Closed Bug 651234 Opened 13 years ago Closed 13 years ago

Still caching items greater than 5 MB even when Content-Length present

Categories

(Core :: Networking: Cache, defect)

Product:
Core
Component:
Networking: Cache
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: jduell.mcbugs, Unassigned)

References

(Blocks 1 open bug)

Details

Bug 651100 (which has a hang that may or may not be related to this) has a test YouTube URL that is apparently getting cached (evicting things) despite having a Content-Length header > 5MB. 

Bug 588507 was supposed to make that not happen.
Note that if the video for whatever reason is marked with the flag nsICache::STORE_ON_DISK_AS_FILE then the check for > 5Mb is skipped.
That's probably what's happening.  We call SetCacheAsFile(PR_TRUE) here: http://mxr.mozilla.org/mozilla-central/source/content/media/nsMediaStream.cpp#262
So then this isn't a bug...
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.