Open Bug 825562 Opened 12 years ago Updated 5 months ago

Creating a PeerConnection object and checking the ice connection state - should be initialized to an acceptable starting ice connection state, but instead is "undefined"

Categories

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

defect

Tracking

()

People

(Reporter: jsmith, Unassigned)

References

Details

(Whiteboard: [WebRTC][blocking-webrtc-][spec-issue])

Attachments

(1 file)

2.01 KB, application/x-zip-compressed
Details
Steps:

1. Load the attached test case
2. Create a peer connection object
3. View the peer connection attributes

Expected:

The ice state should be initialized - not entirely 100% sure which one as the spec contradicts itself (http://dev.w3.org/2011/webrtc/editor/webrtc.html#idl-def-RTCIceState) in two points. Someone will have to clarify what's the right starting value.

Actual:

The ice state is undefined.
Attached file Test Case
Assignee: nobody → ekr
Priority: -- → P2
Whiteboard: [WebRTC], [blocking-webrtc+]
Flags: in-testsuite?
Whiteboard: [WebRTC], [blocking-webrtc+] → [WebRTC][blocking-webrtc-][spec-issue]
Note - this should be titled - "iceConnectionState"
Summary: Creating a PeerConnection object and checking the ice state - should be initialized to an acceptable starting ice state, but instead is "undefined" → Creating a PeerConnection object and checking the ice connection state - should be initialized to an acceptable starting ice connection state, but instead is "undefined"
Component: WebRTC → WebRTC: Networking
Byron, is this still relevant?
Flags: needinfo?(docfaraday)
I'll look into it once I'm back.
Ok, this seems to be handled in PeerConnection.js. The getter for the ice connection state does not pull the data from c++, it is cached in js based on state change callbacks, and is indeed not given an initial value.
Flags: needinfo?(docfaraday)
backlog: --- → webRTC+
Rank: 25
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3

Sorry, there was a problem with the detection of inactive users. I'm reverting the change.

Assignee: nobody → ekr
Severity: normal → S3

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: ekr → nobody
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: