Commit Graph

626 Commits

Author SHA1 Message Date
Carsten "Tomcat" Book
f49979964c Backed out changeset 1913c67263eb (bug 1231975) 2016-04-25 16:28:44 +02:00
Carsten "Tomcat" Book
8353eec0ac Backed out changeset 9d0c273048d6 (bug 1231975) 2016-04-25 16:28:42 +02:00
Carsten "Tomcat" Book
7cc02680ae Backed out changeset b21f8efbd341 (bug 1231975) 2016-04-25 16:28:41 +02:00
Byron Campen [:bwc]
543c799051 Bug 1266468: Respond to STUN requests with 400 if the component is failed. r=drno
MozReview-Commit-ID: 106OvQ7WLvp

--HG--
extra : rebase_source : 797c932080d07ff3d0be2fd20d9082944f1ac51d
2016-04-21 13:00:26 -05:00
Byron Campen [:bwc]
f508596540 Bug 1231975 - Part 5: Fix an intermittent failure caused by the NAT simulator erroneously canceling NR_ASYNC_WAIT_READ. r=drno
MozReview-Commit-ID: HwatO6UaeEX

--HG--
extra : rebase_source : 4fa178e6b2d715d5f7e56e7dd19d607830c31ff7
2016-04-14 15:33:40 -05:00
Byron Campen [:bwc]
4262b60302 Bug 1231975 - Part 4: Add some logging and simplification in TestNrSocket. r=drno
MozReview-Commit-ID: ASbgh62xqQU

--HG--
extra : rebase_source : 3fdcbf69370a69556b6b34d1a9c8c142732e21f8
2016-04-14 09:24:47 -05:00
Byron Campen [:bwc]
fb77b5d2b8 Bug 1231975 - Part 3: Break a reference cycle between PendingResolution and DNSRequestChild. r=drno
MozReview-Commit-ID: CS1q6LyUsIe

--HG--
extra : rebase_source : efc3eb166d01de91c2d628540fda625155d8cd26
2016-04-07 13:12:54 -05:00
Byron Campen [:bwc]
98c17059df Bug 1231975 - Part 1: Basic audio mochitests for NAT scenarios. r=drno
MozReview-Commit-ID: 1chfSrXeCL1

--HG--
extra : rebase_source : 20d6bc3424e433fbc0a98739310598c3c9433c87
2016-03-14 13:31:01 -05:00
Byron Campen [:bwc]
50135632ac Bug 1231981 - Part 3: Set up TURN server for webrtc mochitests, when configured to. r=drno
MozReview-Commit-ID: CVbAYPrwpuB

--HG--
extra : rebase_source : f52583fe21269d4d4214b93315cc75f0535de55d
2016-03-04 15:45:57 -06:00
Nils Ohlmeier [:drno]
df7216ecb4 Bug 1259842: allow RFC1918 pairing again r=mjf
MozReview-Commit-ID: 6afbRF64PM2

--HG--
extra : rebase_source : fbdf8db55ed9fb22cf39b375551741a5e75aac14
2016-04-05 17:10:03 -07:00
Michael Froman
eeafb75c78 Bug 1264344 - Don't restart ICE on first CreateOffer call if iceRestart option is true. r=bwc
MozReview-Commit-ID: 18qpyjAo0By

--HG--
extra : rebase_source : 172dcd1dfe95bdad684a4ce9a352183f2e77bafc
2016-04-13 11:41:05 -05:00
Michael Froman
438bea1ee6 Bug 906986 - Rework rollback/finalize to include a committed state. r=bwc, r=drno
MozReview-Commit-ID: z7uEn5xEBf

--HG--
extra : rebase_source : 18565a3c49045af9445a93de8753110169caf465
2016-04-05 20:12:04 -05:00
Michael Froman
e2e642f12e Bug 906986 - Ice restart and tests. r=bwc, r=drno, r=smaug
MozReview-Commit-ID: AMEi7SZebBG
MozReview-Commit-ID: GS2EkamNGc7

--HG--
extra : rebase_source : f715bc837331028f6b5e22820c530e34ca5bedb8
2016-04-08 09:20:53 -05:00
Michael Froman
3afe176e86 Bug 906986 - Wrap NrIceCtx in NrIceCtxHandler which will allow us to handle ice restart. r=bwc, r=drno
MozReview-Commit-ID: 8MEfEBf2Pvi

--HG--
extra : rebase_source : ca23f499f155593e0601f918ab16ee38fae35f42
2016-03-31 14:12:19 -05:00
Michael Froman
ae71fe6a8c Bug 906986 - Genericize some of the test setup and improve logging. r=bwc, r=drno
MozReview-Commit-ID: 7HddAbcncFC

