Create Mochitest for Video only with data channel connection (send/receive/chat/disconnect)

RESOLVED FIXED

Status

()

Core
WebRTC
P2
normal
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

(Blocks: 1 bug)

18 Branch
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [WebRTC][blocking-webrtc-][tests], URL)

(Assignee)

Description

5 years ago
As part of the initial landing of the WebRTC feature some smoketests should ensure the basics for the Peer Connection feature are working. This bug covers the request for the following test:

Ensure a local video only connection and the chat works:

* The connection can be established
* Packages can be send/received
* Data can be send/received (chat)
* Disconnect

Updated

5 years ago
Whiteboard: [WebRTC], [blocking-webrtc+]

Updated

5 years ago
Whiteboard: [WebRTC], [blocking-webrtc+] → [WebRTC], [blocking-webrtc-]
(Assignee)

Comment 1

5 years ago
Am I right that this request would also need bug 784515 first?
Depends on: 784515
Whiteboard: [WebRTC], [blocking-webrtc-] → [WebRTC][blocking-webrtc-][needs bug 784515]
Version: 14 Branch → 18 Branch

Updated

5 years ago
Blocks: 767072
No longer blocks: 795383
The meta bug was indicated as blocking, but when we originally triaged this, we said not blocking. Which is it? Block or No Block?
Whiteboard: [WebRTC][blocking-webrtc-][needs bug 784515] → [WebRTC]

Updated

5 years ago
Summary: Create Mochitest for Video only with data connection (send/receive/chat/disconnect) → Create Mochitest for Video only with data channel connection (send/receive/chat/disconnect)

Updated

5 years ago
Whiteboard: [WebRTC] → [WebRTC][blocking-webrtc+]
Assignee: nobody → hskupin
Priority: -- → P2
Henrik - What's blocking you from finishing this? Also, if you have a partially completed patch here, can you stick up a WIP here?

Won't repeat this comment for other Data Channel tests, but this comment also applies to bug 796891, bug 796895, and bug 850467.
Flags: needinfo?(hskupin)
(Assignee)

Comment 4

5 years ago
I thought that I have uploaded a WIP before I headed to vacation 1.5 weeks ago. But it seems to be not the case. At least we both cannot find it. :( Now it's even more disappointing that I lost this patch while transitioning over to my new machine. The patch in mq on my new machine is the testcase as attached to the datachannel leak bug. I don't like to say that but it means I have to start from scratch. Most likely I will flip to use the git repo to be able to push to my own fork on github. Sorry.
Flags: needinfo?(hskupin)
Did you upload it on bug 851307?  That's what I've been using

Updated

5 years ago
Whiteboard: [WebRTC][blocking-webrtc+] → [WebRTC][blocking-webrtc-][tests]
(Assignee)

Comment 6

5 years ago
(In reply to Randell Jesup [:jesup] from comment #5)
> Did you upload it on bug 851307?  That's what I've been using

No, it's missing all the changes to the pc.js module. So it's of a little help. Thanks.
This being fixed by bug 796894.
Depends on: 796894

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.