Closed
Bug 1444002
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 9911 - [WIP] Update testing of Request's default credentials mode
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
DUPLICATE
of bug 1394399
People
(Reporter: wpt-sync, Unassigned)
References
Details
(Whiteboard: [wptsync downstream error])
Sync web-platform-tests PR 9911 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/w3c/web-platform-tests/pull/9911 Details from upstream follow. Dominic Farolino wrote: > [WIP] Update testing of Request's default credentials mode > > This is for https://github.com/whatwg/fetch/pull/585. > > I'm a little surprised if only these tests have to be updated, so I'm marking this as a WIP for now. Basically what I did though, was: > > - Ran all of the `/fetch`, `/xor`, `service-workers`, and `/cors` tests on Chromium, and noted failues (none) > - Modified the Request constructor to use the `same-origin` credentials fallback instead of the previous `omit` > - Ran all of the previous tests again and noted the failures (only two) > - Updated the affected tests!
Reporter | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Reporter | ||
Updated•6 years ago
|
Component: web-platform-tests → DOM
Product: Testing → Core
Reporter | ||
Comment 1•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e4333427fed7a8c7651f955d7f370edfea285fb
Comment 2•6 years ago
|
||
I'm going to manually land these test changes as part of bug 1394399.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Comment 3•6 years ago
|
||
Ran 2 tests and 22 subtests OK : 2 PASS : 20 FAIL : 2 Existing tests that now have a worse result (e.g. they used to PASS and now FAIL): /fetch/api/request/request-init-003.sub.html Check request values when initialized from url string: FAIL /fetch/api/request/request-structure.html Check credentials attribute: FAIL
Assignee | ||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•