corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Backed out changeset 07b44dd04f0d (bug 1397646) for causing XHR related mochitest failures.

Browse Source
This commit is contained in:
Iulian Moraru
2023-08-01 01:56:42 +03:00
parent 7beb3fc5e7
commit 1e4a9bd3bd
3 changed files with 61 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
netwerk/protocol/http/HttpBaseChannel.cpp
View File

@@ -2159,14 +2159,7 @@ HttpBaseChannel::GetResponseStatus(uint32_t* aValue) {
NS_IMETHODIMP NS_IMETHODIMP
HttpBaseChannel::GetResponseStatusText(nsACString& aValue) { HttpBaseChannel::GetResponseStatusText(nsACString& aValue) {
if (!mResponseHead) return NS_ERROR_NOT_AVAILABLE; if (!mResponseHead) return NS_ERROR_NOT_AVAILABLE;
nsAutoCString version; mResponseHead->StatusText(aValue);
// https://fetch.spec.whatwg.org :
// Responses over an HTTP/2 connection will always have the empty byte
// sequence as status message as HTTP/2 does not support them.
if (NS_WARN_IF(NS_FAILED(GetProtocolVersion(version))) ||
!version.EqualsLiteral("h2")) {
mResponseHead->StatusText(aValue);
}
return NS_OK; return NS_OK;
} }

40
testing/web-platform/meta/fetch/api/basic/status.h2.any.js.ini
View File

@@ -1,8 +1,48 @@
[status.h2.any.html] [status.h2.any.html]
expected: expected:
if (os == "android") and fission: [OK, TIMEOUT] if (os == "android") and fission: [OK, TIMEOUT]
[statusText over H2 for status 200 should be the empty string]
expected: FAIL
[statusText over H2 for status 210 should be the empty string]
expected: FAIL
[statusText over H2 for status 400 should be the empty string]
expected: FAIL
[statusText over H2 for status 404 should be the empty string]
expected: FAIL
[statusText over H2 for status 410 should be the empty string]
expected: FAIL
[statusText over H2 for status 500 should be the empty string]
expected: FAIL
[statusText over H2 for status 502 should be the empty string]
expected: FAIL
[status.h2.any.worker.html] [status.h2.any.worker.html]
expected: expected:
if (os == "android") and fission: [OK, TIMEOUT] if (os == "android") and fission: [OK, TIMEOUT]
[statusText over H2 for status 200 should be the empty string]
expected: FAIL
[statusText over H2 for status 210 should be the empty string]
expected: FAIL
[statusText over H2 for status 400 should be the empty string]
expected: FAIL
[statusText over H2 for status 404 should be the empty string]
expected: FAIL
[statusText over H2 for status 410 should be the empty string]
expected: FAIL
[statusText over H2 for status 500 should be the empty string]
expected: FAIL
[statusText over H2 for status 502 should be the empty string]
expected: FAIL

20
testing/web-platform/meta/xhr/status.h2.window.js.ini
View File

@@ -1,3 +1,23 @@
[status.h2.window.html] [status.h2.window.html]
expected: expected:
if (os == "android") and fission: [OK, TIMEOUT] if (os == "android") and fission: [OK, TIMEOUT]
[statusText over H2 for status 200 should be the empty string]
expected: FAIL
[statusText over H2 for status 210 should be the empty string]
expected: FAIL
[statusText over H2 for status 400 should be the empty string]
expected: FAIL
[statusText over H2 for status 404 should be the empty string]
expected: FAIL
[statusText over H2 for status 410 should be the empty string]
expected: FAIL
[statusText over H2 for status 500 should be the empty string]
expected: FAIL
[statusText over H2 for status 502 should be the empty string]
expected: FAIL