Open Bug 1664128 Opened 5 years ago Updated 1 year ago

Adjust COOP redirect handling

Categories

(Core :: DOM: Navigation, enhancement)

Product:
Core
Component:
DOM: Navigation
Type:
enhancement

Tracking

()

People

(Reporter: annevk, Unassigned, NeedInfo)

References

(Blocks 1 open bug)

Details

See html/cross-origin-opener-policy/popup-redirect-same-origin-allow-popups.https.html in WPT and https://github.com/whatwg/html/pull/5739.

See the discussion in https://github.com/camillelamy/explainers/issues/12 for context.

Valentin, is this something you can take on?

Flags: needinfo?(valentin.gosu)
Depends on: 1657912

P2/S2

Severity: -- → S2
Priority: -- → P2
QA Whiteboard: qa-not-actionable
Severity: S2 → N/A
Type: defect → enhancement
Priority: P2 → --

I think this is still a problem
https://wpt.fyi/results/html/cross-origin-opener-policy/popup-redirect-same-origin-allow-popups.https.html?label=experimental&label=master&aligned

It's been a while since I worked on this. Looking at the code when computing the COOP mismatch for a redirect this code exits at the end, with no mismatch.

I'm not sure if at this point currentWindowGlobal->IsInitialDocument() should still return true, or if there is a bug in this code.
Maybe Eden knows?

Flags: needinfo?(valentin.gosu) → needinfo?(echuang)
You need to log in before you can comment on or make changes to this bug.