1635922 - Support RTCPeerConnection.connectionstatechange

Status: RESOLVED DUPLICATE of bug 1265827

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

Description

[Joel Maher ( :jmaher ) (UTC -8)]

this testcase in web-platform/tests /webrtc/protocol/dtls-fingerprint-validation.html is expected timeout on all platforms.

running it locally on my windows 10 machine I see it times out on this test:
Connection fails if one side provides a wrong DTLS fingerprint

I am not very familiar with the webrtc tests.

:dminor, could you take a look at this and help determine if this is an issue with tooling, the test, the product, or this isn't supported by Firefox.

Comment 1

[Dan Minor [:dminor]]

Byron has been keeping an eye on our web-platform tests.

Comment 2

[Byron Campen [:bwc]]

We don't support connectionstatechange (as opposed to iceconnectionstatechange) right now, which is why this test is timing out.

Comment 3

[Byron Campen [:bwc]]

Attachment: Bug 1635922: Add bug link to ini file. r?jib

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

do you plan to support this in the short/medium term? I can mark this as:
implementation-status: not-implementing # no plans
implementation-status: backlog # plans to support this in the future

Comment 5

[Byron Campen [:bwc]]

Not in the short/medium term, no, but we should get to it eventually.

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

lets make this 'backlog', that is easier.

I see other instances of tests which might be similar:
https://searchfox.org/mozilla-central/search?q=connectionstatechange&path=testing%2Fweb-platform%2Ftests%2F&case=false&regexp=false

some I spot checked were already disabled, it would be nice to put them all in the same bug reference.

here is an example of adding backlog:
https://searchfox.org/mozilla-central/search?q=implementation-status%3A+backlog&path=

it is best to add a bug (as you have here in the existing patch) so we can reference it.

I am happy to do it, or help review. If I do it, maybe indicate which of the tests in the above searchfox link should be included as well.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3321fb5c6334
Add bug link to ini file. r=jib

Comment 8

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/3321fb5c6334

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:bwc, maybe it's time to close this bug?

Comment 10

[Byron Campen [:bwc]]

Nope. Patch just added a pointer to this bug, the bug is not actually fixed.

Comment 11

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:bwc, maybe it's time to close this bug?

Comment 12

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:bwc, maybe it's time to close this bug?

Comment 13

[Byron Campen [:bwc]]

This breaks a couple of wpt for other things.