--HG--
extra : rebase_source : fd418ab3493011f25ed8aaaea50549474d29108b
2016-03-31 11:31:58 -05:00
Michael Froman
d4c059685e Bug 906986 - Use the streams from the ice_ctx, don't hold a separate set. r=bwc, r=drno
MozReview-Commit-ID: FwomOrICh70

--HG--
extra : rebase_source : bdb08ab1cb8e6c5779171e30aecb282bd5e30e24
2016-03-31 10:17:28 -05:00
Michael Froman
ac55e03a26 Bug 906986 - Rework fix for Bug 1241690 to avoid reliance on NrIceCtx inside NrIceMediaStream. r=bwc, r=drno
MozReview-Commit-ID: 1QCfbZUGdwM

--HG--
extra : rebase_source : e47546e226839410b65013c66eab53b0212e86fe
2016-03-31 09:24:17 -05:00
Michael Froman
04ec3df956 Bug 906986 - Disable TestSrflxCandPairingFilter until bug 1226838 is fixed. r=bwc, r=drno
MozReview-Commit-ID: 5QFUeqDG4dh

--HG--
extra : rebase_source : a7c3f93e33072d0645339a6a169d49a9f58b04f2
2016-03-31 09:16:23 -05:00
Nils Ohlmeier [:drno]
4d0aaaafea Bug 1244926: added TCP socket filter to only allow outgoing STUN. r=jesup
MozReview-Commit-ID: 8PpL6Z0za71

--HG--
rename : media/mtransport/stun_udp_socket_filter.cpp => media/mtransport/stun_socket_filter.cpp
rename : media/mtransport/stun_udp_socket_filter.h => media/mtransport/stun_socket_filter.h
rename : netwerk/base/nsIUDPSocketFilter.idl => netwerk/base/nsISocketFilter.idl
extra : rebase_source : 2acf092e587a19a2019f463fa0a015373d3881d3
2016-02-11 01:18:46 -08:00
Byron Campen [:bwc]
7325e61fd6 Bug 1258753: Base candidate pair priority on controlling/controlled. r=drno
MozReview-Commit-ID: 6RAFaAtBbJq

--HG--
extra : rebase_source : 976b63d44fc7c50efa8f40b067cf45b56c66ed09
extra : source : 7b6263bcd3d7e9148914f5aaace10fc69ea0395d
2016-03-22 12:22:56 -05:00
Nicholas Nethercote
ea9bc73f05 Bug 1254780 (attempt 2) - Shrink log_types from 1024 entries to 16. r=ekr.
MozReview-Commit-ID: KWrOTuTlt6H

--HG--
extra : rebase_source : 36ec95f4584ea0d67286e9956456dffbe5294401
2016-03-11 10:06:32 +11:00
Byron Campen [:bwc]
6765c70f22 Bug 933986. Switch over from index to an id, and ensure uniqueness when feeding into the candidate priority calculation. r=drno
MozReview-Commit-ID: 5LOQJbsnEWN

--HG--
extra : rebase_source : 644695f73f316fa8e71e410442f3796822e3255a
2013-11-04 16:50:10 -08:00
Byron Campen [:bwc]
03253b8221 Bug 1256720: Remove a bad assertion, and simplify some code. r=drno
MozReview-Commit-ID: 3h52q8Wrp8R

--HG--
extra : rebase_source : a1cf4d7fa29e4ed7bb6a1f5be19d1b2af2ef5316
2016-03-24 15:05:53 -05:00
Sebastian Hengst
4029855eed Backed out changeset e139011ede5b (bug 1254780) for crashes in mozilla::gfx::Log<1,mozilla::gfx::CriticalLogger>::WriteLog on Windows 7 debug. r=backout 2016-03-24 10:52:52 +01:00
Nicholas Nethercote
d6246f169d Bug 1254780 - Shrink log_types from 1024 entries to 16. r=ekr.
--HG--
extra : rebase_source : 7cc7a749b080fb3e35c9b8cf285ff88738ef3234
2016-03-11 10:06:32 +11:00
Byron Campen [:bwc]
02772181cb Bug 1257472: Forgive rport of 0. r=drno,ekr
MozReview-Commit-ID: 12YyYqrt3wR

--HG--
extra : rebase_source : b3e8397a0c29f8a9d354e4081fcf76e103fd11d8
2016-03-17 10:50:33 -05:00
Martin Thomson
7d88785757 Bug 1251822 - Disable ChaCha20Poly1305 for unit tests that check cipher suite, r=ekr
MozReview-Commit-ID: H7IW0iOTAzx

