Changeset 181902 in webkit

Timestamp:

Mar 24, 2015, 1:11:38 PM (11 years ago)

Author:

Chris Dumez

Message:

[WK2] Remove NetworkCache FIXME comment about caching responses to OPTIONS requests
​https://bugs.webkit.org/show_bug.cgi?id=143009

Reviewed by Andreas Kling.

Remove NetworkCache FIXME comment about caching responses to OPTIONS
method as RFC 7234 says "Responses to the OPTIONS method are not
cacheable" [1].

[1] ​http://tools.ietf.org/html/rfc7231#page-31

• NetworkProcess/cache/NetworkCache.cpp:

(WebKit::NetworkCache::canRetrieve):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• NetworkProcess/cache/NetworkCache.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2015-03-24  Chris Dumez  <cdumez@apple.com>
+
+        [WK2] Remove NetworkCache FIXME comment about caching responses to OPTIONS requests
+        https://bugs.webkit.org/show_bug.cgi?id=143009
+
+        Reviewed by Andreas Kling.
+
+        Remove NetworkCache FIXME comment about caching responses to OPTIONS
+        method as RFC 7234 says "Responses to the OPTIONS method are not
+        cacheable" [1].
+
+        [1] http://tools.ietf.org/html/rfc7231#page-31
+
+        * NetworkProcess/cache/NetworkCache.cpp:
+        (WebKit::NetworkCache::canRetrieve):
+
 2015-03-24  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebKit2/NetworkProcess/cache/NetworkCache.cpp

@@ -246 +246 @@
 static RetrieveDecision canRetrieve(const WebCore::ResourceRequest& request)
 {
-    // FIXME: Support HEAD and OPTIONS requests.
+    // FIXME: Support HEAD requests.
     if (request.httpMethod() != "GET")
         return RetrieveDecision::NoDueToHTTPMethod;