Closed Bug 1631148 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 23082 - Screen Enumeration: Add a placeholder Screen.id to replace Screen.name

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox77 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla77
Tracking Flags:
Tracking Status
firefox77 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Mike Wasserman <msw@chromium.org> wrote:

Screen Enumeration: Add a placeholder Screen.id to replace Screen.name

Use placeholder ordinal ids for now (0, 1, 2, ...); update tests.
TODO: generate per-origin UUIDs that reset when cookies are deleted

Test Screen.id values returned via window.getScreens() with:
chrome://flags#enable-experimental-web-platform-features OR
$ chrome --enable-blink-features=ScreenEnumeration
Try https://michaelwasserman.github.io/window-placement-demo

Bug: 994889
Test: Automated, Screen.id placeholder values
Change-Id: I0c5b18b92fef251211822f3473ea85472a7ee945
Reviewed-on: https://chromium-review.googlesource.com/2155113
WPT-Export-Revision: 44522bd32993cf89494a8f18c44e836cb0adf1a7

CI Results

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

Total 60 tests

Status Summary

Firefox

OK : 1
PASS: 59
FAIL: 2

Chrome

OK : 1
PASS: 1
FAIL: 1

Safari

OK : 1
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/screen_enumeration/getScreens.tentative.https.window.html
self.getScreens returns at least 1 Screen: FAIL (Chrome: FAIL, Safari: FAIL)
self.getScreens is present: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0e2c09678d3c
[wpt PR 23082] - Screen Enumeration: Add a placeholder Screen.id to replace Screen.name, a=testonly
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/7d0fd0d31d2b
[wpt PR 23082] - Screen Enumeration: Add a placeholder Screen.id to replace Screen.name, a=testonly

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

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox77: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.