Support RTCPeerConnection.connectionstatechange
Categories
(Core :: WebRTC: Networking, defect, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox-esr68 | --- | wontfix |
| firefox75 | --- | wontfix |
| firefox76 | --- | wontfix |
| firefox77 | --- | wontfix |
| firefox78 | --- | fix-optional |
People
(Reporter: jmaher, Assigned: bwc)
References
Details
(Keywords: parity-chrome, parity-edge, parity-safari)
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
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•5 years ago
|
||
Byron has been keeping an eye on our web-platform tests.
|
Assignee | |
Comment 2•5 years ago
|
||
We don't support connectionstatechange (as opposed to iceconnectionstatechange) right now, which is why this test is timing out.
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
||
Updated•5 years ago
|
|
|
Assignee | |
Updated•5 years ago
|
|
Reporter | |
Comment 4•5 years ago
|
||
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
|
|
Assignee | |
Comment 5•5 years ago
|
||
Not in the short/medium term, no, but we should get to it eventually.
|
|
Reporter | |
Comment 6•5 years ago
|
||
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®exp=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 8•5 years ago
|
||
| bugherder | ||
|
||
Comment 9•4 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:bwc, maybe it's time to close this bug?
|
|
Assignee | |
Comment 10•4 years ago
|
||
Nope. Patch just added a pointer to this bug, the bug is not actually fixed.
|
|
||
Comment 11•3 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:bwc, maybe it's time to close this bug?
|
|
Assignee | |
Updated•3 years ago
|
|
|
||
Comment 12•3 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:bwc, maybe it's time to close this bug?
|
|
Assignee | |
Updated•3 years ago
|
|
|
Assignee | |
Comment 13•2 years ago
|
||
This breaks a couple of wpt for other things.
|
|
Assignee | |
Updated•2 years ago
|
Description
•