Skip to content

Commit

Permalink
Merge branch 'main' into noconnectionstateclose
Browse files Browse the repository at this point in the history
  • Loading branch information
jan-ivar authored Jul 13, 2023
2 parents 46a3411 + bc552d5 commit 2b3d707
Show file tree
Hide file tree
Showing 4 changed files with 30 additions and 12 deletions.
9 changes: 9 additions & 0 deletions amendments.json
Original file line number Diff line number Diff line change
Expand Up @@ -493,6 +493,15 @@
"id": 33
}
],
"removetrack-algo": [
{
"description": "Make removeTrack() a no-op after transceiver.stop()",
"pr": 2875,
"type": "correction",
"status": "candidate",
"id": 34
}
],
"connectionstate-algo": [
{
"description": "Don't fire connectionstatechange on pc.close()",
Expand Down
2 changes: 1 addition & 1 deletion base-rec.html
Original file line number Diff line number Diff line change
Expand Up @@ -8195,7 +8195,7 @@ <h4 id="methods-5">
When the <a data-link-type="idl" href="#dom-rtcpeerconnection-removetrack" class="internalDFN" id="ref-for-dom-rtcpeerconnection-removetrack-4"><code><code>removeTrack</code></code></a> method is invoked, the user agent
<em class="rfc2119">MUST</em> run the following steps:
</p>
<ol>
<ol id="removetrack-algo">
<li class="no-test-needed">
<p>
Let <var>sender</var> be the argument to <a data-link-type="idl" href="#dom-rtcpeerconnection-removetrack" class="internalDFN" id="ref-for-dom-rtcpeerconnection-removetrack-5"><code><code>removeTrack</code></code></a>.
Expand Down
7 changes: 4 additions & 3 deletions respec-w3c-common.js

Large diffs are not rendered by default.

24 changes: 16 additions & 8 deletions webrtc.html
Original file line number Diff line number Diff line change
Expand Up @@ -8040,7 +8040,7 @@ <h2>
When the {{removeTrack}} method is invoked, the user agent
MUST run the following steps:
</p>
<ol class=algorithm>
<ol class=algorithm id="removetrack-algo">
<li class="no-test-needed">
<p>
Let <var>sender</var> be the argument to {{removeTrack}}.
Expand All @@ -8067,6 +8067,18 @@ <h2>
{{InvalidAccessError}}.
</p>
</li>
<li class="no-test-needed">
<p>
Let <var>transceiver</var> be the {{RTCRtpTransceiver}}
object corresponding to <var>sender</var>.
</p>
</li>
<li data-tests="RTCPeerConnection-removeTrack.https.html">
<p>
If <var>transceiver</var>.{{RTCRtpTransceiver/[[Stopping]]}} is
<code>true</code>, abort these steps.
</p>
</li>
<li data-tests="RTCPeerConnection-removeTrack.https.html">
<p>
Let <var>senders</var> be the result of executing the
Expand All @@ -8093,12 +8105,6 @@ <h2>
Set <var>sender</var>.{{RTCRtpSender/[[SenderTrack]]}} to null.
</p>
</li>
<li class="no-test-needed">
<p>
Let <var>transceiver</var> be the {{RTCRtpTransceiver}}
object corresponding to <var>sender</var>.
</p>
</li>
<li data-tests="RTCPeerConnection-removeTrack.https.html">
<p>
If <var>transceiver</var>.{{RTCRtpTransceiver/[[Direction]]}} is
Expand Down Expand Up @@ -12693,7 +12699,9 @@ <h3>
</td>
<td>
The {{RTCIceTransport}} has received at least one remote
candidate and is checking candidate pairs and has either
candidate (by means of {{RTCPeerConnection/addIceCandidate()}} or discovered as a
peer-reflexive candidate when receiving a STUN binding
request) and is checking candidate pairs and has either
not yet found a connection or consent checks [[!RFC7675]]
have failed on all previously successful candidate pairs.
In addition to checking, it may also still be gathering.
Expand Down

0 comments on commit 2b3d707

Please sign in to comment.