Closed
Bug 66858
Opened 24 years ago
Closed 24 years ago
[RFE] cached style sheet not used from disk cache
Categories
(Core :: Networking: Cache, enhancement)
Tracking
()
People
(Reporter: bugs4hj, Assigned: gordon)
Details
If a site uses linked Style Sheet, they pop up in disk cache. But after this the
style sheets are always, at least for WinNT4, fetched from the netwerk (server).
The correct behavour would be to use the cached style sheets, and not to force a
reload.
Comment 1•24 years ago
|
||
Fixed spelling error in Summary (cashed -> cached)
Summary: [RFE] cashed style sheet not used from disk cache → [RFE] cached style sheet not used from disk cache
Comment 2•24 years ago
|
||
I think it is dup of #29370
Thank you, high five. And thank you Mirek, but this bug is about disk cache.
Reading the bug writer guidelines, please use only one issue per bug report.
Also in bug report # 66803 Gordon told me: "The memory cache is currently only
used by IMAP". But maybe someone took out Memory support to 'hack around' some
of his problems!?
And do have any idea what the performance gain is if you don't have to reload
_all_ linked Style Sheets? Well I do, but then, how many people do you think are
using a protocol analyzer?
The fact that someone is inserting cache header information brakes HTTP 1.1 (RFC
2116) compliance, and that's seems very important to me! Something I did not see
in that other bug report!
Comment 5•24 years ago
|
||
reporter: can you verify that the cached CSS items have not expired? thx!
No timeout, and I did see a line with something like FORCERELOAD in the source,
that won't help I guess!?
Ok cheched, Prage and Cache-control are set to nocache. And in fact if a
document has expired it should perform a GET not-changed-since request and the
server should reply with a 304, not modified response, right?
Comment 8•24 years ago
|
||
marking bug INVALID since cached HTTP headers indicate that server expects
client to do a fresh GET on each request for the URL.
Status: UNCONFIRMED → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
Darin tell me something, this counts for Mozilla only, why? Microsoft Internet
Explorer, Opera and Amaya don't have this in the header. It's the same server,
it are the same documents. They all use HTTP 1.1 so what's the difference then?
The first GET don't have Pragma and Cache-Control set to nocache. So who put
that in the header? And this seems to count for ALL servers, for ALL Javascript
and ALL CSS files! But this doesn't count for HTML and images!
I really don't buy this, this is clearly a bug in Mozilla because it's clearly a
Mozilla bug.
Reporter | ||
Comment 10•24 years ago
|
||
This is interesting:
nsCSSLoader.cpp
1224 if (isLocalFile)
1225 loadAttributes = (nsLoadFlags)nsIChannel::LOAD_NORMAL;
1226 else
1227 loadAttributes = (nsLoadFlags)nsIChannel::FORCE_RELOAD;
1228 result = NS_NewStreamLoader(&loader, urlClone, aData, nsnull, loadGroup,
1229 nsnull, loadAttributes);
The ball is yours.
Comment 11•24 years ago
|
||
Anyone know why we FORCE_RELOAD online CSS content?
Comment 12•24 years ago
|
||
CVS log for nsCSSLoader.cpp, rev. 3.36:
3.36 pierre%netscape.com Mar 8 2000 29370 "Stylesheets and JS files are not
always loaded". Checked in for valeski, r=pierre, a=jar.
see bug 29370
Comment 13•24 years ago
|
||
Ok, looks like valeski's patch for bug 29370 was a work around because of
problems with the necko cache. Reopening, so we can fix this when the new
cache lands.
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Comment 15•24 years ago
|
||
indeed a workaround.
Reporter | ||
Comment 16•24 years ago
|
||
Has network speed to do with this path? If so, can this work on a SAT connection
without the path? I use an ASTRA VPN SAT connection, where I can use Mozilla on.
The average speed on this SAT is about 8Gb/sec per sec, but I can easily switch
it up to 19 Gb/sec, as I do for downloads. This SAT link is for Europe only, but
we do have an international SAT link also, up to 38Gb/sec. I use this link for
video conferences with the US and I receive file updates via this link.
Reporter | ||
Comment 17•24 years ago
|
||
BTW, "HTTP 1.1 (RFC 2116)" should read "HTTP 1.1 (RFC 2616)" typo.
Comment 18•24 years ago
|
||
Shouldn't this be marked as a duplicate of bug 29370 or at least changed from
RFE/enhancement? This severely impacts the usefulness of CSS. Taking a reloading
hit each time is a bug.
Comment 19•24 years ago
|
||
Status: UNCONFIRMED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → DUPLICATE
Comment 20•24 years ago
|
||
Darin: Are you sure? That other bug is for the memory cache, no?
Comment 21•24 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•