1805480 - RTCDtlsTransport.state is not being marked closed on transports we don't end up using

Status: RESOLVED FIXED

(Core :: WebRTC: Networking, defect, P2)

Description

[Byron Campen [:bwc]]

This is causing a failure in bundle.https.html

We mark these closed if the transceiver is closed, but not when we stop using them because of bundle negotiation or ICE restart. Not a huge bug, but we should fix.

Comment 1

[Byron Campen [:bwc]]

https://wpt.live/webrtc/protocol/bundle.https.html?interop-2026

Comment 2

[Byron Campen [:bwc]]

This might interact with bug 1339203, because that also deals with how many transports we open.

Comment 3

[Byron Campen [:bwc]]

Attachment: Bug 1805480: Fire closed updates for transports that are torn down, even if they were never active. r?mjf

Comment 4

[Byron Campen [:bwc]]

Attachment: Bug 1805480: Remove the cache-clearing side effect of TS_NONE. r?mjf

This is such a tiny amount of information to keep around, much smaller
than other things (eg; SDP history).

Depends on D304311

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/7bade2a8d77a
https://hg.mozilla.org/integration/autoland/rev/58054f912ab5
Fire closed updates for transports that are torn down, even if they were never active. r=mjf
https://github.com/mozilla-firefox/firefox/commit/496c6dadb87d
https://hg.mozilla.org/integration/autoland/rev/2e7df98a28a3
Remove the cache-clearing side effect of TS_NONE. r=mjf

Comment 6

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/58054f912ab5
https://hg.mozilla.org/mozilla-central/rev/2e7df98a28a3