1806923 - [wpt-sync] Sync PR 37640 - Revert "Reject getDisplayMedia calls without user activation"

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 37640 into mozilla-central (this bug is closed when the sync is complete).

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

Peter Williamson <[e-mail]> wrote:

Revert "Reject getDisplayMedia calls without user activation"

This reverts commit 5fbd1e8cdadc327dedfad9ce896e24c562e59fbf.

Reason for revert: Reverted by Sheriff on suspicion of causing a test failure. https://ci.chromium.org/ui/p/chromium/builders/ci/Linux%20Tests%20(dbg)(1)/110084/test-results?sortby=&groupby=&q=ExactID%3Aninja%3A%2F%2F%3Ablink_wpt_tests%2Fexternal%2Fwpt%2Fscreen-capture%2Fpermissions-policy-audio%2Bvideo.https.sub.html+VHash%3A9f5ebdc6f73354e7.

The test failure lists getDisplayMedia failing as the cause, and this CL is in the blame set and looks related.

Original change's description:

Reject getDisplayMedia calls without user activation

This CL makes sure getDisplayMedia() returned promise reject with
InvalidStateError when called without user activation.
Note that speculation rules tests are updated as well[1].

[1] https://github.com/WICG/nav-speculation/issues/225

Intent to remove: https://groups.google.com/a/chromium.org/g/blink-dev/c/YGmuAVOqftI

Bug: 1198918
Change-Id: I530761d95e6db412582d0f19c0170803516c33f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4096797
Reviewed-by: Dale Curtis \<[e-mail]>
Reviewed-by: Hiroki Nakagawa \<[e-mail]>
Reviewed-by: Elad Alon \<[e-mail]>
Commit-Queue: Fr \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1086033}

Bug: 1198918
Change-Id: I14d0e47ee406aa09b6eec763e94d8e83f282a04d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4119423
Auto-Submit: Peter Williamson \<[e-mail]>
Quick-Run: Peter Williamson \<[e-mail]>
Commit-Queue: Rubber Stamper \<[e-mail]>
Owners-Override: Peter Williamson \<[e-mail]>
Bot-Commit: Rubber Stamper \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1086147}

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=9f8a048f2e41f4565a8c5a346ddcb122c7130b42

Comment 2

[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 3 tests and 1 subtests

Status Summary

Firefox

OK : 2
FAIL : 2
ERROR : 1

Chrome

OK : 2
PASS : 1
FAIL : 1
TIMEOUT: 1
ERROR : 1

Safari

OK : 2
FAIL : 2
ERROR : 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /mediacapture-streams/MediaStreamTrack-iframe-audio-transfer.https.html [wpt.fyi]
  • MediaStreamTrack transfer to iframe: FAIL (Chrome: TIMEOUT, Safari: FAIL)
• /mediacapture-streams/MediaStreamTrack-iframe-transfer.https.html [wpt.fyi]
  • MediaStreamTrack transfer to iframe: FAIL (Chrome: FAIL, Safari: FAIL)
• /speculation-rules/prerender/restriction-screen-capture.https.html [wpt.fyi]: ERROR (Chrome: OK, Safari: ERROR)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0e91b78b004a
[wpt PR 37640] - Revert "Reject getDisplayMedia calls without user activation", a=testonly

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/0e91b78b004a