15511 Commits

Author SHA1 Message Date
jianjun.zhu
cc7213e2c0 Remove "video_capture" from modules' public_deps.
WebRTC standalone build may depend on "video_capture_internal_impl"
instead of "video_capture". Including "video_capture" in public_deps
leads to duplicated definition in this case.

BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2657783002
Cr-Commit-Position: refs/heads/master@{#16279}
2017-01-26 00:15:04 +00:00
buildbot
46fa41342a Roll chromium_revision 495d2cc61a..4afe10c428 (446030:446129)
Change log: 495d2cc61a..4afe10c428
Full diff: 495d2cc61a..4afe10c428

Changed dependencies:
* src/base: 350220928e..2d3420f66f
* src/build: 24cce8777f..da7ee60a5f
* src/testing: b885778da8..7555312771
* src/third_party: c9902b8c89..9f14a86a78
* src/third_party/android_tools: https://chromium.googlesource.com/android_tools.git/+log/b43a6a289a..e429db7f48
* src/tools: 8eb1f68dd4..48d7a9c535
DEPS diff: 495d2cc61a..4afe10c428/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2658683002
Cr-Commit-Position: refs/heads/master@{#16278}
2017-01-25 23:44:43 +00:00
VladimirTechMan
1a7f01c6d3 Validate the build type argument value in function build_webrtc
BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2655893003
Cr-Commit-Position: refs/heads/master@{#16277}
2017-01-25 22:30:15 +00:00
VladimirTechMan
2d8cd58c55 Minor updates to the help message printed by build_ios_libs.sh (with -h)
BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2654083003
Cr-Commit-Position: refs/heads/master@{#16276}
2017-01-25 20:33:40 +00:00
haysc
148e370526 iOS: Add SendSideBweWithOverhead field trial key
BUG=0

Review-Url: https://codereview.webrtc.org/2649923011
Cr-Commit-Position: refs/heads/master@{#16275}
2017-01-25 18:02:20 +00:00
buildbot
329ec942c1 Roll chromium_revision 516550732e..495d2cc61a (446004:446030)
Change log: 516550732e..495d2cc61a
Full diff: 516550732e..495d2cc61a

Changed dependencies:
* src/testing: dad9a344e3..b885778da8
* src/third_party: dbc097f7ee..c9902b8c89
* src/tools: 6da7b37c9f..8eb1f68dd4
DEPS diff: 516550732e..495d2cc61a/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2649313004
Cr-Commit-Position: refs/heads/master@{#16274}
2017-01-25 17:36:03 +00:00
philipel
e0754305aa Don't update the jitter estimate with frames containing retransmitted packets.
BUG=chromium:682636

Review-Url: https://codereview.webrtc.org/2645343002
Cr-Commit-Position: refs/heads/master@{#16273}
2017-01-25 16:56:23 +00:00
brandtr
090c9405cc Sort method declarations/definitions in VideoReceiveStream.
Order as given by inheritance in class definition.

No functional changes are intended with this CL.

BUG=None

Review-Url: https://codereview.webrtc.org/2646343005
Cr-Commit-Position: refs/heads/master@{#16272}
2017-01-25 16:28:02 +00:00
ehmaldonado
3373eaa577 Revert of GN: Refactor modules_unittests to eliminate package boundary violations. (patchset #4 id:130001 of https://codereview.webrtc.org/2649563002/ )
Reason for revert:
Did break the bots.
https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/9807

Original issue's description:
> GN: Refactor modules_unittests to eliminate package boundary violations.
>
> Also move bwe_simulator to webrtc/modules/remote_bitrate_estimator
>
> BUG=webrtc:6954
> NOTRY=True
>
> Review-Url: https://codereview.webrtc.org/2649563002
> Cr-Commit-Position: refs/heads/master@{#16270}
> Committed: 36cb55d715

TBR=kjellander@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6954

Review-Url: https://codereview.webrtc.org/2651023005
Cr-Commit-Position: refs/heads/master@{#16271}
2017-01-25 16:11:28 +00:00
ehmaldonado
36cb55d715 GN: Refactor modules_unittests to eliminate package boundary violations.
Also move bwe_simulator to webrtc/modules/remote_bitrate_estimator

BUG=webrtc:6954
NOTRY=True

Review-Url: https://codereview.webrtc.org/2649563002
Cr-Commit-Position: refs/heads/master@{#16270}
2017-01-25 16:00:15 +00:00
mbonadei
26764613ad Moving build_aar.py to new location
We are moving the whole content of "webrtc/build" to new
locations (see crbug.com/611808 for further information).

The new location for the "webrtc/build/android" stuff is
"tools-webrtc/android".

BUG=webrtc:7030

Review-Url: https://codereview.webrtc.org/2647343006
Cr-Commit-Position: refs/heads/master@{#16269}
2017-01-25 15:42:08 +00:00
johan
bfb11b2243 Call RtpStreamReceiver.AddReceiveCodec() with codec_params.
BUG=webrtc:5948

Review-Url: https://codereview.webrtc.org/2649873005
Cr-Commit-Position: refs/heads/master@{#16268}
2017-01-25 15:37:27 +00:00
aleloi
d160fd735d Disabled EndToEndTest.ReceivesFlexfecAndSendsCorrespondingRtcp on Asan
due to timeout-caused build failure (see bugs.webrtc.org/7047). The
timeout is governed by CallTest::kDefaultTimeoutMs, which is set to 30
seconds. This can be too low for Asan.

TBR=brandtr@webrtc.org
BUG=webrtc:7047

Review-Url: https://codereview.webrtc.org/2657823003
Cr-Commit-Position: refs/heads/master@{#16267}
2017-01-25 14:37:58 +00:00
buildbot
8294952473 Roll chromium_revision 66ac8d1a05..516550732e (445993:446004)
Change log: 66ac8d1a05..516550732e
Full diff: 66ac8d1a05..516550732e

Changed dependencies:
* src/testing: 6427e10990..dad9a344e3
* src/third_party: c738010c02..dbc097f7ee
DEPS diff: 66ac8d1a05..516550732e/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2649113006
Cr-Commit-Position: refs/heads/master@{#16266}
2017-01-25 13:53:28 +00:00
mbonadei
e5dc3cefab Fixing cross-compiling issues on android arm
BUG=webrtc:7042

Review-Url: https://codereview.webrtc.org/2647293006
Cr-Commit-Position: refs/heads/master@{#16265}
2017-01-25 13:34:46 +00:00
buildbot
c07cc56a2e Roll chromium_revision 34f2476fb5..66ac8d1a05 (445972:445993)
Change log: 34f2476fb5..66ac8d1a05
Full diff: 34f2476fb5..66ac8d1a05

Changed dependencies:
* src/base: eb43b51ed9..350220928e
* src/testing: c88c7816e8..6427e10990
* src/third_party: c9e15b526a..c738010c02
DEPS diff: 34f2476fb5..66ac8d1a05/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2651053003
Cr-Commit-Position: refs/heads/master@{#16264}
2017-01-25 11:35:08 +00:00
nisse
7d2542623a Delete unneeded includes of base/common.h.
Bulk of the changes were done using

   git grep -l '#include "webrtc/base/common.h"' | \
     xargs sed -i '\,^#include.*webrtc/base/common\.h,d'

followed by adding back the include in the few places where it is
still needed, and in one case (pseudotcp.cc) instead deleting its use
of RTC_UNUSED.

BUG=webrtc:6424

Review-Url: https://codereview.webrtc.org/2644103002
Cr-Commit-Position: refs/heads/master@{#16263}
2017-01-25 09:47:24 +00:00
Mirko Bonadei
b63a8ac5da Moving gn_isolate_map.pyl to tools-webrtc/mb
BUG=webrtc:7030
R=kjellander@webrtc.org

Review-Url: https://codereview.webrtc.org/2648353004 .
Cr-Commit-Position: refs/heads/master@{#16262}
2017-01-25 08:36:50 +00:00
buildbot
8d362740a0 Roll chromium_revision 900b07d425..34f2476fb5 (445935:445972)
Change log: 900b07d425..34f2476fb5
Full diff: 900b07d425..34f2476fb5

Changed dependencies:
* src/third_party: 28c657aa45..c9e15b526a
* src/tools: cd25259b3d..6da7b37c9f
DEPS diff: 900b07d425..34f2476fb5/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2652833007
Cr-Commit-Position: refs/heads/master@{#16261}
2017-01-25 08:06:22 +00:00
mbonadei
630f46a31e Moving adb_shell script to tools-webrtc
BUG=webrtc:7030

Review-Url: https://codereview.webrtc.org/2650653005
Cr-Commit-Position: refs/heads/master@{#16260}
2017-01-25 07:49:13 +00:00
mbonadei
18e83ea598 Moving sanitizers from build/ to base/
We have to move the content of the 'webrtc/build/' directory up of one level (see crbug.com/611808 for further details).

To avoid a collision with the DEPSed build/ at the top level we are moving all the content of 'webrtc/build/' to different locations (see webrtc:7030 for the suggested locations).

BUG=webrtc:7030
NOTRY=True

Review-Url: https://codereview.webrtc.org/2651913002
Cr-Commit-Position: refs/heads/master@{#16259}
2017-01-25 07:04:51 +00:00
deadbeef
f534659ee6 Adding ability for BaseChannel to use PacketTransportInterface.
... As opposed to DtlsTransportInternal.

The code is suboptimal right now, storing two pointers to the different
interfaces. This will all be cleaned up when we have an "RtpTransport"
abstraction that BaseChannel can use.

This CL also cleans up the "fake transport" classes a bit, and gives
them their own header files.

BUG=None

Review-Url: https://codereview.webrtc.org/2648233003
Cr-Commit-Position: refs/heads/master@{#16258}
2017-01-25 05:51:21 +00:00
buildbot
6b75f4d0e7 Roll chromium_revision 34daea6e90..900b07d425 (445884:445935)
Change log: 34daea6e90..900b07d425
Full diff: 34daea6e90..900b07d425

Changed dependencies:
* src/base: 2f3b8d7a83..eb43b51ed9
* src/build: e381bc2fc0..24cce8777f
* src/third_party: 6055e0b34b..28c657aa45
* src/tools: 0e62146d2b..cd25259b3d
DEPS diff: 34daea6e90..900b07d425/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2653183002
Cr-Commit-Position: refs/heads/master@{#16257}
2017-01-25 04:56:14 +00:00
buildbot
3a1eb04545 Roll chromium_revision e003d59373..34daea6e90 (445829:445884)
Change log: e003d59373..34daea6e90
Full diff: e003d59373..34daea6e90

Changed dependencies:
* src/base: afbd629fac..2f3b8d7a83
* src/build: b5a787ac0d..e381bc2fc0
* src/testing: 00f95df571..c88c7816e8
* src/third_party: 50be8edc32..6055e0b34b
* src/tools: 571766af1d..0e62146d2b
DEPS diff: e003d59373..34daea6e90/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2650273003
Cr-Commit-Position: refs/heads/master@{#16256}
2017-01-25 02:05:48 +00:00
VladimirTechMan
eaae505d02 Removing unused variable OUTPUT_LIB
BUG=None
NOTRY=True

Review-Url: https://codereview.webrtc.org/2652553004
Cr-Commit-Position: refs/heads/master@{#16255}
2017-01-25 00:07:59 +00:00
buildbot
0dabfbfd89 Roll chromium_revision 15a29620cf..e003d59373 (445766:445829)
Change log: 15a29620cf..e003d59373
Full diff: 15a29620cf..e003d59373

Changed dependencies:
* src/base: d4eef52eaf..afbd629fac
* src/third_party: bab07305b6..50be8edc32
* src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/1252f8758a..358baeb9a4
* src/tools: 824663d804..571766af1d
DEPS diff: 15a29620cf..e003d59373/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2649353006
Cr-Commit-Position: refs/heads/master@{#16254}
2017-01-24 22:54:40 +00:00
deadbeef
ed111dad4d Adding deadbeef@webrtc.org to webrtc/base/OWNERS.
Also removing jiayl@webrtc.org, who's no longer working on webrtc.

BUG=None

Review-Url: https://codereview.webrtc.org/2652763005
Cr-Commit-Position: refs/heads/master@{#16253}
2017-01-24 22:17:09 +00:00
buildbot
eadcf36014 Roll chromium_revision 057e94297e..15a29620cf (445739:445766)
Change log: 057e94297e..15a29620cf
Full diff: 057e94297e..15a29620cf

Changed dependencies:
* src/base: ae6760788e..d4eef52eaf
* src/build: 8ffcf1460f..b5a787ac0d
* src/testing: af6da07e59..00f95df571
* src/third_party: 6d3258111a..bab07305b6
* src/tools: ec97c74192..824663d804
DEPS diff: 057e94297e..15a29620cf/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2657643002
Cr-Commit-Position: refs/heads/master@{#16252}
2017-01-24 19:58:43 +00:00
buildbot
ae2551232b Roll chromium_revision 319b885718..057e94297e (445689:445739)
Change log: 319b885718..057e94297e
Full diff: 319b885718..057e94297e

Changed dependencies:
* src/base: cce542e3d7..ae6760788e
* src/build: bba7d3dfe3..8ffcf1460f
* src/testing: ffe8a52fdd..af6da07e59
* src/third_party: d3d7bc6bb3..6d3258111a
* src/tools: fa1e6e62b9..ec97c74192
DEPS diff: 319b885718..057e94297e/DEPS

No update to Clang.

TBR=
BUG=None

Review-Url: https://codereview.webrtc.org/2652653008
Cr-Commit-Position: refs/heads/master@{#16251}
2017-01-24 17:30:37 +00:00
stefan
e9f36d55da Make sure min and max bitrate is always set for audio.
BUG=webrtc:5079

Review-Url: https://codereview.webrtc.org/2654503005
Cr-Commit-Position: refs/heads/master@{#16250}
2017-01-24 16:18:45 +00:00
nisse
e256bc582a Delete left-over using declaration.
BUG=webrtc:6796

Review-Url: https://codereview.webrtc.org/2638763002
Cr-Commit-Position: refs/heads/master@{#16249}
2017-01-24 15:43:16 +00:00
ossu
a388310af2 Added api/webrtcsdp.h forwarding header to work around upstream projects.
NOTRY=true  # Small change, in a hurry and msan is being slow
TBR=tommi@webrtc.org
BUG=webrtc:5883

Review-Url: https://codereview.webrtc.org/2653703004
Cr-Commit-Position: refs/heads/master@{#16248}
2017-01-24 15:13:59 +00:00
mbonadei
9aa3f0a200 Reland of Moving webrtc.gni up one level from build/ (patchset #1 id:1 of https://codereview.webrtc.org/2657563002/ )
Reason for revert:
Starting to work on a fix (it seems that there are third_party dependencies that depends on the path to the webrtc.gni file)

Original issue's description:
> Revert of Moving webrtc.gni up one level from build/ (patchset #1 id:1 of https://codereview.webrtc.org/2651543003/ )
>
> Reason for revert:
> This was causing the following failure: https://build.chromium.org/p/chromium.webrtc.fyi/builders/Android%20Builder/builds/838/steps/generate_build_files/logs/stdio
>
> Original issue's description:
> > Moving webrtc.gni up one level from build/
> >
> > BUG=webrtc:7030
> >
> > Review-Url: https://codereview.webrtc.org/2651543003
> > Cr-Commit-Position: refs/heads/master@{#16241}
> > Committed: 35a32700fc
>
> TBR=kjellander@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7030
>
> Review-Url: https://codereview.webrtc.org/2657563002
> Cr-Commit-Position: refs/heads/master@{#16244}
> Committed: 69dc7dbe24

TBR=kjellander@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7030

Review-Url: https://codereview.webrtc.org/2654773002
Cr-Commit-Position: refs/heads/master@{#16247}
2017-01-24 14:58:22 +00:00
mbonadei
b54c63ffb0 Moving no_op_function.cc out of webrtc/build
BUG=webrtc:7030

Review-Url: https://codereview.webrtc.org/2645223003
Cr-Commit-Position: refs/heads/master@{#16246}
2017-01-24 13:25:12 +00:00
mbonadei
dabbea670c Moving whitespace file up by one folder
BUG=webrtc:7030
NOTRY=True

Review-Url: https://codereview.webrtc.org/2650593004
Cr-Commit-Position: refs/heads/master@{#16245}
2017-01-24 13:19:42 +00:00
mbonadei
69dc7dbe24 Revert of Moving webrtc.gni up one level from build/ (patchset #1 id:1 of https://codereview.webrtc.org/2651543003/ )
Reason for revert:
This was causing the following failure: https://build.chromium.org/p/chromium.webrtc.fyi/builders/Android%20Builder/builds/838/steps/generate_build_files/logs/stdio

Original issue's description:
> Moving webrtc.gni up one level from build/
>
> BUG=webrtc:7030
>
> Review-Url: https://codereview.webrtc.org/2651543003
> Cr-Commit-Position: refs/heads/master@{#16241}
> Committed: 35a32700fc

TBR=kjellander@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7030

Review-Url: https://codereview.webrtc.org/2657563002
Cr-Commit-Position: refs/heads/master@{#16244}
2017-01-24 13:14:35 +00:00
mandermo
e6b4723790 Statically linked zxing. Without dependency on libMagick
BUG=webrtc:6633
NOTRY=True

Review-Url: https://codereview.webrtc.org/2653903002
Cr-Commit-Position: refs/heads/master@{#16243}
2017-01-24 13:02:35 +00:00
minyue
4b7c952376 Reland of "Log audio network adapter decisions in event log."
This was originally reviewed https://codereview.webrtc.org/2559953002/

It was reverted due to a bug in the original CL, see https://codereview.webrtc.org/2631703002/

This CL is to fix and reland.

BUG=webrtc:6845

Review-Url: https://codereview.webrtc.org/2644863002
Cr-Commit-Position: refs/heads/master@{#16242}
2017-01-24 12:54:59 +00:00
mbonadei
35a32700fc Moving webrtc.gni up one level from build/
BUG=webrtc:7030

Review-Url: https://codereview.webrtc.org/2651543003
Cr-Commit-Position: refs/heads/master@{#16241}
2017-01-24 12:49:35 +00:00
johan
62d02c328d Unit test out of band H264 SPS,PPS within RtpStreamReceiver.
This CL introduces a dedicated unit test for webrtc::RtpStreamReceiver.
Focus of this CL is testing RtpStreamReciver::OnReceivedPayloadData().
Dependencies with virtual interfaces are (g)mocked, non-virtual
dependencies are instantiated.

This CL is chained to https://codereview.webrtc.org/2638933002/ .

BUG=webrtc:5948

Review-Url: https://codereview.webrtc.org/2641463002
Cr-Commit-Position: refs/heads/master@{#16240}
2017-01-24 12:38:27 +00:00
mbonadei
822d2586fb Move webrtc/build/android -> tools-webrtc/android
BUG=webrtc:7030

Review-Url: https://codereview.webrtc.org/2649883004
Cr-Commit-Position: refs/heads/master@{#16239}
2017-01-24 12:22:11 +00:00
brandtr
81eab61172 Count FlexFEC packets in |fec_bitrate_| in RTPSenderVideo.
BUG=webrtc:5654

Review-Url: https://codereview.webrtc.org/2649913002
Cr-Commit-Position: refs/heads/master@{#16238}
2017-01-24 12:06:09 +00:00
buildbot
0608ffdcb4 Roll chromium_revision 59592eaa98..319b885718 (445345:445689)
Change log: 59592eaa98..319b885718
Full diff: 59592eaa98..319b885718

Changed dependencies:
* src/base: d23c26e094..cce542e3d7
* src/build: 92cbc7ad5a..bba7d3dfe3
* src/buildtools: cb12d6e864..8e94621c36
* src/testing: 52fa364615..ffe8a52fdd
* src/third_party: 56ef664776..d3d7bc6bb3
* src/third_party/libvpx/source/libvpx: 5b1a8ca5e8..f4169936ee
* src/tools: b12278e6ba..fa1e6e62b9
DEPS diff: 59592eaa98..319b885718/DEPS

No update to Clang.

TBR=marpan@webrtc.org,
BUG=None

Review-Url: https://codereview.webrtc.org/2651863003
Cr-Commit-Position: refs/heads/master@{#16237}
2017-01-24 11:08:53 +00:00
nisse
365aebdade Make CongestionController::remote_bitrate_estimator_ a non-pointer.
It was only assigned at construction, and this improves consistency
with remote_estimator_proxy_.

The declaration of the private WrappingBitrateEstimator had to be
moved to the header file, and it was also converted from
system_wrappers' CriticalSectionWrapper to rtc::CriticalSection.

BUG=webrtc:6847

Review-Url: https://codereview.webrtc.org/2642363003
Cr-Commit-Position: refs/heads/master@{#16236}
2017-01-24 10:53:38 +00:00
johan
d2b092f38a Reland of H264SpsPpsTracker.InsertSpsPpsNalus() should accept Nalus with header.
Fixed memory leak in test case.

Original issue's description
> Revert of H264SpsPpsTracker.InsertSpsPpsNalus() should accept Nalus with header. (patchset #3 id:40001 of https://codereview.webrtc.org/2638933002/ )
>
> Reason for revert:
> Triggers leak on Linux memcheck (non-default trybot):
>
> Original issue's description:
> > H264SpsPpsTracker.InsertSpsPpsNalus() should accept Nalus with header.
> >
> > - Changed method name to clarify that entire Nalus are expected.
> > - Added unit test code.
> > - Adjusted InsetSpsPpsNalus() implementation to above requirement.
> >
> > BUG=webrtc:5948
> >
> > Review-Url: https://codereview.webrtc.org/2638933002
> > Cr-Commit-Position: refs/heads/master@{#16221}
> > Committed: f53d7374cf
>
> TBR=philipel@webrtc.org,sprang@webrtc.org,johan@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:5948
> Review-Url: https://codereview.webrtc.org/2649113003
> Cr-Commit-Position: refs/heads/master@{#16225}
> Committed: 914d49d0fd
>

BUG=webrtc:5948

Review-Url: https://codereview.webrtc.org/2651593004
Cr-Commit-Position: refs/heads/master@{#16235}
2017-01-24 10:38:17 +00:00
nisse
15389c034d Drop pacer and retransmission_rate_limiter from RtpStreamReceiver constructor.
They were passed on via RtpRtcp::Configuration, but unused for a
receive only RtpRtcp module.

BUG=webrtc:6847

Review-Url: https://codereview.webrtc.org/2639423007
Cr-Commit-Position: refs/heads/master@{#16234}
2017-01-24 10:36:58 +00:00
terelius
568c9e72d1 New simulators to test BWE at low bitrates (15-50kbps range).
I've tested both the old and the new delay estimators and they all work ok at 15 kbps. The new estimators are a bit slower to adapt down so the maximum delay is a bit higher, especially at lower bitrates.

None of the estimators work at 10 kbps, but that is likely because the bitrate controller is configured to never go below 10 kbps. This means that it's impossible to empty the queues after a capacity drop to 10kbps (regardless of what the estimators do), so the delay stays high until the capacity increases.

BUG=webrtc:7022

Review-Url: https://codereview.webrtc.org/2644463009
Cr-Commit-Position: refs/heads/master@{#16233}
2017-01-24 10:03:31 +00:00
sakal
a4a753857e Android: Script for building libwebrtc.aar.
BUG=webrtc:7023

Review-Url: https://codereview.webrtc.org/2653533004
Cr-Commit-Position: refs/heads/master@{#16232}
2017-01-24 09:25:50 +00:00
aleloi
e04064deb2 Revert of Delete unused class/template ScopedMessageData. (patchset #1 id:1 of https://codereview.webrtc.org/2652663002/ )
Reason for revert:
ScopedMessageData can't be removed just yet. It broke an internal project.

Original issue's description:
> Delete unused class/template ScopedMessageData.
>
> This appears unused since cl https://codereview.webrtc.org/2564333002
>
> BUG=webrtc:6424
>
> Review-Url: https://codereview.webrtc.org/2652663002
> Cr-Commit-Position: refs/heads/master@{#16229}
> Committed: d83fb921a8

TBR=pthatcher@webrtc.org,deadbeef@webrtc.org,nisse@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6424

Review-Url: https://codereview.webrtc.org/2654753003
Cr-Commit-Position: refs/heads/master@{#16231}
2017-01-24 09:20:39 +00:00
nisse
dc2b3f3b9f Delete unused class CompositeMediaEngineWithFakeVoiceEngine.
BUG=None

Review-Url: https://codereview.webrtc.org/2645333002
Cr-Commit-Position: refs/heads/master@{#16230}
2017-01-24 08:54:59 +00:00