--HG--
extra : rebase_source : 7d637e24ea89c0d070fbe0511dd445f8d11e51f6
2016-03-16 17:52:49 +11:00
Nils Ohlmeier [:drno]
eca4273031 Bug 1233181: improve misleading log message about missing STUN & TURN servers. r?=mjf
MozReview-Commit-ID: sQzU9u96yP

--HG--
extra : rebase_source : 9a5d3198ee4d52fab556d2eace08aa41ea75cf5a
2015-12-16 12:00:27 -08:00
Boris Zbarsky
1228b7cbd3 Bug 1255269. Get rid of nsITCPSocketCallback.fireDataEvent. r=jdm 2016-03-10 18:07:28 -05:00
Byron Campen [:bwc]
736356238e Bug 1248637: Prune duplicate CANCELLED candidate pairs. r=drno
MozReview-Commit-ID: HeErN7xMxbQ

--HG--
extra : rebase_source : d8f355e6cc74c73c3e9bc100a2bb2b2603931e55
2016-03-01 10:47:14 -06:00
Nils Ohlmeier [:drno]
6d382efd5e Bug 1253198: add WebRtcIce prefix to all ICE unit tests. r=bwc
MozReview-Commit-ID: 5wD8Dl9ETkB

--HG--
extra : rebase_source : edf942c35be1d4caba5501b9395a70953eaa8344
2016-03-03 01:40:52 -08:00
Nils Ohlmeier [:drno]
01210346dd Bug 1217677: increase UDP socket receive buffer for <= Win7. r=jesup,mcmanus
MozReview-Commit-ID: A3yCZZ3Pwcu

--HG--
extra : rebase_source : 67d200194ed72076fcb6064f16ec94334f8fb5e7
2016-03-01 21:46:50 -08:00
Nils Ohlmeier [:drno]
1a7d746ecc Bug 1252777: skip over ICE TCP host candidate creation failures. r=bwc
MozReview-Commit-ID: 6l0ierHz5eL

--HG--
extra : rebase_source : 5bcce0063cf092f9eba278ea6790a0ada07eb82c
2016-03-02 01:27:01 -08:00
Nathan Froyd
b938c57967 Bug 1251714 - use UniquePtr instead of ScopedDeletePtr in media/; r=jesup
UniquePtr is more standard than ScopedDeletePtr; using standard
constructs whenever possible is preferable.

This patch merits a couple explanations:

- Where it made sense, I tried to convert:

    T* foo() {
      UniquePtr<T> x = ...;
      ...
      return x.release();
    }

  into:

    UniquePtr<T> foo()

  with corresponding changes inside |foo|'s body.

- The attentive reader will note that:

    auto x = MakeUnique<T>(...);

  is used sometimes and:

    UniquePtr<T> x(new T(...));

  is used sometimes.  I would prefer to use the former, but was stymied
  in several places due to protected constructors.  (MakeUnique doesn't
  have access to those protected constructors, natch.)
2016-03-02 15:28:27 -05:00
Byron Campen [:bwc]
f1f2ee96b5 Bug 1037618 - Part 1: Add some logging to highlight TCP connection failures. r=drno
MozReview-Commit-ID: 4q84yGspvyX

--HG--
extra : rebase_source : 099d1dd32cc46ed96ac234fe0a65c85333acfdf1
2016-03-01 12:06:12 -06:00
Byron Campen [:bwc]
f3294c2565 Bug 1252585: Clear received_ct_ in TestStunServer::Reset r=drno
MozReview-Commit-ID: F7Dgk5gN4cs

--HG--
extra : rebase_source : 5a5b536b906a01dfb171008ba02ce3650abafca9
2016-03-01 13:59:43 -06:00
Byron Campen [:bwc]
8e9153abea Bug 1252163: Fix CheckTcpConnectivity to wait for readable/writeable when necessary. r=drno
MozReview-Commit-ID: 6hz0eCghYE0

--HG--
extra : rebase_source : 46b1221ab29c33455f7ceb3c0e149e4c8aaa2397
2016-02-29 12:33:47 -06:00
Byron Campen [:bwc]
3a66dfe493 Bug 1252171: Update last_used_ on TCP port mappings when they are used, similar to UDP. r=drno
MozReview-Commit-ID: DNinTza44la

