Commit Graph

3815 Commits

Author SHA1 Message Date
Christoph Kerschbaumer
b47c0eea98 Bug 1254691 - Remove SEC_NORMAL from webrtc/. r=bwc 2016-03-10 08:52:34 -08:00
Boris Zbarsky
1228b7cbd3 Bug 1255269. Get rid of nsITCPSocketCallback.fireDataEvent. r=jdm 2016-03-10 18:07:28 -05:00
Timothy Nikkel
1f99404503 Bug 1187420. r=drc r=jmuizelaar a=abillings 2016-03-10 13:23:27 -06: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
Noel Gordon
6b3d2d0174 Bug 1050342. Fix a case where the fast huffman decoder in libjpeg-turbo can produce different results depending on how data is fed to it. r=seth
This change comes from the blink repo https://codereview.appspot.com/229430043/ and is unlikely to be accepted upstream into libjpeg-turbo.
2016-03-09 17:56:35 -06:00
Gian-Carlo Pascutto
568a64e79a Bug 1254507 - Fix leak in WebRTC DesktopApplication class. r=jesup
MozReview-Commit-ID: FLuQZcPyv0d

--HG--
extra : rebase_source : 24bc9507bb4abee94a1238176675948249c102f1
2016-03-08 14:14:05 +01:00
Gerald Squelart
2d4fdb1b57 Bug 1253471 - Remove Metadata hard-coded limit - r=jya
Instead of relying on some arbitrary limit for ftyp+moov box sizes, we check
for overflow and possible type truncations, and then let memory allocation
routines (e.g. MediaByteBuffer::SetLength) deal with actual memory limitations.

MozReview-Commit-ID: AXXxvdDYnnr
2016-03-08 17:25:33 +11: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
Byron Campen [:bwc]
b0ba04fb55 Bug 1221473: Do not treat answer as authoritative wrt payload types. r=drno
MozReview-Commit-ID: 2uVn60j8QfC

--HG--
extra : rebase_source : 4f5ee469a514deb4e89745885cda34cb6061a660
2015-11-06 10:14:23 -06:00
[:ng]
9e95296e04 bug 1241064 - updating stats filter SSRC when audio channel SSRC changes; r=jib
bug: getStats was returning statistics for the shortlived, initial SSRC
now updating SSRC filter on statistics update callback to match audio channel ssrc
getStats API now returns statistics for correct SSRC: jitter, packets lost, etc.

MozReview-Commit-ID: WCd71WMkUW

--HG--
extra : rebase_source : 5d3a5a14e04313749173d264894e44411c3417bf
2016-03-03 08:03:06 -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
Makoto Kato
89dd964ab1 Bug 1252737 - use size_t instead of uint32_t for InitEncode(). r=jesup
Although 3rd parameter VideoEncodeer::InitEncode() is defiend as size_t, WebrtcIMXH264VideoCodec and WebrtcMediaCodecVP8VideoCodec uses uint32_t.  This will cause build error for 64bit platform.

MozReview-Commit-ID: 20NgE5e7Cuz

--HG--
extra : rebase_source : 6da7b73504843b1062daff2b6977743114c2591e
2016-03-02 13:05:43 +09:00
Nathan Froyd
9d78c6759b Bug 1252901 - don't use ScopedFreePtrTraits in MediaEngineWrapper.h; r=jesup
ScopedFreePtrTraits is somewhat of an internal implementation detail of
Scoped.h, and it's not hard to write out the two static functions we
need from ScopedFreePtrTraits anyway.  Removing this means that we can
clear the way for ScopedFreePtr to be removed.
2016-03-02 11:43:58 -05:00
Mike Hommey
1ba1737ec0 Bug 1252699 - Set WEBRTC_DETECT_ARM_NEON when optional neon is requested. r=jesup 2016-03-03 06:28:10 +09: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
Jan-Ivar Bruaroey
fe08f0669c Bug 1250990 - Make RTCRtpEncodingParameters.scaleResolutionDownBy work with H.264 unicast. r=jesup
MozReview-Commit-ID: 2j8rRzZemql

--HG--
extra : rebase_source : c3750d3ed62d99d05cf2b637ef06001b5a9c347e
2016-02-28 09:19:23 -05:00
Dan Minor
3b0b90c4ba Bug 1158741 - Implement a version of omxSP_FFTInv_CCSToR_F32_Sfs in openmax DL's FFT that is not scaled r=padenot
The new routine actually multiplies by two for consistency with the other FFT
routines in use.

MozReview-Commit-ID: Hk2Dg3fR2cQ

