SetCacheAsFile on http protocol returns successful even when talking to https. This function must return a failure code if cacheing as a file is illegal. The three requirements that http should enforce before return success are (a) a disk cache present, (b) not using https, and (c) there are no response headers that ask for nonpersistance.
Severity: normal → blocker
Status: NEW → ASSIGNED
OS: Windows 2000 → All
Priority: -- → P1
Hardware: PC → All
Target Milestone: --- → mozilla0.9.2
Darin, this fixes most of the problem for me. As so long as this caching flag is being set in the right places, this should work. It works for the https case which is my blocker. Can I get you to sr this patch as is?
r=gagan. asa can we get some a= love here?
a=blizzard on behalf of drivers for the trunk
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
thanks. fixed checked in: Checking in nsHttpChannel.cpp; /cvsroot/mozilla/netwerk/protocol/http/src/nsHttpChannel.cpp,v <-- nsHttpChannel.cpp new revision: 1.24; previous revision: 1.23 done
Status: RESOLVED → VERIFIED
QA Contact: benc → junruh
You need to log in before you can comment on or make changes to this bug.