No need to clobber all ResourceRequest fields in FrameLoader::reload() Presently, FrameLoader::reload clobbers all fields of the initialRequest when there is an unreachableURL associated with the current DocumentLoader. This causes problems for the Chromium port since we store some extra data on the ResourceRequest object that we need to have preserved across a FrameLoader::reload call. FrameLoader::reloadAllowingStaleData is careful to only call setURL when there is an unreachableURL, and I think that FrameLoader::reload should do the same. I suspect that it was not intentional to make FrameLoader::reload behave differently in this respect. Please see the attached patch for further details.
Created attachment 24751 [details] v1 patch
To clarify, this causes problems for us because the HTTP method is forced to be GET. If the previous request was something else, then that information is lost. I guess this doesn't cause a problem for Safari, but it still seems like an error to me.
Comment on attachment 24751 [details] v1 patch Anders, can you please take a look at this patch? Thanks!
Comment on attachment 24751 [details] v1 patch Looks good, I can't think of a reason why it would break things (but I've been wrong before ;) r=me!
http://trac.webkit.org/changeset/38107