Closed Bug 1436497 Opened 2 years ago Closed 2 years ago

[wpt-sync] PR 9352 - Check CORS using PassesAccessControlCheck() with supplied SecurityOrigin

Categories

(Core :: Networking, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: wptsync, Assigned: jgraham)

Details

(Whiteboard: [wptsync upstream error])

Partial revert of https://chromium-review.googlesource.com/535694.

Bug: 799477
Change-Id: I878bb9bcb83afaafe8601293db9aa644fc5929b3
Reviewed-on: https://chromium-review.googlesource.com/898427
WPT-Export-Revision: 3db1eff2f63a55bcdb1cbe3536a4aa1ab162cb4b
Component: web-platform-tests → DOM
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Priority: -- → P3
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Ran 1 tests and 3 subtests
OK     : 1
PASS   : 1
FAIL   : 2

New tests that have failures or other problems:
/html/semantics/scripting-1/the-script-element/muted-errors.sub.html
    Errors for cross-origin script should be muted: FAIL
    Errors for cross-origin script should be muted even if the script is once loaded as same-origin: FAIL
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed58e17e8abe
[wpt PR 9352] - Check CORS using PassesAccessControlCheck() with supplied SecurityOrigin, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/176a3931afef
[wpt PR 9352]- Update wpt metadata, a=testonly
Whiteboard: [wptsync downstream] → [wptsync upstream error]
https://hg.mozilla.org/mozilla-central/rev/ed58e17e8abe
https://hg.mozilla.org/mozilla-central/rev/176a3931afef
Status: UNCONFIRMED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
After backing out https://bugzilla.mozilla.org/show_bug.cgi?id=1443558#c4 the wpt tests started mass failing as you can see on this push https://goo.gl/Awm6eR  

I decided to backout all the changesets from the original push https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=5138dcf1eddd750c06e05a784f170b771ba58f60 because of this mass wpt failures.
Status: RESOLVED → REOPENED
Ever confirmed: true
Flags: needinfo?(overholt)
Resolution: FIXED → ---
Target Milestone: mozilla60 → ---
Christoph, where should this bug live?
Component: DOM → DOM: Security
Flags: needinfo?(overholt) → needinfo?(ckerschb)
(In reply to Andrew Overholt [:overholt] from comment #7)
> Christoph, where should this bug live?

CORS itself lives in Necko, hence I am pushing this over to Necko.
Component: DOM: Security → Networking
Flags: needinfo?(ckerschb)
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1bc0d730f900
[wpt PR 9352] - Check CORS using PassesAccessControlCheck() with supplied SecurityOrigin, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/1484950a2822
[wpt PR 9352]- Update wpt metadata, a=testonly
https://hg.mozilla.org/mozilla-central/rev/1bc0d730f900
https://hg.mozilla.org/mozilla-central/rev/1484950a2822
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Assignee: nobody → james
You need to log in before you can comment on or make changes to this bug.