Skip to content
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

Dominant Speaker logic mutes all other participants in Chrome only #741

Open
andygikling opened this issue Aug 19, 2022 · 3 comments
Open
Labels
bug Something isn't working

Comments

@andygikling
Copy link

andygikling commented Aug 19, 2022

Describe the bug
On Firefox, with Dominant Speaker = standard, 3 people in the room, when I listen to two remote participants talking at the same time, the Dominant Speaker does focus the louder talking person (for the most part) but you can still hear the other speaker in the background talking at the same time.

The problem is, on Chrome and Edge, when using the exact same server with the same connection settings, when the Dominate Speaker gets video focus on the screen, all other participants are completely muted (even if they are talking).

With this bug, on Chrome and Edge, you can only hear one person talking at a time. This is a huge problem for Group Room types. This does not happen on Firefox, Safari and Android...

To Reproduce
Steps to reproduce the behavior:

Using twilio-video-app-react v0.7.0. Have three participants in a Group room.

On Chrome have the two other participants begin talking at the same time. Observe that you can only hear the Dominant Speaker, and the other participant is 100% muted, even though their mic audio-level indicator shows they are also speaking.

On Firefox or Android, repeat the Chrome test above. Observe that you can clearly hear the Dominant Speaker, but also you can still hear the other participant speaking in the background.

Expected behavior
Non Dominant Speaker should not be completely muted in Chrome and Edge. This is necessary for conversations with multiple people in a Group room setting. Only hearing one audio track at a time is a show stopper for Group calls. It works correctly and as expected in Firefox, Safari and Android.

Screenshots
NA

Environment:

  • OS: [Windows 10 and Ubuntu 22.04]
  • Browser:
    Chrome Version 103.0.5060.134 (Official Build) (64-bit) broken
    Edge Version 104.0.1293.25 (Official build) beta (64-bit) broken
    Forefox 103.0.2 (64-bit) working
  • App Version: [0.7.0]
  • Twillio Video SDK [2.22.0]
  • Node.js version: [12.12.26]

Additional context
Playing with all possible Connection Settings when joining the call doesn't appear to fix the issue.
Tested these non-default settings in Connection Settings:

  • Dominant Speaker Priorety = low
  • Client Track Switch Off Control = Manual
  • Track Switch Off Mode = Disabled
@andygikling andygikling added the bug Something isn't working label Aug 19, 2022
@andygikling
Copy link
Author

andygikling commented Aug 19, 2022

@rjunsk Is this bug ticket related to your changes in 4565ab6? Seems like it might be.

@timmydoza or similarly is your change in 10cfac9 related?

@timmydoza
Copy link
Contributor

Hey @andygikling thanks for the issue! I agree that this is a big problem.

Yes - this does sound like the issue that we recently encountered with the bug that was introduced in Chrome 103. The issue was - that if you stop the clone of a MediaStreamTrack, Chrome 103 would also stop the MediaStreamTrack that the clone was cloned from. It affected some users of the twilio-video SDK as well: twilio/twilio-video.js#1811

I think that this commit here should have fixed the issue: 5f3a6f2. Can you make sure that these changes are in your AudioLevelIndicator.tsx file? I see that the fix should be included in the 0.7.0 version, which makes me a little concerned that it isn't working for you. I hope it isn't a new problem in Chrome. Please let me know if your AudioLevelIndicator.tsx file is up to date with the same file on master and we'll go from there.

@fahmyfarahat
Copy link

Is that issue still exist?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants