Closed Bug 1494312 Opened 6 years ago Closed 6 years ago

Make mtransport API entirely async

Categories

(Core :: WebRTC: Networking, enhancement, P2)

60 Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: bwc, Assigned: bwc)

References

Details

Attachments

(3 files)

There are a few places where the mtransport API has a function that returns stuff when called on STS (eg; GetDefaultCandidates). This will need to be refactored to be async, since we're going to be serving this API vie IPC.
I think we'll end up using some cacheing for transport state, and keep MediaTransportHandler::GetState, but we need to make stats calls async, and also plumb a way for errors to get back in an async manner.
I will also remove nsresult returns that don't actually do anything besides trigger some logging on the signaling side of things.
No longer depends on: 1494311
Blocks: 1507861
Blocks: socket-proc-webrtc
No longer blocks: socket-proc
Pushed by bcampen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4185df9ce3e1 Part 1: Remove unused nsresult returns from MediaTransportHandler. r=mjf https://hg.mozilla.org/integration/autoland/rev/ca59ca222bd5 Part 2: Make the internal PC stats API based on MozPromise. r=mjf,mayhemer https://hg.mozilla.org/integration/autoland/rev/f28667d0d2cb Part 3: Expose RLog stuff via MediaTransportHandler. r=mjf
Depends on: 1419325
Blocks: 1419325
No longer depends on: 1419325
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: