Closed
Bug 1444002
Opened 7 years ago
Closed 7 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•7 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Reporter | ||
Updated•7 years ago
|
Component: web-platform-tests → DOM
Product: Testing → Core
Reporter | ||
Comment 1•7 years ago
|
||
Comment 2•7 years ago
|
||
I'm going to manually land these test changes as part of bug 1394399.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Comment 3•7 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•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•