Bug 66020

Summary: Load each instance of a cached resource if the cache policy says so
Product: WebKit Reporter: James Simonsen <simonjam>
Component: New BugsAssignee: James Simonsen <simonjam>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: abarth, ap, japhet, koivisto
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

James Simonsen
Reported 2011-08-10 16:24:00 PDT
Load each instance of a cached resource if the cache policy says so
Attachments
Patch (8.25 KB, patch)
2011-08-10 16:24 PDT, James Simonsen 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
James Simonsen
Comment 1 2011-08-10 16:24:24 PDT
Created attachment 103553 [details] Patch
James Simonsen
Comment 2 2011-08-10 16:33:17 PDT
There seems to be a new policy added in change 74807. It's causing problems with Google Docs as reported here: http://code.google.com/p/chromium/issues/detail?id=86695 I couldn't find anything in either the HTML or HTTP spec that says we can skip reloading instances of the same resource on the same page in this case. So it seems we should probably take this out.
Alexey Proskuryakov
Comment 3 2011-08-10 18:27:04 PDT
The bug report seems rather vague. What exactly does cache policy say? For instance, what are the differences with bug 52044? Is this a duplicate of bug 52153? I'm glad that cache-simulator.cgi was useful for this test, but it also hides some of the details, which should probably be spelled out in tests themselves. Just add a <p> explaining what exactly is being tested, and mention it in ChangeLog that the behavior matches IE and Firefox, if it does.
James Simonsen
Comment 4 2011-08-10 18:37:15 PDT
(In reply to comment #3) > Is this a duplicate of bug 52153? Indeed. Glad that you noticed that. > > I'm glad that cache-simulator.cgi was useful for this test, but it also hides some of the details, which should probably be spelled out in tests themselves. Just add a <p> explaining what exactly is being tested, and mention it in ChangeLog that the behavior matches IE and Firefox, if it does. Thanks. I'll fix that in a patch for the other bug. I'll also update my patch to only revalidate after the initial load completes, instead of all the time. *** This bug has been marked as a duplicate of bug 52153 ***
Note You need to log in before you can comment on or make changes to this bug.