1751455 - [wpt-sync] Sync PR 32498 - Revert "Window Placement: Add multi-screen window.open() popup WPTs"

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

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

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

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

Stefan Zager <[e-mail]> wrote:

Revert "Window Placement: Add multi-screen window.open() popup WPTs"

This reverts commit a55dbd61c51f3b6f64f5323c96e6e936cb4eebf8.

Reason for revert: this test is flaky on the bots:

https://ci.chromium.org/p/chromium/builders/ci/Win7%20Tests%20%281%29?limit=200

Original change's description:

Window Placement: Add multi-screen window.open() popup WPTs

Add a multi-screen window.open WPT that can run manually or automated.
(automated content_shell runs only test one screen w/ (0,0) placement)
(manual wpt serve + Chromium runs test real multi-screen placements)
Make some minor naming updates in the existing getScreenDetails test.

To run manually with Blink/Chromium:
$ third_party/blink/tools/run_blink_wptserve.py -t Default
$ out/Default/chrome -- http://localhost:8001/window-placement/multi-screen-window-open-manual.tentative.https.html

Bug: 1022988
Test: automated (or manual) WPT
Change-Id: I00293336bee15460a48b9c15f8655c00d7e5d4a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3373713
Reviewed-by: Reilly Grant \<[e-mail]>
Commit-Queue: Mike Wasserman \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#961626}

Bug: 1289727
Change-Id: I91444f7d1a5b8179bd4b2cc7f179b8e485493e19
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

Reviewed-on: https://chromium-review.googlesource.com/3407776
WPT-Export-Revision: c977d8803089e75223187cf77de40f543663c705

Comment 1

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

PR 32498 applied with additional changes from upstream: 16c49f7b9752b3d859823ab0de9d5fe2de113a3e

Comment 2

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

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

Comment 3

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

CI Results

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

Total 1 tests and 5 subtests

Status Summary

Firefox

OK : 1
FAIL: 5

Chrome

OK : 1
PASS: 5

Safari

OK : 1
FAIL: 5

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/screen_enumeration/getScreenDetails.tentative.https.window.html
getScreenDetails() is present: FAIL (Chrome: PASS, Safari: FAIL)
getScreenDetails() returns at least 1 Screen with permission granted: FAIL (Chrome: PASS, Safari: FAIL)
getScreenDetails() rejects the promise with permission denied: FAIL (Chrome: PASS, Safari: FAIL)
getScreenDetails() resolves for attached iframe; rejects for detached iframe: FAIL (Chrome: PASS, Safari: FAIL)
Cached Screens interface from detached iframe doesn't crash, behaves okay: FAIL (Chrome: PASS, Safari: FAIL)

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/7d8532466a15