Skip to content

Fix crash on RTCPeerConnection.setRemote/LocalDescription #10

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

Closed

Conversation

mininny
Copy link
Contributor

@mininny mininny commented Sep 3, 2021

No description provided.

@mininny
Copy link
Contributor Author

mininny commented Sep 7, 2021

Moved to #11

@mininny mininny closed this Sep 7, 2021
sf-jed-kyung pushed a commit that referenced this pull request Jul 7, 2025
This reverts commit c92fc76.

Reason for revert: Gerritt submitted patch set #10 not #12 - this broke

Original change's description:
> Assign supported codecs per section, not globally.
>
> This uses the PayloadTypeSuggester for PT disambiguation,
> ensuring that conficts are managed across the transport.
>
> Bug: webrtc:360058654
> Change-Id: Ic8f0569d0d96d97c13765169903929ec8a110104
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/383400
> Commit-Queue: Harald Alvestrand <[email protected]>
> Reviewed-by: Evan Shrubsole <[email protected]>
> Reviewed-by: Henrik Boström <[email protected]>
> Cr-Commit-Position: refs/heads/main@{#44284}

Bug: webrtc:360058654
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I16d0d92fb9a1ecc65eda060f81d41faefcca1e4c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/384000
Commit-Queue: Harald Alvestrand <[email protected]>
Reviewed-by: Danil Chapovalov <[email protected]>
Bot-Commit: [email protected] <[email protected]>
Reviewed-by: Mirko Bonadei <[email protected]>
Cr-Commit-Position: refs/heads/main@{#44285}
sf-jed-kyung pushed a commit that referenced this pull request Jul 7, 2025
This reverts commit 374b892.

Reason for revert: Racing CLs sorted out

Original change's description:
> Revert "Assign supported codecs per section, not globally."
>
> This reverts commit c92fc76.
>
> Reason for revert: Gerritt submitted patch set #10 not #12 - this broke
>
> Original change's description:
> > Assign supported codecs per section, not globally.
> >
> > This uses the PayloadTypeSuggester for PT disambiguation,
> > ensuring that conficts are managed across the transport.
> >
> > Bug: webrtc:360058654
> > Change-Id: Ic8f0569d0d96d97c13765169903929ec8a110104
> > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/383400
> > Commit-Queue: Harald Alvestrand <[email protected]>
> > Reviewed-by: Evan Shrubsole <[email protected]>
> > Reviewed-by: Henrik Boström <[email protected]>
> > Cr-Commit-Position: refs/heads/main@{#44284}
>
> Bug: webrtc:360058654
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Change-Id: I16d0d92fb9a1ecc65eda060f81d41faefcca1e4c
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/384000
> Commit-Queue: Harald Alvestrand <[email protected]>
> Reviewed-by: Danil Chapovalov <[email protected]>
> Bot-Commit: [email protected] <[email protected]>
> Reviewed-by: Mirko Bonadei <[email protected]>
> Cr-Commit-Position: refs/heads/main@{#44285}

Bug: webrtc:360058654
Change-Id: I80b59aab396abbc0afa60dab3516f2a22e822a5d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/383981
Reviewed-by: Mirko Bonadei <[email protected]>
Commit-Queue: Mirko Bonadei <[email protected]>
Commit-Queue: Harald Alvestrand <[email protected]>
Reviewed-by: Danil Chapovalov <[email protected]>
Cr-Commit-Position: refs/heads/main@{#44288}
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