Bug 66020 – Load each instance of a cached resource if the cache policy says so

• Component: New Bugs
• Status: RESOLVED DUPLICATE

[James Simonsen]

Load each instance of a cached resource if the cache policy says so

[James Simonsen]

Created attachment 103553 [details]
Patch

[James Simonsen]

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]

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]

(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 ***