Bug 62221 – [Qt] Fix methods.html and methods-lower-case.html

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Robert Hogan]

[Qt] Fix methods.html and methods-lower-case.html

[Robert Hogan]

Created attachment 96265 [details]
Patch

[Robert Hogan]

Created attachment 96268 [details]
Patch

[Robert Hogan]

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.

[Luiz Agostini]

Comment on attachment 96268 [details]
Patch

I am not an official reviewer but it LGTM.

[Andreas Kling]

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.

[Robert Hogan]

Committed r88612: <http://trac.webkit.org/changeset/88612>

[Robert Hogan]

*** Bug 60566 has been marked as a duplicate of this bug. ***