1444166 - [wpt-sync] Sync PR 9931 - Add used body replacement test for Request constructor

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P3)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 9931 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/w3c/web-platform-tests/pull/9931
Details from upstream follow.

Harris Hancock wrote:
>  Add used body replacement test for Request constructor
>  
>  This is a WPT companion PR for a Fetch spec change: https://github.com/whatwg/fetch/pull/675. That change allows this currently throwy code snippet to function as expected:
>  
>  ```js
>  let request = new Request(url, { method: "POST", body: "foo" })
>  await request.text()  // disturb the body
>  
>  // currently throws because request is disturbed,
>  // even thought we're providing a new body
>  request = new Request(request, { body: "bar" })
>  ```
>  
>  I tested this on Chrome and Epiphany Tech Preview. As expected, it fails on both because: Chrome does not implement `request.body`, and EPT implements the spec as currently written.
>  
>  /cc @yutakahirano, @annevk

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=2bca8a35410f448fe9917ad9bfc1403a2f6d91cc

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=f73276b2e761e241eab5f3a9b4c89bd6d8fee74e

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Ran 1 tests and 9 subtests
OK     : 1
PASS   : 4
FAIL   : 5

New tests that have failures or other problems:
/fetch/api/request/request-disturbed.html
    Check creating a new request with a new body from a disturbed request: FAIL

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ad7b8e6c3816
[wpt PR 9931] - Add used body replacement test for Request constructor, a=testonly

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/ad7b8e6c3816

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/ad7b8e6c3816