1494301 - Write a single API surface for mtransport

Status: RESOLVED FIXED

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

Description

[Byron Campen [:bwc]]

Right now, the mtransport API is inconsistently encapsulated; PeerConnectionMedia wraps some functionality, but the MediaPipeline code uses TransportLayer/TransportFlow directly. We should have a single class that exposes the API we need, and no other code should use any of the mtransport classes (except some of the threading utility stuff maybe). This will get us closer to being able to move mtransport into a separate process.

Comment 1

[Byron Campen [:bwc]]

Attachment: Bug 1494301: Single API for mtransport.

Comment 2

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=df675dca1772a7b5aa8da13a6e5c97c5032b5e55

Comment 3

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9d633733a959dff587402fced751a850445a78b0

Comment 4

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f9ef8c4e327715a974ddbb7cd015f9d73503a386

Comment 5

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=da4be746f0b3dd6777c557b787015bbbdab5f3f7

Comment 6

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=79a23e20082cacede1d0fd4c062df1ca6343cdd9

Comment 7

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b4f878e9ab9d28951a32e047c3c112fca536df8d

Comment 8

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6aa52e5936ae347cf5f98e9dcdaf626a701649a6

Comment 9

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e3f68f8c1a64ef46109d2d014bb426eac2a2cd41

Comment 10

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=637b1752d9906f3e08c435ca6e4bcd234892f63d

Comment 11

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=bbea2ad0f74588c55e9dc80acbb3b781097dd430

Comment 12

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=31184922710215e013b75b4c41d94352bbc35b50

Comment 13

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b023744e1d820054c87c04ec3725bdada6fe7350

Comment 14

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=36a6d723929359ecd1079eeb3e37327bc104f257

Comment 15

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=169858ddfee2c22898aa19f027720d9529ec1326

Comment 16

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d1ec7cfaba3406d6e96549f3e1562e90aab976fa

Comment 17

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f61f335f53b6dcc1b8444450113a13fb252a48f2

Comment 18

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3e6dd84336798deb3e63d27c26ce0255d8a2d557

Comment 19

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=90b0d9003f0126ed6ec994c0934b3446df8d8db5

Comment 20

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=39b761f1e8772c3ee5c20ad2c14fcd4454ec19d8

Comment 21

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fe48d6a8b31a24e7cb73dcb5056e28def00470be

Comment 22

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7aa9958e22e3aa87b063d7de18e7845271676b0f

Comment 23

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7c41dd926a38f1411201413be3b39d43be83d59e

Comment 24

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e5a33a4418601dca1150379359f71760311b0ab6

Comment 25

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=61980e8846992db2d7e4794e5bf81056992b3785

Comment 26

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a803bc60ad5a8c9e848abb03622057a93ce23fa2

Comment 27

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b626ac9416dfe62836716d8f2db8b9aa4981ea3c

Comment 28

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=bdc8cd22fd44597bdabea6c140be8651284da6a0

Comment 29

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b5acb81ef5172750ca85e473f1111c6e2dce6b95

Comment 30

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b945f85bd67ef33c13b54e82e17fa8d78af1711

Comment 31

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/11819394462e
Single API for mtransport. r=mjf

Comment 32

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/11819394462e