<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><div><div id="patch" style="width: 100%; "><div class="modfile" style="border-top-width: 1px; border-right-width: 1px; border-bottom-width: 1px; border-left-width: 1px; border-top-style: solid; border-right-style: solid; border-bottom-style: solid; border-left-style: solid; border-top-color: rgb(204, 204, 204); border-right-color: rgb(204, 204, 204); border-bottom-color: rgb(204, 204, 204); border-left-color: rgb(204, 204, 204); margin-top: 10px; margin-right: 0px; margin-bottom: 10px; margin-left: 0px; position: static; z-index: auto; "><pre class="diff" style="line-height: 1.2em; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; width: 100%; background-image: initial; background-repeat: initial; background-attachment: initial; -webkit-background-clip: initial; -webkit-background-origin: initial; background-color: rgb(238, 238, 238); padding-top: 0px; padding-right: 0px; padding-bottom: 10px; padding-left: 0px; overflow-x: auto; overflow-y: auto; background-position: initial initial; "><span style="display: block; padding-top: 0px; padding-right: 10px; padding-bottom: 0px; padding-left: 10px; "><span class="cx" style="display: block; padding-top: 0px; padding-right: 10px; padding-bottom: 0px; padding-left: 10px; ">
</span><ins style="background-image: initial; background-repeat: initial; background-attachment: initial; -webkit-background-clip: initial; -webkit-background-origin: initial; background-color: rgb(221, 255, 221); text-decoration: none; display: block; padding-top: 0px; padding-right: 10px; padding-bottom: 0px; padding-left: 10px; background-position: initial initial; ">+ // FIXME: POST documents are always Reloads, but their subresources should still be Revalidate.</ins></span></pre></div></div></div></span></blockquote><div>I think the comment should say "should still be Verify, unless actually reloading the page".</div><br><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><div><div id="patch" style="width: 100%; "><div class="modfile" style="border-top-width: 1px; border-right-width: 1px; border-bottom-width: 1px; border-left-width: 1px; border-top-style: solid; border-right-style: solid; border-bottom-style: solid; border-left-style: solid; border-top-color: rgb(204, 204, 204); border-right-color: rgb(204, 204, 204); border-bottom-color: rgb(204, 204, 204); border-left-color: rgb(204, 204, 204); margin-top: 10px; margin-right: 0px; margin-bottom: 10px; margin-left: 0px; position: static; z-index: auto; "><pre class="diff" style="line-height: 1.2em; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; width: 100%; background-image: initial; background-repeat: initial; background-attachment: initial; -webkit-background-clip: initial; -webkit-background-origin: initial; background-color: rgb(238, 238, 238); padding-top: 0px; padding-right: 0px; padding-bottom: 10px; padding-left: 0px; overflow-x: auto; overflow-y: auto; background-position: initial initial; "><span style="display: block; padding-top: 0px; padding-right: 10px; padding-bottom: 0px; padding-left: 10px; "><ins style="background-image: initial; background-repeat: initial; background-attachment: initial; -webkit-background-clip: initial; -webkit-background-origin: initial; background-color: rgb(221, 255, 221); text-decoration: none; display: block; padding-top: 0px; padding-right: 10px; padding-bottom: 0px; padding-left: 10px; background-position: initial initial; ">
+ // If we bring the CachePolicy.h and ResourceRequest cache policy enums in sync with each other and
+ // remember "Revalidate" in ResourceRequests, we can remove this "POST" check and return either "Reload"
+ // or "Revalidate" if the DocumentLoader was requested with either.
+ const ResourceRequest& request(documentLoader()->request());
+ if (request.cachePolicy() == ReloadIgnoringCacheData && !equalIgnoringCase(request.httpMethod(), "post"))
+ return CachePolicyRevalidate;
+
</ins><span class="cx" style="display: block; padding-top: 0px; padding-right: 10px; padding-bottom: 0px; padding-left: 10px; "></span></span></pre></div></div></div></span></blockquote></div><div>Will this work correctly for reloading a POSTed page? I think that it probably will, as the code below returns CachePolicyRevalidate anyway, but given the comment, I'm a bit confused.</div><div><br></div><div apple-content-edited="true"> <span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><div style="word-wrap: break-word; -khtml-nbsp-mode: space; -khtml-line-break: after-white-space; "><div>- WBR, Alexey Proskuryakov</div></div></span></div><br></body></html>