[webkit-reviews] review requested: [Bug 35300] (GTK) HTTP 307 after a 303 after a POST re-sends POST data from the original request : [Attachment 196679] Patch

[bugzilla-daemon at webkit.org]

Alberto Garcia <agarcia at igalia.com> has asked  for review:
Bug 35300: (GTK) HTTP 307 after a 303 after a POST re-sends POST data from the
original request
https://bugs.webkit.org/show_bug.cgi?id=35300

Attachment 196679: Patch
https://bugs.webkit.org/attachment.cgi?id=196679&action=review

------- Additional Comments from Alberto Garcia <agarcia at igalia.com>
(In reply to comment #3)
> As of the DRT URL bug, the WebKitWebResource private uri field is
> changed in FrameLoaderClient::dispatchWillSendRequest.

> I was wondering whether it would be a more appropriate model to keep
> this field unchanged accross redirections?

Yes, this actually seems to be the problem.

Do we need to change that field? The identifier is supposed to be
associated to the initial request.

It works fine if we do it like that, here's the patch, I tried all
http/tests and it doesn't seem to break anything.