1039655 - [meta] Support TURN TCP on FxOS/e10s for WebRTC

Status: RESOLVED FIXED

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

Description

[Randell Jesup [:jesup] (needinfo me)]

We want to implement TURN TCP on FxOS (which wasn't feasible to do in the initial TURN TCP implementation).

Info from Honza about interfaces to use for implementing TURN TCP in FxOS (TCP use from the child):

http://mxr.mozilla.org/mozilla-central/source/dom/network/interfaces/nsITCPSocketChild.idl
XPCOM usage from JS http://mxr.mozilla.org/mozilla-central/source/dom/network/src/TCPSocket.js#583

Comment 1

[Patrick Wang (Chih-Kai Wang) (:kk1fff)]

I have WIP patches on bug 950660 and bug 951677, but they haven't been updated for a while. I can be back with them in one or two months.

Comment 2

[Maire Reavy [:mreavy]]

This is not something we need for v2.1

Comment 3

[Nils Ohlmeier [:drno]]

From comments like this:
https://dxr.mozilla.org/mozilla-central/source/media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp#596
it looks like TURN TCP is not supported on e10s either. I guess FxOS and e10s are in the same in this case.

Comment 4

[Nils Ohlmeier [:drno]]

Looks like I'm not the only one confused about the interdependency of bug 1039655 and bug 950660: the patch in bug 1053454 which disabled TURN TCP for e10s reference in one file bug 1039655, but bug 950660 in the other...

Comment 5

[Maire Reavy [:mreavy]]

Fixing the dependencies (as Nils notes, they were backward).  After chatting with bwc, drno and jesup yesterday, I/we believe that resolving bug 950660 should be sufficient for getting TURN TCP under e10s to work.  Jesup has taken over bug 950660; he believes he can un-rot and update Patrick's patches fairly quickly and get them up for review by Monday.