Bug 214873 - REGRESSION (r264950): [ iOS wk2 ] broke 4 xmlhttprequest tests
Summary: REGRESSION (r264950): [ iOS wk2 ] broke 4 xmlhttprequest tests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-07-28 07:03 PDT by Karl Rackler
Modified: 2020-07-28 10:50 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Rackler 2020-07-28 07:03:43 PDT 
Description:
http/tests/xmlhttprequest/methods-async.html
http/tests/xmlhttprequest/methods.html
http/tests/xmlhttprequest/workers/methods-async.html
http/tests/xmlhttprequest/workers/methods.html

4 xmlhttprequest tests are a constant failure after adding test expectations and baselines for iOS 14
https://trac.webkit.org/changeset/264950/webkit

History:
https://results.webkit.org/?platform=ios&flavor=wk2&limit=50000&suite=layout-tests&suite=layout-tests&suite=layout-tests&suite=layout-tests&test=http%2Ftests%2Fxmlhttprequest%2Fmethods-async.html&test=http%2Ftests%2Fxmlhttprequest%2Fmethods.html&test=http%2Ftests%2Fxmlhttprequest%2Fworkers%2Fmethods-async.html&test=http%2Ftests%2Fxmlhttprequest%2Fworkers%2Fmethods.html

Diff:
http/tests/xmlhttprequest/methods-async.html

--- /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/methods-async-expected.txt
+++ /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/methods-async-actual.txt
@@ -12,9 +12,9 @@
 DELETE(""): DELETE(0 bytes), Content-Type: text/plain;charset=UTF-8
 DELETE(null): DELETE(0 bytes), Content-Type: null
 DELETE("123"): DELETE(3 bytes), Content-Type: text/plain;charset=UTF-8
-HEAD(""): HEAD(null bytes), Content-Type: null
-HEAD(null): HEAD(null bytes), Content-Type: null
-HEAD("123"): HEAD(null bytes), Content-Type: null
+HEAD(""): HEAD(0 bytes), Content-Type: null
+HEAD(null): HEAD(0 bytes), Content-Type: null
+HEAD("123"): HEAD(0 bytes), Content-Type: null
 WKFOOBAR(""): WKFOOBAR(0 bytes), Content-Type: text/plain;charset=UTF-8
 WKFOOBAR(null): WKFOOBAR(0 bytes), Content-Type: null
 WKFOOBAR("123"): WKFOOBAR(3 bytes), Content-Type: text/plain;charset=UTF-8

Diff:
http/tests/xmlhttprequest/methods.html

--- /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/methods-expected.txt
+++ /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/methods-actual.txt
@@ -12,9 +12,9 @@
 DELETE(""): DELETE(0 bytes), Content-Type: text/plain;charset=UTF-8
 DELETE(null): DELETE(0 bytes), Content-Type: null
 DELETE("123"): DELETE(3 bytes), Content-Type: text/plain;charset=UTF-8
-HEAD(""): HEAD(null bytes), Content-Type: null
-HEAD(null): HEAD(null bytes), Content-Type: null
-HEAD("123"): HEAD(null bytes), Content-Type: null
+HEAD(""): HEAD(0 bytes), Content-Type: null
+HEAD(null): HEAD(0 bytes), Content-Type: null
+HEAD("123"): HEAD(0 bytes), Content-Type: null
 WKFOOBAR(""): WKFOOBAR(0 bytes), Content-Type: text/plain;charset=UTF-8
 WKFOOBAR(null): WKFOOBAR(0 bytes), Content-Type: null
 WKFOOBAR("123"): WKFOOBAR(3 bytes), Content-Type: text/plain;charset=UTF-8

Diff:
http/tests/xmlhttprequest/workers/methods-async.html

--- /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/workers/methods-async-expected.txt
+++ /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/workers/methods-async-actual.txt
@@ -12,9 +12,9 @@
 DELETE(""): DELETE(0 bytes), Content-Type: text/plain;charset=UTF-8
 DELETE(null): DELETE(0 bytes), Content-Type: null
 DELETE("123"): DELETE(3 bytes), Content-Type: text/plain;charset=UTF-8
-HEAD(""): HEAD(null bytes), Content-Type: null
-HEAD(null): HEAD(null bytes), Content-Type: null
-HEAD("123"): HEAD(null bytes), Content-Type: null
+HEAD(""): HEAD(0 bytes), Content-Type: null
+HEAD(null): HEAD(0 bytes), Content-Type: null
+HEAD("123"): HEAD(0 bytes), Content-Type: null
 WKFOOBAR(""): WKFOOBAR(0 bytes), Content-Type: text/plain;charset=UTF-8
 WKFOOBAR(null): WKFOOBAR(0 bytes), Content-Type: null
 WKFOOBAR("123"): WKFOOBAR(3 bytes), Content-Type: text/plain;charset=UTF-8

Diff:

http/tests/xmlhttprequest/workers/methods.html
--- /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/workers/methods-expected.txt
+++ /Volumes/Data/slave/ipados-simulator-13-debug-tests-wk2/build/layout-test-results/http/tests/xmlhttprequest/workers/methods-actual.txt
@@ -12,9 +12,9 @@
 DELETE(""): DELETE(0 bytes), Content-Type: text/plain;charset=UTF-8
 DELETE(null): DELETE(0 bytes), Content-Type: null
 DELETE("123"): DELETE(3 bytes), Content-Type: text/plain;charset=UTF-8
-HEAD(""): HEAD(null bytes), Content-Type: null
-HEAD(null): HEAD(null bytes), Content-Type: null
-HEAD("123"): HEAD(null bytes), Content-Type: null
+HEAD(""): HEAD(0 bytes), Content-Type: null
+HEAD(null): HEAD(0 bytes), Content-Type: null
+HEAD("123"): HEAD(0 bytes), Content-Type: null
 WKFOOBAR(""): WKFOOBAR(0 bytes), Content-Type: text/plain;charset=UTF-8
 WKFOOBAR(null): WKFOOBAR(0 bytes), Content-Type: null
 WKFOOBAR("123"): WKFOOBAR(3 bytes), Content-Type: text/plain;charset=UTF-8
Comment 1 Radar WebKit Bug Importer 2020-07-28 07:04:06 PDT 
<rdar://problem/66215972>
Comment 2 Karl Rackler 2020-07-28 07:17:32 PDT 
I have marked these tests as failing while this issue is investigated.
https://trac.webkit.org/changeset/264982/webkit
Comment 3 Ryan Haddad 2020-07-28 10:43:40 PDT 
I fixed this in https://trac.webkit.org/changeset/264989/webkit
Comment 4 Ryan Haddad 2020-07-28 10:50:26 PDT 
Committed r264996: <https://trac.webkit.org/changeset/264996>