Skip to content

Commit

Permalink
Reject getDisplayMedia calls without user activation
Browse files Browse the repository at this point in the history
This CL makes sure getDisplayMedia() returned promise reject with
InvalidStateError when called without user activation.
Note that speculation rules tests are removed as well[1].

[1] WICG/nav-speculation#225

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

Bug: 1198918
Change-Id: I530761d95e6db412582d0f19c0170803516c33f7
  • Loading branch information
beaufortfrancois authored and chromium-wpt-export-bot committed Dec 19, 2022
1 parent 2f096ab commit 3893909
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 87 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,12 @@
<title>MediaStreamTrack transfer to iframe</title>
<script src=/resources/testharness.js></script>
<script src=/resources/testharnessreport.js></script>
<script src="/resources/testdriver.js"></script>
<script src="/resources/testdriver-vendor.js"></script>
<script>
promise_test(async () => {
const iframe = document.createElement("iframe");
await test_driver.bless('getDisplayMedia');
const stream = await navigator.mediaDevices.getDisplayMedia({audio:true, video: true});
const track = stream.getAudioTracks()[0];
const cloned_track = track.clone();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,12 @@
<title>MediaStreamTrack transfer to iframe</title>
<script src=/resources/testharness.js></script>
<script src=/resources/testharnessreport.js></script>
<script src="/resources/testdriver.js"></script>
<script src="/resources/testdriver-vendor.js"></script>
<script>
promise_test(async () => {
const iframe = document.createElement("iframe");
await test_driver.bless('getDisplayMedia');
const stream = await navigator.mediaDevices.getDisplayMedia({video: true});
const track = stream.getVideoTracks()[0];
const cloned_track = track.clone();
Expand Down
31 changes: 0 additions & 31 deletions speculation-rules/prerender/resources/screen-capture.https.html

This file was deleted.

56 changes: 0 additions & 56 deletions speculation-rules/prerender/restriction-screen-capture.https.html

This file was deleted.

0 comments on commit 3893909

Please sign in to comment.