Skip to content

android: Allow for skipping checking the audio playstate if needed#129

Merged
davidliu merged 1 commit intom125_releasefrom
dl/tenten_webrtc_audio_track_assert
Jul 9, 2024
Merged

android: Allow for skipping checking the audio playstate if needed#129
davidliu merged 1 commit intom125_releasefrom
dl/tenten_webrtc_audio_track_assert

Conversation

@davidliu
Copy link
Copy Markdown
Contributor

@davidliu davidliu commented Jul 8, 2024

Pausing/stopping the audio track can lead to a race condition against the AudioTrackThread due to this assert. Normally this is fine since directly pausing/stopping isn't possible, but user is using reflection to workaround another audio issue (muted participants still have a sending audio stream which keeps the audio alive, affecting global sound if in the background).

Not a full fix, as would like to manually control the audio track directly (needs a bigger fix to handle proper synchronization before allowing public access), but this will work through reflection (user takes responsibility for usage).

Not a full fix, as would like to manually control the audio track directly,
but this will work through reflection (user takes responsibility for usage)
@davidliu davidliu merged commit 7ddfc43 into m125_release Jul 9, 2024
@davidliu davidliu deleted the dl/tenten_webrtc_audio_track_assert branch July 9, 2024 03:31
vigazzola pushed a commit to vigazzola/webrtc-sdk that referenced this pull request Nov 17, 2025
Co-authored-by: Anka <runner@Mac-1756902489761.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants