[webkit-reviews] review requested: [Bug 41813] XHRs get unnecessary Cache-Control http header after reload : [Attachment 61100] patch #2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jul 9 14:41:33 PDT 2010


Nate Chapin <japhet at chromium.org> has asked  for review:
Bug 41813: XHRs get unnecessary Cache-Control http header after reload
https://bugs.webkit.org/show_bug.cgi?id=41813

Attachment 61100: patch #2
https://bugs.webkit.org/attachment.cgi?id=61100&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
This change merges the places that are setting cache policy based on
DocumentLoader's original request into addExtraFieldsToRequest.  As it
currently stands, those settings often get overridden anyway, and it makes it
much harder to follow the logic of the cache policy.  Hopefully this is
clearer.


More information about the webkit-reviews mailing list