--HG--
extra : rebase_source : 5d65d9a4c21ad113a2762ebf397b1e7b8f289dd5
2016-02-29 13:04:42 -06:00
Wes Kocher
6866309c97 Backed out 2 changesets (bug 1251714, bug 1251715) for gtest failures in media code
Backed out changeset 1bbd0cd10f76 (bug 1251714)
Backed out changeset 80b197c5608f (bug 1251715)

MozReview-Commit-ID: EHOtiKLS4Xr
2016-03-01 11:36:35 -08:00
Nathan Froyd
70d7688ee3 Bug 1251714 - use UniquePtr instead of ScopedDeletePtr in media/; r=jesup
UniquePtr is more standard than ScopedDeletePtr; using standard
constructs whenever possible is preferable.

This patch merits a couple explanations:

- Where it made sense, I tried to convert:

    T* foo() {
      UniquePtr<T> x = ...;
      ...
      return x.release();
    }

  into:

    UniquePtr<T> foo()

  with corresponding changes inside |foo|'s body.

- The attentive reader will note that:

    auto x = MakeUnique<T>(...);

  is used sometimes and:

    UniquePtr<T> x(new T(...));

  is used sometimes.  I would prefer to use the former, but was stymied
  in several places due to protected constructors.  (MakeUnique doesn't
  have access to those protected constructors, natch.)
2016-02-26 14:52:15 -05:00
Eric Rahm
e9c93e58a3 Bug 1251737 - Remove remaining references to MOZILLA_XPCOMRT_API from media. r=jesup 2016-02-27 11:12:07 -08:00
Eric Rahm
2917e8395e Bug 1239870 - Part 5: Switch over mtransport tests to mozilla gtests. r=bwc
This converts the individual cppunit gtests into the combined mozilla gtest
which has access to xpcom internals. The build file is simplified to reflect
this change, individual main functions are removed, and duplicate symbols are
removed.
2016-02-09 10:02:40 -08:00
Eric Rahm
082b84fde2 Bug 1239870 - Part 4: Disable broken proxy tunnel tests. r=bwc
Several of the proxy tunnel tests are broken. The proxy tunnel test is also
not run in automation.
2016-02-25 15:54:04 -08:00
Eric Rahm
c9f8e6756f Bug 1239870 - Part 3: Add a base mtransport gtest. r=bwc
This adds a base test for other mtransport tests to be derived from. It
handles common setup used by the mtransport tests and parses relevant env
vars.
2016-02-25 14:25:49 -08:00
Eric Rahm
d5bd208a66 Bug 1239870 - Part 2: Split out NrIceCtx initialization. r=bwc
This splits NrIceCtx initialization into its own function so that the tests
can initialize without having to create a dummy instance.
2016-02-24 18:37:18 -08:00
Eric Rahm
0d0d8d2c9e Bug 1239870 - Part 1: Remove declaration of test_utils from header. r=bwc
Once files are all compiled into the same gtest this will cause duplicate
symbol errors. It's also unused.
2016-02-09 10:02:37 -08:00
Wes Kocher
72d9604d91 Backed out 8 changesets (bug 1251482, bug 1251494, bug 1251473, bug 1239870) for gtest failures
Backed out changeset f064a5efbb8c (bug 1251494)
Backed out changeset 9e33adec1aa6 (bug 1251482)
Backed out changeset ab0347657e25 (bug 1251473)
Backed out changeset 1d385d4f195d (bug 1239870)
Backed out changeset ceb3e1ee7dda (bug 1239870)
Backed out changeset 8574075bf42f (bug 1239870)
Backed out changeset ba077a3afbc7 (bug 1239870)
Backed out changeset eb99ab06414d (bug 1239870)

MozReview-Commit-ID: 7r9SEk4VGNU
2016-02-26 17:14:57 -08:00
Eric Rahm
6968bce012 Bug 1239870 - Part 5: Switch over mtransport tests to mozilla gtests. r=bwc
This converts the individual cppunit gtests into the combined mozilla gtest
which has access to xpcom internals. The build file is simplified to reflect
this change, individual main functions are removed, and duplicate symbols are
removed.
2016-02-09 10:02:40 -08:00
Eric Rahm
6c7be83907 Bug 1239870 - Part 4: Disable broken proxy tunnel tests. r=bwc
Several of the proxy tunnel tests are broken. The proxy tunnel test is also
not run in automation.
2016-02-25 15:54:04 -08:00
Eric Rahm
dc59103e98 Bug 1239870 - Part 3: Add a base mtransport gtest. r=bwc
This adds a base test for other mtransport tests to be derived from. It
handles common setup used by the mtransport tests and parses relevant env
vars.
2016-02-25 14:25:49 -08:00