[wpt-sync] Sync PR 32501 - Reland "Window Placement: Add multi-screen window.open() popup WPTs"
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox98 fixed)
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)
(manualwpt 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.htmlBug: 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: If8e8d79dec8fd725934b257b6fcccf3f8cbecf1dReviewed-on: https://chromium-review.googlesource.com/3408016
WPT-Export-Revision: 923d965ccd8f7c884d4b177ce16ef9e9451b057b
Assignee | ||
Comment 1•3 years ago
|
||
Assignee | ||
Comment 2•3 years ago
|
||
Assignee | ||
Comment 3•3 years ago
|
||
Assignee | ||
Comment 4•3 years ago
|
||
Assignee | ||
Comment 5•3 years ago
|
||
Assignee | ||
Comment 6•3 years ago
|
||
Assignee | ||
Comment 7•3 years ago
|
||
Assignee | ||
Comment 8•3 years ago
|
||
Assignee | ||
Comment 9•3 years ago
|
||
Assignee | ||
Comment 10•3 years ago
|
||
Assignee | ||
Comment 11•3 years ago
|
||
Assignee | ||
Comment 12•3 years ago
|
||
Comment 13•3 years ago
|
||
bugherder |
Description
•