The default bug view has changed. See this FAQ.

Accept candidate messages that are zero numbered

RESOLVED FIXED in mozilla20

Status

()

Core
WebRTC: Signaling
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: ekr, Assigned: jib)

Tracking

unspecified
mozilla20
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [WebRTC],[blocking-webrtc-interop],[blocking-webrtc+][qa-])

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Reporter)

Comment 1

4 years ago
Created attachment 687554 [details] [diff] [review]
Accept candidate messages that are zero numbered
(Reporter)

Comment 2

4 years ago
http://dev.w3.org/2011/webrtc/editor/webrtc.html#rtcicecandidate-type

Requires that candidates are 0-based indexed. We use 1-based indexing internally for m-lines.
This should be fixed in the JS.

Here is a prototype patch that needs to be cleaned-up and landed.
(Reporter)

Updated

4 years ago
Whiteboard: [WebRTC],[blocking-webrtc-interop]
(Assignee)

Updated

4 years ago
Assignee: ekr → jib
(Assignee)

Comment 3

4 years ago
Created attachment 692999 [details] [diff] [review]
Accept candidate messages that are zero numbered
(Assignee)

Updated

4 years ago
Attachment #687554 - Attachment is obsolete: true
Whiteboard: [WebRTC],[blocking-webrtc-interop] → [WebRTC],[blocking-webrtc-interop],[blocking-webrtc+]
(Assignee)

Updated

4 years ago
Attachment #692999 - Flags: review?(rjesup)

Updated

4 years ago
Attachment #692999 - Flags: review?(rjesup) → review+
(Assignee)

Updated

4 years ago
Attachment #692999 - Flags: checkin?(rjesup)
https://hg.mozilla.org/integration/mozilla-inbound/rev/08e0e0862006
Target Milestone: --- → mozilla20

Updated

4 years ago
Attachment #692999 - Flags: checkin?(rjesup) → checkin+
https://hg.mozilla.org/mozilla-central/rev/08e0e0862006
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Updated

4 years ago
Whiteboard: [WebRTC],[blocking-webrtc-interop],[blocking-webrtc+] → [WebRTC],[blocking-webrtc-interop],[blocking-webrtc+][qa-]
You need to log in before you can comment on or make changes to this bug.