gecko-dev/media/mtransport
Nicholas Nethercote 9252435548 Bug 1410794 (attempt 2) - Change some |string| occurrences in nsIPrefBranch.idl to |ACString|. r=erahm.
This makes the code nicer. In particular, it removes many getter_Copies()
calls. The patch also converts a lot of nsCStrings to nsAutoCString, which will
avoid heap allocation in the common case.

The patch also renames PREF_CopyCharPref() as PREF_GetCStringPref(), because
it's actually getting a string, not a char, and that matches the existing
GetCString() and GetDefaultCString() methods. Correspondingly, it also renames
PREF_SetCharPref() as PREF_SetCStringPref().

The |aPrefName| arguments in nsIPrefBranch.idl remain as |string| because they
almost always involve passing in C string literals, and passing "foo" is much
nicer than passing NS_LITERAL_CSTRING("foo").

It's worth noting that early versions of this patch used |AUTF8String| instead
of |ACString|. But it turns out that libpref stores prefs internally as Latin1.
And |ACString| is compatible with Latin1 but |AUTF8String| isn't, because
non-ASCII Latin1 strings are not valid UTF-8!

MozReview-Commit-ID: D3f7a1Vl1oE

--HG--
extra : rebase_source : e6e4b15d6d210cfd93686f96400281f02bd1d06b
2017-10-27 10:30:33 +11:00
..
build Bug 1345511 - pt 2 - add IPC mechanism for getting stun addrs on main process. r=bwc 2017-03-21 19:59:05 -05:00
fuzztest Bug 1373843: Add a libfuzzer test for the stun parser; r=decoder 2017-06-16 16:01:19 -07:00
ipc Bug 1343743 - Part6: Pass a labeled event target to StunAddrsRequestChild's constructor., r=bwc 2017-06-12 00:25:00 +02:00
test Bug 1404198: Part 2g - Switch to NS_NewTimer* in media. r=njn 2017-10-15 23:14:33 -07:00
third_party Bug 1403346 - Replace all uses of ALLOW_COMPILER_WARNINGS with a template, remove ALLOW_COMPILER_WARNINGS. r=glandium 2017-10-25 15:12:09 -07:00
common.build Bug 1389598 - Part 2: Remove gonk references from media/ r=jesup 2017-08-11 17:46:15 -07:00
databuffer.h
dtlsidentity.cpp Bug 1372977 - Use nullptr in /media and /dom/media (clang-tidy: modernize-use-nullptr). r=bwc 2017-06-28 15:38:00 -04:00
dtlsidentity.h
logging.h Bug 1354222: fix log level casting warnings. r=erahm 2017-04-06 11:00:35 -07:00
m_cpp_utils.h
moz.build Bug 1373843: Add a libfuzzer test for the stun parser; r=decoder 2017-06-16 16:01:19 -07:00
nr_socket_prsock.cpp Bug 1387002 - Replace .size() by .empty() when applicable in webrtc & mtransport r=jesup 2017-08-04 09:36:19 +02:00
nr_socket_prsock.h Bug 1341374: ensure mtransport refcounting is happening on the same thread r=jesup 2017-02-23 21:57:50 -08:00
nr_timer.cpp Bug 1404198: Part 2g - Switch to NS_NewTimer* in media. r=njn 2017-10-15 23:14:33 -07:00
nricectx.cpp Bug 1410794 (attempt 2) - Change some |string| occurrences in nsIPrefBranch.idl to |ACString|. r=erahm. 2017-10-27 10:30:33 +11:00
nricectx.h Bug 1391857 - fixing ctx flags for e10s stun addr gathering. r=drno 2017-08-22 10:13:06 -05:00
nricectxhandler.cpp Bug 1342523 - Part 1: Clean up how controlling/offerer is specified. r=drno 2017-02-24 14:05:27 -06:00
nricectxhandler.h Bug 1342523 - Part 1: Clean up how controlling/offerer is specified. r=drno 2017-02-24 14:05:27 -06:00
nricemediastream.cpp Bug 1387002 - Replace .size() by .empty() when applicable in webrtc & mtransport r=jesup 2017-08-04 09:36:19 +02:00
nricemediastream.h Bug 1339906 - pt 4 - add last sent and received timestamps to RTCIceCandidatePairStats. r=drno,qdot 2017-06-06 17:36:40 -05:00
nriceresolver.cpp Bug 1372977 - Use nullptr in /media and /dom/media (clang-tidy: modernize-use-nullptr). r=bwc 2017-06-28 15:38:00 -04:00
nriceresolver.h
nriceresolverfake.cpp Bug 1372977 - Use nullptr in /media and /dom/media (clang-tidy: modernize-use-nullptr). r=bwc 2017-06-28 15:38:00 -04:00
nriceresolverfake.h
nricestunaddr.cpp Bug 1345511 - pt 2 - add IPC mechanism for getting stun addrs on main process. r=bwc 2017-03-21 19:59:05 -05:00
nricestunaddr.h Bug 1345511 - pt 2 - add IPC mechanism for getting stun addrs on main process. r=bwc 2017-03-21 19:59:05 -05:00
nrinterfaceprioritizer.cpp Bug 1408218: ignore EUI 64 and Teredo addresses if not needed. r=bwc 2017-10-13 08:32:16 -07:00
nrinterfaceprioritizer.h
README
rlogconnector.cpp
rlogconnector.h
runnable_utils.h bug 1390286 receive WrapRunnable*() parameters by reference r=jesup 2017-08-14 11:33:53 +12:00
sigslot.h Bug 1378712 - Remove all trailing whitespaces r=Ehsan 2017-07-06 14:00:35 +02:00
simpletokenbucket.cpp
simpletokenbucket.h
stun_socket_filter.cpp
stun_socket_filter.h
test_nr_socket.cpp Bug 1372977 - Use nullptr in /media and /dom/media (clang-tidy: modernize-use-nullptr). r=bwc 2017-06-28 15:38:00 -04:00
test_nr_socket.h Bug 1355947 - Add NatDelegate to TestNat; r=bwc 2017-06-13 10:08:45 -04:00
transportflow.cpp
transportflow.h
transportlayer.cpp
transportlayer.h
transportlayerdtls.cpp Bug 1404198: Part 2g - Switch to NS_NewTimer* in media. r=njn 2017-10-15 23:14:33 -07:00
transportlayerdtls.h
transportlayerice.cpp Bug 1405940 - Fix Null Pointer dereference in sigslot::lock_block r=bwc 2017-10-12 22:38:01 -05:00
transportlayerice.h
transportlayerlog.cpp
transportlayerlog.h
transportlayerloopback.cpp Bug 1404198: Part 2g - Switch to NS_NewTimer* in media. r=njn 2017-10-15 23:14:33 -07:00
transportlayerloopback.h Bug 1382172 - Name nsITimerCallback instances in native implementation. r=billm 2017-07-27 02:18:20 +08:00
transportlayerprsock.cpp
transportlayerprsock.h

