[webkit-reviews] review requested: [Bug 5499] Page reload does not send any cache control headers : [Attachment 8891] proposed fix

[bugzilla-request-daemon at opendarwin.org]

Alexey Proskuryakov <ap at nypop.com> has asked  for review:
Bug 5499: Page reload does not send any cache control headers
http://bugzilla.opendarwin.org/show_bug.cgi?id=5499

Attachment 8891: proposed fix
http://bugzilla.opendarwin.org/attachment.cgi?id=8891&action=edit

------- Additional Comments from Alexey Proskuryakov <ap at nypop.com>
This fixes both manual reload and window.location.reload; not sure if there are
other ways to cause reload that should send cache control headers
(window.location.assign to the same URL doesn't do that in Firefox).

Unfortunately, there doesn't seem to be a way to send If-Modified-Since
together with max-age=0 (in other words, to validate freshness) with
NSURLRequest, rdar://4590785. This doesn't affect correctness, but wastes
bandwidth.

The TransferJob-related code in WebCore::Frame seemed to be unused, other than
for passing response HTTP headers.