Bug 143121 – Respect cache-control directives in request

• Component: Page Loading
• Status: RESOLVED FIXED

[Antti Koivisto]

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

[Antti Koivisto]

rdar://problem/19714040

[Antti Koivisto]

Created attachment 249543 [details]
patch

[WebKit Commit Bot]

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.

[Chris Dumez]

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.

[Chris Dumez]

But please fix build for other ports :)

[Antti Koivisto]

https://trac.webkit.org/r182059

[Csaba Osztrogonác]

(In reply to comment #5)
> But please fix build for other ports :)

Done in https://trac.webkit.org/changeset/182063

[Chris Dumez]

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

[Antti Koivisto]

> I meant Antti but thanks Ossy for taking care of it.

I believe I fixed it in https://trac.webkit.org/r182064 .