Closed
Bug 1576893
Opened 6 years ago
Closed 6 years ago
webrtc/RTCPeerConnection-iceConnectionState.https.html has an invalid test
Categories
(Core :: WebRTC: Signaling, defect, P2)
Core
WebRTC: Signaling
Tracking
()
RESOLVED
FIXED
mozilla71
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: bwc, Assigned: bwc)
References
Details
Attachments
(1 file)
The "Responder ICE connection state behaves as expected" test-case has been marked as intermittent on linux and android. Need to figure out why this is.
Assignee | ||
Updated•6 years ago
|
Type: defect → task
Assignee | ||
Comment 1•6 years ago
|
||
This looks like a bug in the test-case to me; it waits for ICE to complete on pc1, but not pc2, then asserts that pc2 has already reached ICE complete. The test case needs to wait for pc2 to reach ICE complete also.
Assignee | ||
Updated•6 years ago
|
Summary: webrtc/RTCPeerConnection-iceConnectionState.https.html has an inconsistent test → webrtc/RTCPeerConnection-iceConnectionState.https.html has an invalid test
Assignee | ||
Comment 4•6 years ago
|
||
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Looks like this change was backed out, but it will likely re-land. We'll have to wait for that to happen before landing this.
Comment hidden (Intermittent Failures Robot) |
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7134186c04b2
Wait for ICE to complete on pc2 before checking ICE state transitions. r=jib
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/18782 for changes under testing/web-platform/tests
Comment 11•6 years ago
|
||
The patch conflicted with the changes here: https://hg.mozilla.org/integration/autoland/rev/dcf08f0a3f22f0ead12e75e7bdda56c573dac71c so I rebased and landed it. Byron, if it's not ok with you I'll back it out.
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Comment hidden (Intermittent Failures Robot) |
![]() |
||
Comment 14•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Upstream PR merged by moz-wptsync-bot
![]() |
||
Updated•6 years ago
|
status-firefox71:
--- → fixed
Target Milestone: mozilla70 → mozilla71
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•