This is a generic media transport system for WebRTC.

The basic model is that you have a TransportFlow which contains a
series of TransportLayers, each of which gets an opportunity to
manipulate data up and down the stack (think SysV STREAMS or a
standard networking stack). You can also address individual
sublayers to manipulate them or to bypass reading and writing
at an upper layer; WebRTC uses this to implement DTLS-SRTP.


DATAFLOW MODEL
Unlike the existing nsSocket I/O system, this is a push rather
than a pull system. Clients of the interface do writes downward
with SendPacket() and receive notification of incoming packets
via callbacks registed via sigslot.h. It is the responsibility
of the bottom layer (or any other layer which needs to reference
external events) to arrange for that somehow; typically by
using nsITimer or the SocketTansportService.

This sort of push model is a much better fit for the demands
of WebRTC, expecially because ICE contexts span multiple
network transports.


THREADING MODEL
There are no thread locks. It is the responsibility of the caller to
arrange that any given TransportLayer/TransportFlow is only
manipulated in one thread at once. One good way to do this is to run
everything on the STS thread. Many of the existing layer implementations
(TransportLayerPrsock, TransportLayerIce, TransportLayerLoopback)
already run on STS so in those cases you must run on STS, though
you can do setup on the main thread and then activate them on the
STS.


EXISTING TRANSPORT LAYERS
The following transport layers are currently implemented:

* DTLS -- a wrapper around NSS's DTLS [RFC 6347] stack
* ICE  -- a wrapper around the nICEr ICE [RFC 5245] stack.
* Prsock -- a wrapper around NSPR sockets
* Loopback -- a loopback IO mechanism
* Logging -- a passthrough that just logs its data

The last three are primarily for debugging.