search

w3c / mediacapture-viewport

Specification for getViewportMedia, an API to capture the content of a browser viewport.
https://w3c.github.io/mediacapture-viewport/
Other
6 stars 4 forks source link

Set [Restrictable] on the video track #31

Open eladalon1983 opened 1 week ago

eladalon1983 commented 1 week ago

This marks video tracks obtained using getViewportMedia() as eligible tracks for Element Capture.

Preview | Diff

youennf commented 1 week ago

Should there be a PR on element capture as well?

eladalon1983 commented 1 week ago

Should there be a PR on element capture as well?

There was: https://github.com/screen-share/element-capture/pull/49

eladalon1983 commented 1 week ago

Looks ok to me.

Great!

Overall, I feel like we should use https://w3c.github.io/mediacapture-main/#dfn-create-a-mediastreamtrack here, and pass the restrictable value to it.

If you feel strongly, I am not opposed. But please consider that the current change is less intrusive, and could be rolled back more easily if need be, with less changes to other specs.