875346 - Queue ondatachannel and onopen events until peer connection has been setup

Status: RESOLVED INCOMPLETE

(Core :: WebRTC: Networking, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Mozilla/5.0 (X11; Linux x86_64; rv:23.0) Gecko/20130522 Firefox/23.0 ID:20130522004016 CSet: 10738444bf15

At the moment this is not clear in the spec when ondatachannel and onopen events are getting fired. As what I have seen is that those can be fired even before the peer connection has been established. Please see the following test output:

00:44:49     INFO -  18193 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_dataChannel_basicAudio.html | Run step: PC_REMOTE_SET_LOCAL_DESCRIPTION
00:44:49     INFO -  18194 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_dataChannel_basicAudio.html | pcRemote: Register callbacks for 'ondatachannel' and 'onopen'
00:44:50     INFO -  18195 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_dataChannel_basicAudio.html | pcLocal_channel_0: 'onopen' event fired
00:44:50     INFO -  18196 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_dataChannel_basicAudio.html | pcRemote: 'ondatachannel' event fired for channel_0
00:44:50     INFO -  18197 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_dataChannel_basicAudio.html | Creating new DataChannelWrapper: pcRemote_channel_0
00:44:50     INFO -  18198 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_dataChannel_basicAudio.html | pcRemote_channel_0: 'onopen' event fired

It should be made clear in the spec how this should work. Filed the bug on behave of Randells request.

Comment 1

[Jason Smith [:jsmith]]

Probably isn't a blocker for 1.0 ship, although we need to resolve this eventually.

Comment 2

[Randell Jesup [:jesup] (needinfo me)]

No longer relevant - if there are still issues flagged by the test framework we can open a new bug.  Also the spec has had major updates since this was filed.