Bug 18393 – Move POST cache policy from platform dependent file

• Status: ASSIGNED
• Component: Page Loading

[David Carson]

The cache policy for POST is set in the file ResourceHandleMac.mm
This means that every port of WebKit needs to set the policy in their implementation.

[David Carson]

Created attachment 20439 [details]
Patch to move cache setting

[Darin Adler]

Comment on attachment 20439 [details]
Patch to move cache setting

+        Move setting cache policy out of platform dependant implementation.

Misspelling of dependent here.

     // FIXME: Where's the code that implements what the comment above says?
+    // ANSWER: The cache policy is set in loadItem()

This is a bit silly. Lets remove the "FIXME" if there's nothing that needs fixing.

                 if (ResourceHandle::willLoadFromCache(request))
+                    request.setCachePolicy(ReturnCacheDataDontLoad);
                     action = NavigationAction(itemURL, loadType, false);
                 else {

This won't compile unless you add braces.

review- because it won't compile.

[David Carson]

Created attachment 20799 [details]
new patch

I was waaay to hasty with the last patch. Attached is a new patch that will compile and corrects the items pointed out in the review.

[Darin Adler]

Comment on attachment 20799 [details]
new patch

This patch, as is, breaks the Mac platform. That's because the request sent into sendSynchronousRequest won't have the cache policy set to ReturnCacheDataDontLoad.

The rearranged code sets the policy after the call, whereas the old code set it before!

I think that the old behavior where the cache policy was set to ReturnCacheDataDontLoad for the navigation itself was actually a bug -- I think the intent was to set it only for the "willLoadFromCache" check itself.

review- because we don't want to break the Mac. I won't try to rework this myself, but it should be relatively straightforward.