Skip to content

fix: ignore duplicate native signaling events#26

Merged
mertushka merged 1 commit into
mainfrom
fix-signalingstatechange-duplicates
Jun 24, 2026
Merged

fix: ignore duplicate native signaling events#26
mertushka merged 1 commit into
mainfrom
fix-signalingstatechange-duplicates

Conversation

@mertushka

Copy link
Copy Markdown
Member

Summary

Fix late duplicate native signalingstatechange notifications when the JavaScript facade has already exposed the same description-derived signaling state.

This keeps repeated remote offers from reporting an extra signalingstatechange after the first have-remote-offer transition.

Impact

  • Adds focused regression coverage for a late native duplicate after setRemoteDescription(offer).
  • No public API, native addon, WPT selection, or documentation changes.

@mertushka mertushka marked this pull request as ready for review June 24, 2026 13:02
@mertushka mertushka merged commit 02a0408 into main Jun 24, 2026
19 checks passed
@mertushka mertushka deleted the fix-signalingstatechange-duplicates branch June 24, 2026 13:03
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.

1 participant