Closed Bug 1751495 Opened 3 years ago Closed 3 years ago

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

Categories

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

task

Tracking

(firefox98 fixed)

RESOLVED FIXED
98 Branch
Tracking Status
firefox98 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Mike Wasserman <msw@chromium.org> wrote:

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

This reverts commit 8fc260bdceac8c1eaf24c31831a31f11172d38c0.

Reason for revert: Updating test to avoid flaky failures.

Original change's description:

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 \<reillyg@chromium.org>
Commit-Queue: Mike Wasserman \<msw@chromium.org>
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/c/chromium/src/+/3407776
Bot-Commit: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Mike Wasserman \<msw@chromium.org>
Commit-Queue: Stefan Zager \<szager@chromium.org>
Cr-Commit-Position: refs/heads/main@{#961970}

Bug: 1289727
Change-Id: If8e8d79dec8fd725934b257b6fcccf3f8cbecf1d

Reviewed-on: https://chromium-review.googlesource.com/3408016
WPT-Export-Revision: 923d965ccd8f7c884d4b177ce16ef9e9451b057b

Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
You need to log in before you can comment on or make changes to this bug.