--HG--
extra : rebase_source : 08bdbbd65d372a3d0eb69568313cec33ccea6af3
2016-01-25 06:38:29 -05:00
Julian Seward
0c7e11d71b Bug 1252073 - Uninitialised value uses in mozilla::EncodingConstraints::operator==. r=docfaraday@gmail.com.
--HG--
extra : rebase_source : 705e0e821163e8f49d591d77ad306e317884d1c9
2016-03-02 12:29:36 +01: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
Byron Campen [:bwc]
cb39275975 Bug 818618: Honor (and emit) opus stereo fmtp param. r=jesup
MozReview-Commit-ID: IgA305eiu1s

--HG--
extra : rebase_source : bc94c0c18bb95af7e2f9eb27dcda6a5699264ab1
2016-02-26 10:47:03 -06:00
Matthew Gregan
9ef5badfe1 Bug 1251504 - Update libnestegg to remove generated nestegg-stdint.h. r=giles 2016-02-26 15:38:14 +13:00
Sebastian Hengst
b63895fdbf Backed out changeset 4cc3fae66ffb (bug 1250990) for frequent failure of modified test test_peerConnection_scaleResolution.html. r=frequentorange 2016-02-28 09:59:16 +01:00
Jan-Ivar Bruaroey
2f66effab3 Bug 1250990 - Make RTCRtpEncodingParameters.scaleResolutionDownBy work with H.264 unicast. r=jesup
MozReview-Commit-ID: 2j8rRzZemql

--HG--
extra : rebase_source : 1e039392219000830d869f0ad5432a297f6d6987
2016-02-26 16:16:11 -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
Eric Rahm
c93c9b8dbe 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
a90c690c9a 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
Glenn Randers-Pehrson
95cc84e607 Bug 1230757 - Update in-tree libpng to version 1.6.21. r=seth
--HG--
extra : rebase_source : a3385a5b90d364900095f037355141f342e43c49
2016-01-16 09:27:00 -05:00
Byron Campen [:bwc]
ca4206043b Bug 1251214: Ignore R_WOULDBLOCK in nr_stun_client_send_request r=ekr
MozReview-Commit-ID: HLrvq4BqT9D

--HG--
extra : rebase_source : 4fdef84e82f8eec2f013b7c667bb0ac606ce0751
2016-02-25 11:34:27 -06:00
Byron Campen [:bwc]
50802e775a Bug 1249098: Support maxplaybackrate for opus. r=jesup
MozReview-Commit-ID: 7BKVFkbPgV2

--HG--
extra : rebase_source : 3ef372486a8c608c780e864412942e8819cc3f88
2016-02-18 14:13:35 -06:00
Byron Campen [:bwc]
a2f128dc8b Bug 1247656: Make sure that remote reoffer does not change the media type of an m-line. r=drno
MozReview-Commit-ID: DFXgLuSW6nM

--HG--
extra : rebase_source : f8872a6e7d9dc2aecbbfbfa46497176f63d21282
2016-02-11 13:00:58 -06:00
Jan-Ivar Bruaroey
c4ea91fa82 Bug 1247619 - add console warning about OAuth for STUN not yet supported. r=jesup,smaug
MozReview-Commit-ID: L0nViIWiIDn

--HG--
extra : rebase_source : 3d463e85c811426384a758dca3b0a82370c22099
2016-02-25 00:24:16 -05:00
Jan-Ivar Bruaroey
ceba79f2b0 Bug 1244913 - resolution-based bitrates for each simulcast layer, scaleResolutionDownBy, and working maxBitrate in unicast. r=bwc,jesup
MozReview-Commit-ID: 347J1ElsOEx

--HG--
extra : rebase_source : 33eff52e6082815d732de49a2bac584cfc9c87c4
2016-02-12 19:56:56 -05:00
Jan-Ivar Bruaroey
f7040b1412 Bug 1244913 - change SelectBandwidth to SelectBitrates. r=jesup
MozReview-Commit-ID: Gc5WN7JIozV

--HG--
extra : rebase_source : 86bc3281c3f2db284555c498436c4a6469148647
2016-02-12 16:08:00 -05:00
Randell Jesup
831c3445c4 Bug 1248335: avoid using SvcInternal structure entirely, as system-vpx may not have it r=pkerr
MozReview-Commit-ID: 146FTSGQ8Ck
2016-02-23 11:55:24 -05:00
Nathan Froyd
689320c0d0 Bug 1249369 - use UniquePtr instead of nsAutoArrayPtr in MediaPipeline.cpp; r=jesup 2016-02-18 14:09:23 -05:00