Bug 214873

Summary:	REGRESSION (r264950): [ iOS wk2 ] broke 4 xmlhttprequest tests
Product:	WebKit	Reporter:	Karl Rackler <rackler>
Component:	New Bugs	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

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>