850467 - Create some mochitests for verifying that onXXX handlers fire at the correct time with Data Channels

Status: RESOLVED FIXED

(Core :: WebRTC, defect, P2)

Description

[Jason Smith [:jsmith]]

Create some mochitests that verify that the onXXX handlers with Data Channels fire at the time we expect them to fire. Such onXXX handlers you'll want to analyze includes:

- onopen
- onerror
- onclose
- onmessage

Comment 1

[Jason Smith [:jsmith]]

Important cause we need at least one test that makes sure events fire at the *right* time. Into the ? pool for blocking analysis.

Comment 2

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

All of those events will be covered by my first data connection test which will most likely be bug 796895 (if I hit no other road blockers).

Comment 3

[Jason Smith [:jsmith]]

Sounds like this should block if bug 796895 blocks then. Although fixing one will fix the other.

Comment 4

[Jason Smith [:jsmith]]

Since whimboo is working on the other bug, I'm going to set you as the assignee working on this one as well by caveat.

Comment 5

[Jason Smith [:jsmith]]

bug 796894 fixes most of these cases, except the onerror case.

Comment 6

[Jason Smith [:jsmith]]

(In reply to Jason Smith [:jsmith] from comment #5)
> bug 796894 fixes most of these cases, except the onerror case.

Turns out we injected checks for onerror as well, although for the case of when we shouldn't expect it to fire. However, that covers enough of this to call this fixed by bug 796894. If there are specific cases we haven't covered yet, then let's file specific bugs for those cases.