Skip to content

[stable23] Reduce sent information with disabled videos #7710

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Aug 11, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #7673

Until now the stream locally shown was the stream sent to the other
peers. Now there are separate streams for each case to be able to adjust
them independently to work around specific browser behaviours.

Nevertheless, the sent stream is a single stream shared by all sender
peers, not a stream specific for each peer, so it is still generated in
the local media class, just like the local stream.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
This ensures that other participants will receive a black video track
when no visible video should be sent, even if the browser does not
properly update a previously sent track when disabled or stopped.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Although the previous behaviour should be fine this aligns the behaviour
of removed tracks with disabled and stopped tracks.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Sometimes Chromium does not render one or more frames to the stream
captured from a canvas, so the canvas is now rendered several times and
for a longer period to try to avoid that issue.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@backportbot-nextcloud backportbot-nextcloud bot added bug feature: call 📹 Voice and video calls feature: frontend 🖌️ "Web UI" client feature: WebRTC 🚡 WebRTC connection between browsers and/or mobile clients labels Aug 11, 2022
@backportbot-nextcloud backportbot-nextcloud bot added this to the 💖 Next Patch (23) milestone Aug 11, 2022
@nickvergessen nickvergessen merged commit 3d2fb59 into stable23 Aug 11, 2022
@nickvergessen nickvergessen deleted the backport/7673/stable23 branch August 11, 2022 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release bug feature: call 📹 Voice and video calls feature: frontend 🖌️ "Web UI" client feature: WebRTC 🚡 WebRTC connection between browsers and/or mobile clients
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants