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

Name and export the RTCIceTransport state update algo #2905

Merged
merged 3 commits into from
Nov 16, 2023

Conversation

sam-vi
Copy link
Contributor

@sam-vi sam-vi commented Nov 6, 2023

Fixes: #2904.

The RTCIceTransport algorithm for a change of the selected candidate pair or RTCIceTransportState is referenced in webrtc-extensions currently without a proper link because it is not exported from webrtc-pc. So name and export the algorithm.


Preview | Diff

Fixes: w3c#2904.

The RTCIceTransport algorithm for a change of the selected candidate
pair or RTCIceTransportState is referenced in webrtc-extensions
currently without a proper link because it is not exported from
webrtc-pc. So name and export the algorithm.
@sam-vi sam-vi marked this pull request as ready for review November 6, 2023 14:23
@sam-vi
Copy link
Contributor Author

sam-vi commented Nov 6, 2023

Re. the failing check, I believe this change qualifies as Editorial rather than an amendment.

webrtc.html Outdated
Comment on lines 12163 to 12164
changing. The <dfn class="export" data-lt="change of selected candidate pair or state">change of the selected pair and
{{RTCIceTransportState}}</dfn> are related
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this needs to go in the paragraph below where the algorithm is invoked. E.g.

      When the [= ICE Agent =] indicates that an {{RTCIceTransport}} has
      changed either the selected candidate pair, the
      {{RTCIceTransportState}} or both, the user agent MUST queue a task
      that runs these [steps to change pair and state] algorithm:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@henbos henbos merged commit 1d0deef into w3c:main Nov 16, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Name and export the RTCIceTransport algorithm for change in selected pair or state
4 participants