Closed Bug 834837 Opened 12 years ago Closed 11 years ago

Verify correct remote MediaStream tracks in mochitests for PeerConnection

Categories

(Core :: WebRTC, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: ekr, Assigned: drno)

References

(Blocks 1 open bug)

Details

(Whiteboard: [webrtc][blocking-webrtc-][tests])

Now that we have removed hints, this no longer works. Once bug 834835 is completed, we need to add tests.
Depends on: 834835
We probably should implement an automated test for this first in gum automation.
Whiteboard: [webrtc][blocking-webrtc+]
Summary: Verify correct tracks in mochitests for PeerConnection → Verify correct remote MediaStream tracks in mochitests for PeerConnection
Assignee: nobody → jsmith
Whiteboard: [webrtc][blocking-webrtc+] → [webrtc][blocking-webrtc+][blocked by bug 834835]
Putting this into blocking? until the other bug this is blocked on gets a blocking call. Unassigning for now until the blocking call is made - there's nothing I can do here at the moment.
Assignee: jsmith → nobody
Whiteboard: [webrtc][blocking-webrtc+][blocked by bug 834835] → [webrtc][blocking-webrtc?][blocked by bug 834835]
Making this blocking since Bug 834835 is now blocking.
Whiteboard: [webrtc][blocking-webrtc?][blocked by bug 834835] → [webrtc][blocking-webrtc+][blocked by bug 834835]
Assignee: nobody → jsmith
Blocks: pc-tests
Priority: -- → P2
We are still blocked here. Putting assignee back into the pool.
Assignee: jsmith → nobody
OS: Mac OS X → All
Hardware: x86 → All
Assignee: nobody → hskupin
If we're having assignees for all blockers even though we're blocked, let me put myself back on here on the list then.
Assignee: hskupin → jsmith
Whiteboard: [webrtc][blocking-webrtc+][blocked by bug 834835] → [webrtc][blocking-webrtc-][tests][blocked by bug 834835]
Depends on: 866514
Whiteboard: [webrtc][blocking-webrtc-][tests][blocked by bug 834835] → [webrtc][blocking-webrtc-][tests]
Depends on: 873888
Assignee: jsmith → nobody
Assignee: nobody → drno
QA Contact: jsmith → drno
This is probably easiest achieved by factoring out numTracks() in PCW_checkStats() in pc.js and check against a fixed number rather then comparing to the stats.
This will get implemented through the patch for Bug 873049.
Depends on: 873049
Fixed through the fix in bug 873049 which landed on m-c.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.