We should respect Cache-control: no-cache, no-store, max-age=0 in the request too. https://tools.ietf.org/html/rfc7234#section-5.2.1
rdar://problem/19714040
Created attachment 249543 [details] patch
Attachment 249543 [details] did not pass style-queue: ERROR: Source/WebKit2/NetworkProcess/cache/NetworkCache.cpp:188: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] Total errors found: 1 in 13 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 249543 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=249543&action=review r=me > Source/WebCore/loader/cache/CacheValidation.cpp:155 > + // See RFC 2616, Section 2.2 We may want to point to the new RFC instead of the deprecated one since we're moving this code :) > Source/WebCore/loader/cache/CacheValidation.cpp:182 > +static bool isControlCharacter(UChar c) We could probably mark this one as inline. > Source/WebCore/loader/cache/CacheValidation.cpp:256 > + // RFC2616 14.9.1: A no-cache directive with a value is only meaningful for proxy caches. Ditto about outdated RFC. > Source/WebKit2/NetworkProcess/cache/NetworkCache.cpp:186 > + auto requestCacheControl = WebCore::parseCacheControlDirectives(request.httpHeaderFields()); I feel that this variable should have 'request' somewhere in the name to avoid confusion with the response ones. > Source/WebKit2/NetworkProcess/cache/NetworkCache.cpp:285 > + auto cacheControlDirectives = WebCore::parseCacheControlDirectives(originalRequest.httpHeaderFields()); I feel that this variable should have 'request' somewhere in the name to avoid confusion with the response ones.
But please fix build for other ports :)
https://trac.webkit.org/r182059
(In reply to comment #5) > But please fix build for other ports :) Done in https://trac.webkit.org/changeset/182063
(In reply to comment #7) > (In reply to comment #5) > > But please fix build for other ports :) > > Done in https://trac.webkit.org/changeset/182063 I meant Antti but thanks Ossy for taking care of it.
> I meant Antti but thanks Ossy for taking care of it. I believe I fixed it in https://trac.webkit.org/r182064 .