[Qt] Fix methods.html and methods-lower-case.html
Created attachment 96265 [details] Patch
Created attachment 96268 [details] Patch
Fyi for the reviewer, this is how the Qt results for the unskipped methods-* tests compare to Mac: --- /tmp/layout-test-results/http/tests/xmlhttprequest/methods-expected.txt 2011-06-07 19:37:36.000000000 +0100 +++ /tmp/layout-test-results/http/tests/xmlhttprequest/methods-actual.txt 2011-06-07 19:37:36.000000000 +0100 @@ -4,17 +4,17 @@ GET(null): GET(null bytes), Content-Type: null GET("123"): GET(null bytes), Content-Type: null POST(""): POST(0 bytes), Content-Type: application/xml -POST(null): POST(0 bytes), Content-Type: null +POST(null): POST(0 bytes), Content-Type: application/octet-stream POST("123"): POST(3 bytes), Content-Type: application/xml PUT(""): PUT(0 bytes), Content-Type: application/xml PUT(null): PUT(0 bytes), Content-Type: null PUT("123"): PUT(3 bytes), Content-Type: application/xml -DELETE(""): DELETE(0 bytes), Content-Type: application/xml -DELETE(null): DELETE(0 bytes), Content-Type: null -DELETE("123"): DELETE(3 bytes), Content-Type: application/xml -HEAD(""): HEAD(0 bytes), Content-Type: null -HEAD(null): HEAD(0 bytes), Content-Type: null -HEAD("123"): HEAD(0 bytes), Content-Type: null +DELETE(""): DELETE(null bytes), Content-Type: null +DELETE(null): DELETE(null bytes), Content-Type: null +DELETE("123"): DELETE(null bytes), Content-Type: null +HEAD(""): HEAD(null bytes), Content-Type: null +HEAD(null): HEAD(null bytes), Content-Type: null +HEAD("123"): HEAD(null bytes), Content-Type: null WKFOOBAR(""): WKFOOBAR(0 bytes), Content-Type: application/xml WKFOOBAR(null): WKFOOBAR(0 bytes), Content-Type: null WKFOOBAR("123"): WKFOOBAR(3 bytes), Content-Type: application/xml The differences are described in the Changelog. This bug is a relative of https://bugs.webkit.org/show_bug.cgi?id=60440 so cc'ing Luiz and Adam.
Comment on attachment 96268 [details] Patch I am not an official reviewer but it LGTM.
Comment on attachment 96268 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=96268&action=review r=me > Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp:601 > +void QNetworkReplyHandler::clearContentsHeaders() clearContentHeaders() > Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp:603 > + // Clearing Contents-length and Contents-type of the requests that do not have contents. Content-length and Content-type > Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp:613 > + // We may be uploading files so prevent QNR from buffering data Period at end of Kenneth.
Committed r88612: <http://trac.webkit.org/changeset/88612>
*** Bug 60566 has been marked as a duplicate of this bug. ***