[webkit-reviews] review denied: [Bug 60144] [Qt] QNetworkReplyHandler processes redirect in the wrong slot : [Attachment 101156] With description.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jul 18 11:18:21 PDT 2011


Luiz Agostini <luiz at webkit.org> has denied Pierre Rossi
<pierre.rossi at gmail.com>'s request for review:
Bug 60144: [Qt] QNetworkReplyHandler processes redirect in the wrong slot
https://bugs.webkit.org/show_bug.cgi?id=60144

Attachment 101156: With description.
https://bugs.webkit.org/attachment.cgi?id=101156&action=review

------- Additional Comments from Luiz Agostini <luiz at webkit.org>
I think that with this change sendResponseIfNeeded could get called twice, what
would be wrong.
Anyway, to be more confident, I would like to see a layout test that receives a
redirection response with content, and that the content of the response is
splited in chunks with some delay between them.

Although I am not a formal reviewer, for now I will r- because of the lack of
the test.


More information about the webkit-reviews mailing list