Closed
Bug 1494312
Opened 6 years ago
Closed 6 years ago
Make mtransport API entirely async
Categories
(Core :: WebRTC: Networking, enhancement, P2)
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.
Assignee | ||
Comment 1•6 years ago
|
||
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.
Assignee | ||
Comment 2•6 years ago
|
||
I will also remove nsresult returns that don't actually do anything besides trigger some logging on the signaling side of things.
Assignee | ||
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Assignee | ||
Comment 7•6 years ago
|
||
Assignee | ||
Comment 8•6 years ago
|
||
Assignee | ||
Comment 9•6 years ago
|
||
Assignee | ||
Comment 10•6 years ago
|
||
Assignee | ||
Comment 11•6 years ago
|
||
Assignee | ||
Comment 12•6 years ago
|
||
Assignee | ||
Comment 13•6 years ago
|
||
Assignee | ||
Comment 14•6 years ago
|
||
Updated•6 years ago
|
Comment 16•6 years ago
|
||
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
Comment 17•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4185df9ce3e1
https://hg.mozilla.org/mozilla-central/rev/ca59ca222bd5
https://hg.mozilla.org/mozilla-central/rev/f28667d0d2cb
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•