We need code that sanity-checks and diagnoses problems in a PeerConnection

NEW
Assigned to

Status

()

Core
WebRTC
P3
normal
Rank:
24
3 years ago
3 months ago

People

(Reporter: bwc, Assigned: bwc)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox41 affected)

Details

(Whiteboard: [test])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

3 years ago
Such a thing would make it a little easier to write unit-tests. If exposed to JS, it would be a very useful enhancement to mochitests (or any other automated browser test), and would allow the developers of webrtc services to diagnose problems much more effectively. If exposed on about:webrtc, it would allow users/testers to give us much better bug reports.

I've thought about basing this on the stats API, but the downside to that is that it would not be usable in unit-tests, and would require lots more work since we want to be able to dig very deep into the internal state of things like the ICE stack, pipeline code, conduit code, etc. However, any diagnostics we put in place could be exposed via the stats API as needed.
(Assignee)

Updated

3 years ago
Assignee: nobody → docfaraday
(Assignee)

Comment 1

3 years ago
Created attachment 8607187 [details]
MozReview Request: bz://1165140/bwc

/r/8955 - Bug 1165140: (WIP) PeerConnection sanity check code.

Pull down this commit:

hg pull -r 61d70eee60951f68242c3b68e1ba7a3bd193f0bb https://reviewboard-hg.mozilla.org/gecko/

Updated

3 years ago
backlog: --- → webRTC+
Rank: 24
Priority: -- → P2
Whiteboard: [test]
(Assignee)

Comment 2

3 years ago
Comment on attachment 8607187 [details]
MozReview Request: bz://1165140/bwc

/r/8955 - Bug 1165140: (WIP) PeerConnection sanity check code.

Pull down this commit:

hg pull -r 13a1de4ea0eca5d5c73d753c14205a3c09b6d36a https://reviewboard-hg.mozilla.org/gecko/
(Assignee)

Comment 3

3 years ago
Comment on attachment 8607187 [details]
MozReview Request: bz://1165140/bwc

/r/8955 - Bug 1165140: (WIP) PeerConnection sanity check code.

Pull down this commit:

hg pull -r d1fae7beb949e8340dc0924a9dee8a2b0c9334b5 https://reviewboard-hg.mozilla.org/gecko/
(Assignee)

Comment 4

3 years ago
Comment on attachment 8607187 [details]
MozReview Request: bz://1165140/bwc

/r/8955 - Bug 1165140: (WIP) PeerConnection sanity check code.

Pull down this commit:

hg pull -r 307a730db7b2ab495c1fc0dbb8860157aad5d826 https://reviewboard-hg.mozilla.org/gecko/
(Assignee)

Comment 5

3 years ago
Comment on attachment 8607187 [details]
MozReview Request: bz://1165140/bwc
Attachment #8607187 - Attachment is obsolete: true
(Assignee)

Comment 6

3 years ago
Created attachment 8620304 [details]
MozReview Request: Bug 1165140: (WIP) PeerConnection sanity check code.
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.