1722100 - [wpt-sync] Sync PR 29769 - cross-origin-opener-policy/resource-popup.https.html makes incorrect …

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task, P4)

Description

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

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

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

b'Chris Dumez <[e-mail]>' wrote:

cross-origin-opener-policy/resource-popup.https.html makes incorrect …

…assumption about win.location.href

The test expects that win.location.href starts returning something else than "about:blank" after closing.
While this is true in Firefox and Blink, none of the browser engines actually agree on this (Firefox
returns the empty string, Chrome returns undefined and Safari returns "about:blank").

Per the specification, it seems returning "about:blank" after a window is closed is the expected behavior,
see https://github.com/whatwg/html/issues/6899. I am fixing thus fixing the test so that it can run in
all browsers by checking if "the window is closed" OR "href is no longer about:blank because it was
navigated".

Comment 1

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

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

Comment 2

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

CI Results

Ran 11 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 12 subtests

Status Summary

Firefox

OK : 1
PASS: 10[Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview] 11[Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 12[Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-32-qr-debug, Gecko-windows10-32-qr-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, GitHub]
FAIL: 1[Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 2[Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview]

Chrome

OK : 1
PASS: 12

Safari

OK : 1
FAIL: 12

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/html/cross-origin-opener-policy/resource-popup.https.html
/common/dummy.xml - parent COOP: ""; child COOP: "same-origin": FAIL [Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview], PASS [Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-32-qr-debug, Gecko-windows10-32-qr-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, GitHub] (Chrome: PASS, Safari: FAIL)
/images/red.png - parent COOP: ""; child COOP: "same-origin": FAIL [Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview], PASS [Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-32-qr-debug, Gecko-windows10-32-qr-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, GitHub] (Chrome: PASS, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/549f074a5089
[wpt PR 29769] - cross-origin-opener-policy/resource-popup.https.html makes incorrect …, a=testonly

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/059e38a41bba
[wpt PR 29769] - cross-origin-opener-policy/resource-popup.https.html makes incorrect …, a=testonly

Comment 5

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/059e38a41bba