Commit Graph

23569 Commits

Author SHA1 Message Date
c5a2862a20 Roll chromium_revision 06e87fb3dc..62642dd6fe (580149:580526)
Change log: 06e87fb3dc..62642dd6fe
Full diff: 06e87fb3dc..62642dd6fe

Changed dependencies:
* src/base: f9ce552913..6b0c28c299
* src/build: fbf9211933..c1f37fdd0b
* src/ios: ff92b8db88..7addb925d2
* src/testing: 340252637e..8c05ec074f
* src/third_party: a42c5d9439..66bea27c42
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/c988ddf7b1..d38bbdff04
* src/third_party/depot_tools: 6f812e132d..29b7b99e5d
* src/tools: 0a1519743e..8449a143a6
DEPS diff: 06e87fb3dc..62642dd6fe/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: I4f9e15d24a9f55352c3a774242f0d6b9c8d6148d
Reviewed-on: https://webrtc-review.googlesource.com/92422
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24187}
2018-08-03 14:04:33 +00:00
827c63a5d7 Remove Syzygy from deps
It is unused;
it was removed in 96d692b3f7
and this is blocking DEPS roll.

Bug: None
Change-Id: Idb4ae4e43c35787e2f34111356b68e41f0bdd201
Reviewed-on: https://webrtc-review.googlesource.com/92388
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24186}
2018-08-03 13:37:55 +00:00
688f7f8fc2 Fix gitignore pattern to never exclude sha1 files
For example, currently "resources/audio_coding/F02_tlm10.OUT20.sha1" would have been ignored by the pattern "**/*.OUT*".

No-Try: True
Bug: None
Change-Id: I91243a301950485cb61d5f72a00af08372ec7951
Reviewed-on: https://webrtc-review.googlesource.com/92085
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Yves Gerey <yvesg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24185}
2018-08-03 12:45:45 +00:00
d54f5f5c76 Rename rtc_instrumentation_test_apk targets to end with _test_apk.
This matches Chromium pattern of naming instrumentation test apks with
a name ending in _test_apk. The old naming confuses generate_gradle.py.

Renames:
 - AppRTCMobileTest
   -> AppRTCMobile_test_apk
 - AppRTCMobileTestStubbedVideoIO
   -> AppRTCMobile_stubbed_video_io_test_apk
 - libjingle_peerconnection_android_unittest
   -> android_instrumentation_test_apk

Bug: webrtc:9588
TBR: phoglund
Change-Id: Idb82dc4bd089bc7c90e9373f7c3d572f9fd2d95a
Reviewed-on: https://webrtc-review.googlesource.com/92380
Commit-Queue: Sami Kalliomäki <sakal@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24184}
2018-08-03 12:44:40 +00:00
0cbe05cc86 Android: Remove custom matrix helper functions in RendererCommon
Bug: webrtc:9487
Change-Id: I2b5720d55cae9684a7ef2b14cabce262a5321ef0
Reviewed-on: https://webrtc-review.googlesource.com/87820
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Commit-Queue: Magnus Jedvert <magjed@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24183}
2018-08-03 10:57:07 +00:00
489767830b Remove definition of FEATURE_ENABLE_VOICEMAIL, FEATURE_ENABLE_PSTN.
FEATURE_ENABLE_VOICEMAIL is never used.

FEATURE_ENABLE_PSTN is used in //third_party/libjingle_xmpp but WebRTC
doesn't depend on it, so it is reasonable to assume that no one needs
to define it.

Bug: webrtc:9564
Change-Id: Idfb04081f497ef52fc5c140ffb82fa2dc7b9824d
Reviewed-on: https://webrtc-review.googlesource.com/92081
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24182}
2018-08-03 10:08:21 +00:00
7d745e5a89 Reland "Remove RTPVideoHeader::h264() accessors."
Downstream projects have been updated, so this can now be relanded.
This is a revert (and rebase) of: https://webrtc-review.googlesource.com/c/src/+/88820

Bug: none
Change-Id: I424664ddef7aeebd3c6c94ae67c7f70a342dc9a4
Reviewed-on: https://webrtc-review.googlesource.com/92082
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Commit-Queue: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24181}
2018-08-03 09:16:50 +00:00
9f6450d5a5 JNI generation: Replace base::subtle::AtomicWord with std::atomic<>
This also rolls up //base in DEPS, because it needs to be landed together with
54f759310c

Bug: chromium:867475
Change-Id: I5792cb0610d2df46a9368fd3b1846583aa134b38
Reviewed-on: https://webrtc-review.googlesource.com/90404
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24180}
2018-08-03 08:54:30 +00:00
e8b4e7e53c Roll chromium_revision 4dd959af57..06e87fb3dc (579686:580149)
Change log: 4dd959af57..06e87fb3dc
Full diff: 4dd959af57..06e87fb3dc

Changed dependencies:
* src/base: 4b0b86b8b4..e15177f81a
* src/build: e9eade234d..fbf9211933
* src/ios: 69485848c6..ff92b8db88
* src/testing: 39667a68df..340252637e
* src/third_party: 74ddeed04a..a42c5d9439
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/5c2d9e757a..c988ddf7b1
* src/third_party/gtest-parallel: cb3514a085..fe7f791f14
* src/tools: e26055aead..0a1519743e
DEPS diff: 4dd959af57..06e87fb3dc/DEPS

Clang version changed 337439:338452
Details: 4dd959af57..06e87fb3dc/tools/clang/scripts/update.py

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: I626f49a90bcf67b2ab10970bdfb02ddb75dc7387
Reviewed-on: https://webrtc-review.googlesource.com/92100
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24179}
2018-08-03 08:00:09 +00:00
7d984ee1d7 Don't allocate an array of size 0, it causes a UBSan failure
Bug: webrtc:9587
Change-Id: I56bdf3c5c8744044b2d0d1fa3531fca504ea200f
Reviewed-on: https://webrtc-review.googlesource.com/92091
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24178}
2018-08-03 07:49:46 +00:00
54bd8f54e9 Remove dead code.
This code never executes as we always get passed a nil codecSpecificInfo.

Bug: webrtc:9580
Change-Id: I5c5311c20877494978df45d409a53ad5b0e86a9b
Reviewed-on: https://webrtc-review.googlesource.com/92083
Reviewed-by: Peter Hanspers <peterhanspers@webrtc.org>
Commit-Queue: Kári Helgason <kthelgason@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24177}
2018-08-03 07:10:14 +00:00
e20867ff6d Add AsyncResolverFactory interface and basic implementation.
The factory is plumbed down to P2PTransportChannel and will eventually
be used to resolve hostnames. Uses of PacketSocketFacotry::CreateAsyncResolver
will eventually be migrated to use this factory instead.

Bug: webrtc:4165
Change-Id: I1c48b2ffb8649609a831eba291f67ce544bb10eb
Reviewed-on: https://webrtc-review.googlesource.com/91300
Commit-Queue: Zach Stein <zstein@webrtc.org>
Reviewed-by: Emad Omara <emadomara@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Qingsi Wang <qingsi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24176}
2018-08-02 21:20:15 +00:00
da2ec40590 Always sends probes when they are generated.
This changes makes the usage of the new probe controller reflect how the
old probe controller was used. That is probes are now sent as soon as
they are generated. This is to avoid regressions in performance doe to
the timing of the sent probes.

Bug: chromium:868776
Change-Id: I722585689258c9b01e8f1dc47249b284a05a2793
Reviewed-on: https://webrtc-review.googlesource.com/91441
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24175}
2018-08-02 15:36:12 +00:00
dc6e68b4a7 Delete class TelephoneEventHandler and related code.
Followup to https://webrtc-review.googlesource.com/91125.

Bug: webrtc:7135
Change-Id: I7011cc65ac756931d8134763da57ec1bc9c584d6
Reviewed-on: https://webrtc-review.googlesource.com/91163
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24174}
2018-08-02 15:02:23 +00:00
ee1e74fb86 Fix occasional crash in iOS ADM.
RTCNativeAudioSessionDelegateAdapter has a raw pointer to AudioDeviceIOS,
and receives callbacks from RTCAudioSession and forwards them to AudioDeviceIOS.

During teardown of these components the situation can occur that the dtor for
AudioDeviceIOS has been called but the ObjC runtime has not yet dealloced
RTCNativeAudioSessionDelegateAdapter, so it's still receiving callbacks while
the pointer it keeps to AudioDeviceIOS has been invalidated.

This occasionally triggers a crash when WebRTC is shutting down.

The fix in this CL is to make sure to deregister the adapter from RTCAudioSession
_before_ the dtor for AudioDeviceIOS returns.

Bug: webrtc:9523
Change-Id: Ica85420d76efc63940472bc43e3ec71d16036ccf
Reviewed-on: https://webrtc-review.googlesource.com/90245
Reviewed-by: Peter Hanspers <peterhanspers@webrtc.org>
Commit-Queue: Kári Helgason <kthelgason@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24173}
2018-08-02 14:25:37 +00:00
ab4a530b87 Delete telephone-event handling from RTPReceiverAudio.
Bug: webrtc:7135
Change-Id: Ic8b96f44ba25ff9265570dd43d3c76ed0177abfb
Reviewed-on: https://webrtc-review.googlesource.com/91125
Reviewed-by: Oskar Sundbom <ossu@webrtc.org>
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24172}
2018-08-02 12:55:40 +00:00
31f1596c7c Fix a warning about comparison of integers of different signs
Bug: webrtc:9583
Change-Id: I94df0efa0a0fd5666a42dbd6997ac1b00b3844bd
Reviewed-on: https://webrtc-review.googlesource.com/92040
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24171}
2018-08-02 12:23:36 +00:00
f8d81d33ed Add members for the codec agnostic descriptor to RTPVideoHeader.
TBR=danilchap@webrtc.org

Bug: webrtc:9361, webrtc:9582
Change-Id: I0303fc89bafab59e68ec81979e0e4372e79a4f51
Reviewed-on: https://webrtc-review.googlesource.com/91866
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Commit-Queue: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24170}
2018-08-02 09:12:31 +00:00
e468ba77ae Delete MockRenderer and replace it with identical MockSink class.
This was left by a mistake in a previous refactoring.

R=magjed

Bug: None
Change-Id: Ia2b469e730844780fa3b9ce5540d4bdd4d10b556
Reviewed-on: https://webrtc-review.googlesource.com/91480
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Sami Kalliomäki <sakal@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24169}
2018-08-02 08:29:31 +00:00
f5cf02ac2c Use randomly generated IDs for candidate pairs in ICE event logs.
We used to use an integer-valued hash as the candidate pair ID in ICE
event logs, and the preimage of this hash contains address information.

Bug: None
Change-Id: Ib24aa89164600c62e0b0a7d771af379ace80a0e3
Reviewed-on: https://webrtc-review.googlesource.com/91920
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Qingsi Wang <qingsi@webrtc.org>
Commit-Queue: Qingsi Wang <qingsi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24168}
2018-08-01 20:08:43 +00:00
d3b7ec2e91 Allow all "token" chars from RFC 4566 when checking for legal mid names.
Previously only alphanumeric characters were allowed.

Bug: webrtc:9537
Change-Id: I3fd793ad88520b25ecd884efe3a698f2f0af4639
Reviewed-on: https://webrtc-review.googlesource.com/89388
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24167}
2018-08-01 18:20:42 +00:00
78026754a7 AEC3: Utilize shadow filter output to respond to audio path changes
This CL adds functionality to use the shadow filter output instead
of the main filter output for cases when the former is better than
the latter. One case when that happens is when there have been an
echo path change, either in the acoustic path, in the audio buffers
or due to some active audio processing effects being applied on
the device.

The CL causes less echo leaks, in particular on devices with
active render processing.

Bug: webrtc:9581,chromium:869821
Change-Id: Icb8df1b94141598da82dc188051ac59e43338938
Reviewed-on: https://webrtc-review.googlesource.com/91820
Reviewed-by: Sam Zackrisson <saza@webrtc.org>
Commit-Queue: Per Åhgren <peah@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24166}
2018-08-01 15:20:33 +00:00
d2f4e8bd90 Explicitly add -mfpu=neon to all targets that use NEON
Remove obsolete comment about Chromium not defining NEON for Android.

Semi-related fix: don't use `rtc_remove_configs` directly, `suppressed_configs` is the "public interface".

Bug: webrtc:9579
Change-Id: I512628feb462a29432f1356cfef00efe1ddaf84f
Reviewed-on: https://webrtc-review.googlesource.com/91761
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Per Åhgren <peah@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24165}
2018-08-01 13:15:42 +00:00
7a1c7f782a Modified peerconnection's "observer" slot to be nulled on close.
This prevents usage of the observer post-close; modified the "usage
report notification" handler to not report when called post-close.
This fits the description of the original bug, so likely fixes it.

Bug: chromium:868337
Change-Id: Ic6757d2fb335203a6a6aacb2c9b52854b40332f7
Reviewed-on: https://webrtc-review.googlesource.com/91121
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Guido Urdaneta <guidou@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24164}
2018-08-01 09:44:57 +00:00
9014324bb1 Support compiling with the lastest iOS SDK.
Bug: None
Change-Id: I2bc4b4f3eba9c5f6b3a94fce076dc575c5be057d
Reviewed-on: https://webrtc-review.googlesource.com/90720
Reviewed-by: Peter Hanspers <peterhanspers@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Kári Helgason <kthelgason@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24163}
2018-08-01 09:17:59 +00:00
55bf92adf4 RNN VAD: more specific build target names.
Bug: webrtc:9076
Change-Id: Ie35ce0f864318a1ddc552285a5535fe411168202
Reviewed-on: https://webrtc-review.googlesource.com/91760
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Per Åhgren <peah@webrtc.org>
Commit-Queue: Alessio Bazzica <alessiob@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24162}
2018-08-01 09:07:26 +00:00
f90528673a The pixel buffer pool is currently recreated on every call to encode.
After this change, it is only recreated when needed.

This change also clarifies the relation between the compression
session and the pixel buffer pool, and handles invalid sessions
explicitly.

Change-Id: Iae4aa02b60b0d5c153db3ae2d4cd2a0cfa05757b
Bug: webrtc:9562
Reviewed-on: https://webrtc-review.googlesource.com/90403
Reviewed-by: Kári Helgason <kthelgason@webrtc.org>
Commit-Queue: Peter Hanspers <peterhanspers@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24161}
2018-08-01 08:54:24 +00:00
e3a38421cf Include RTCMTLVideoView even on archs without metal.
This makes fixes some weirdness with the order of architectures in
fat binaries, and with the umbrella header generation.

Bug: webrtc:9573
Change-Id: Ibf4318c761da58e724820eba623d37b4739e6860
Reviewed-on: https://webrtc-review.googlesource.com/91421
Reviewed-by: Anders Carlsson <andersc@webrtc.org>
Commit-Queue: Kári Helgason <kthelgason@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24160}
2018-08-01 08:31:30 +00:00
fc25ccb34d Reland "Delete root header file typedefs.h. Part 2."
This is a reland of 4e307f2d880152bbf2a102c8279ebaad7b0aabd7

Usage in chromium was deleted in cl https://chromium-review.googlesource.com/c/chromium/src/+/1156388

Original change's description:
> Delete root header file typedefs.h. Part 2.
>
> This is a followup to
> https://webrtc-review.googlesource.com/c/src/+/90249, which deleted
> all uses of the typedefs.h throughout the source tree, but didn't
> actually delete the file itself.
>
> This cl deletes the file, and also deletes mention of it in the top-level
> DEPS file.
>
> TBR: kwiberg@webrtc.org
> Bug: webrtc:6854
> Change-Id: I90bb72961ff7e9f3ac8b7123cfc1177143d77a20
> Reviewed-on: https://webrtc-review.googlesource.com/90410
> Reviewed-by: Niels Moller <nisse@webrtc.org>
> Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
> Commit-Queue: Niels Moller <nisse@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#24149}

TBR: kwiberg@webrtc.org
Bug: webrtc:6854
Change-Id: I732a405402647dda6c3890a0540978da453440e7
Reviewed-on: https://webrtc-review.googlesource.com/91740
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24159}
2018-08-01 08:28:28 +00:00
6e250820b1 Removing rtc_base:base build target.
This build target is unused and outdated (the name was still base
because rtc_base is the real target). It is better for users to
depend on real targets instead of this group.

Bug: None
Change-Id: I870669dc8688810b132616b8edebb8196826701f
Reviewed-on: https://webrtc-review.googlesource.com/91720
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24158}
2018-08-01 08:10:52 +00:00
da3dbe83a3 Roll chromium_revision bf3721aec7..4dd959af57 (579581:579686)
Change log: bf3721aec7..4dd959af57
Full diff: bf3721aec7..4dd959af57

Changed dependencies:
* src/base: 7c5fb81ff2..4b0b86b8b4
* src/build: 148c8b5dba..e9eade234d
* src/ios: 72dd36f3b4..69485848c6
* src/testing: 7c11d7b6cd..39667a68df
* src/third_party: 29f5eecf5e..74ddeed04a
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/100f0ee2ce..5c2d9e757a
* src/third_party/depot_tools: 1aefad2a59..6f812e132d
* src/tools: 5d32618eb7..e26055aead
DEPS diff: bf3721aec7..4dd959af57/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: Ie6e6ee221492c5e8fe73c1e9e77fc34a6558a6ed
Reviewed-on: https://webrtc-review.googlesource.com/91701
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24157}
2018-08-01 04:15:08 +00:00
1a9c028506 Roll chromium_revision 19a9e3bb00..bf3721aec7 (579465:579581)
Change log: 19a9e3bb00..bf3721aec7
Full diff: 19a9e3bb00..bf3721aec7

Changed dependencies:
* src/base: 51d2ddff76..7c5fb81ff2
* src/ios: 7fd42c2a94..72dd36f3b4
* src/testing: e7fff15f90..7c11d7b6cd
* src/third_party: c3719302e9..29f5eecf5e
* src/tools: c0a441e9a1..5d32618eb7
DEPS diff: 19a9e3bb00..bf3721aec7/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: I3ff7b1a73e0fbacc659472664a54cf5fae477ccd
Reviewed-on: https://webrtc-review.googlesource.com/91600
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24156}
2018-07-31 22:08:28 +00:00
68be5dab98 Roll chromium_revision 36ff521efe..19a9e3bb00 (579363:579465)
Change log: 36ff521efe..19a9e3bb00
Full diff: 36ff521efe..19a9e3bb00

Changed dependencies:
* src/base: 9ca0da16a1..51d2ddff76
* src/build: fe020ad573..148c8b5dba
* src/ios: c9a5fc747a..7fd42c2a94
* src/testing: 3d1c2c9a44..e7fff15f90
* src/third_party: 30ff371b22..c3719302e9
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/65f883bb77..100f0ee2ce
* src/tools: f0401962e3..c0a441e9a1
DEPS diff: 36ff521efe..19a9e3bb00/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: I2449351b1c24e68a031cf1e550aaa194791a26cd
Reviewed-on: https://webrtc-review.googlesource.com/91520
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24155}
2018-07-31 18:15:43 +00:00
45a4c41eda Never invoke rtc::LogMessage::SetLogToStderr outside of main.
rtc::LogMessage::SetLogToStderr should only be invoked by the main
function in order to enable or disable logging in a consistent way [1].

Usage of rtc::LogMessage::SetLogToStderr in other parts of the codebase
creates complex behaviors and confusion.

[1] - https://cs.chromium.org/chromium/src/third_party/webrtc/test/test_main.cc?l=88&rcl=665174fdbb4e0540eccb27cf7412348f1b65534c

Bug: None
Change-Id: Iae86fb14d7ca40af6d78d0f0cd81c5a39f65068d
Reviewed-on: https://webrtc-review.googlesource.com/91442
Reviewed-by: Erik Språng <sprang@webrtc.org>
Reviewed-by: Jonas Olsson <jonasolsson@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24154}
2018-07-31 17:24:09 +00:00
2a99c0bf67 Fix MovingMoments::CalculateMoments.
Protect from negative second moments, which are unexpected in TransientDetector::Detect
and may lead to invalid results.

Bug: chromium:866925
Change-Id: Id1d5b2ebb51e54d9d332b869c6f63dcd03cc461c
Reviewed-on: https://webrtc-review.googlesource.com/91164
Commit-Queue: Alessio Bazzica <alessiob@webrtc.org>
Reviewed-by: Sam Zackrisson <saza@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24153}
2018-07-31 15:08:12 +00:00
52233a3f28 Increase RtpFrameReferenceFinder's frame buffer length to 100 frames.
This mitigates the long freeze issue caused by overflow of frame
buffer in RtpFrameReferenceFinder and subsequent removal of old, but
not yet decoded frames, from the buffer.

Bug: webrtc:9550
Change-Id: I03390bb58847688c6cb3f4868bf21269ad07073a
Reviewed-on: https://webrtc-review.googlesource.com/91124
Commit-Queue: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24152}
2018-07-31 13:41:25 +00:00
b61f83e23f Roll chromium_revision fb3a9711ab..36ff521efe (579262:579363)
Change log: fb3a9711ab..36ff521efe
Full diff: fb3a9711ab..36ff521efe

Changed dependencies:
* src/build: 0549427f8e..fe020ad573
* src/ios: e72a4dcd0f..c9a5fc747a
* src/testing: c59763c7c7..3d1c2c9a44
* src/third_party: bb787d3733..30ff371b22
* src/third_party/depot_tools: b0e855a333..1aefad2a59
* src/third_party/freetype/src: b532d7ce70..578bcf103a
* src/tools: 77a27c3bb2..f0401962e3
DEPS diff: fb3a9711ab..36ff521efe/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: Ib1375aed3831ac4f8f0767b0ecde478180683bd6
Reviewed-on: https://webrtc-review.googlesource.com/91407
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24151}
2018-07-31 10:07:06 +00:00
304da796ab Revert "Delete root header file typedefs.h. Part 2."
This reverts commit 4e307f2d880152bbf2a102c8279ebaad7b0aabd7.

Reason for revert: Breaks chromium, file was included in third_party/libjingle_xmpp/xmpp/xmpplogintask_unittest.cc

Original change's description:
> Delete root header file typedefs.h. Part 2.
> 
> This is a followup to
> https://webrtc-review.googlesource.com/c/src/+/90249, which deleted
> all uses of the typedefs.h throughout the source tree, but didn't
> actually delete the file itself.
> 
> This cl deletes the file, and also deletes mention of it in the top-level
> DEPS file.
> 
> TBR: kwiberg@webrtc.org
> Bug: webrtc:6854
> Change-Id: I90bb72961ff7e9f3ac8b7123cfc1177143d77a20
> Reviewed-on: https://webrtc-review.googlesource.com/90410
> Reviewed-by: Niels Moller <nisse@webrtc.org>
> Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
> Commit-Queue: Niels Moller <nisse@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#24149}

TBR=solenberg@webrtc.org,mbonadei@webrtc.org,kwiberg@webrtc.org,nisse@webrtc.org

Change-Id: I9c3e2099ecd5bf848827f89374fe1a0488a98823
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: webrtc:6854
Reviewed-on: https://webrtc-review.googlesource.com/91420
Reviewed-by: Niels Moller <nisse@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24150}
2018-07-31 09:19:52 +00:00
4e307f2d88 Delete root header file typedefs.h. Part 2.
This is a followup to
https://webrtc-review.googlesource.com/c/src/+/90249, which deleted
all uses of the typedefs.h throughout the source tree, but didn't
actually delete the file itself.

This cl deletes the file, and also deletes mention of it in the top-level
DEPS file.

TBR: kwiberg@webrtc.org
Bug: webrtc:6854
Change-Id: I90bb72961ff7e9f3ac8b7123cfc1177143d77a20
Reviewed-on: https://webrtc-review.googlesource.com/90410
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24149}
2018-07-31 09:01:10 +00:00
df9e9ae9ee Delete RtpVideoStreamReceiver methods GetRtpReceiver and rtp_rtcp
Replaced by new method GetSyncInfo.

Bug: webrtc:7135
Change-Id: I541567a5ca173dc334fd85e83f15b25a3120b8aa
Reviewed-on: https://webrtc-review.googlesource.com/91123
Commit-Queue: Niels Moller <nisse@webrtc.org>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24148}
2018-07-31 07:02:18 +00:00
168b4977f3 Roll chromium_revision fdf31b15b2..fb3a9711ab (579160:579262)
Change log: fdf31b15b2..fb3a9711ab
Full diff: fdf31b15b2..fb3a9711ab

Changed dependencies:
* src/build: b425156dd9..0549427f8e
* src/ios: 25918ccac3..e72a4dcd0f
* src/testing: c076780de8..c59763c7c7
* src/third_party: 1fdaaa480a..bb787d3733
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/0f2c4fec12..65f883bb77
* src/third_party/depot_tools: d795ab891c..b0e855a333
* src/tools: 22b35fcfb9..77a27c3bb2
DEPS diff: fdf31b15b2..fb3a9711ab/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: I8f2b840d6cf41539b98e50b6ef9e814e0e13902a
Reviewed-on: https://webrtc-review.googlesource.com/91360
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24147}
2018-07-31 01:07:13 +00:00
ef5d5af3a0 AEC3: Increasing the accuracy of the detection for early reverb
This CL introduces an adaptive estimation of the early reverb
in the estimation for the room reverberation. The benefits of
this is that for room with long early reflections there is
a lower risk of underestimating the reverberation.

This CL is for a landing the code in
https://webrtc-review.googlesource.com/c/src/+/87420,
and the review of the code was done in that CL. The author of
code is devicentepena@webrtc.org

Bug: webrtc:9479, chromium:865397
Change-Id: Id6f57e2a684664aef96e8c502e66775f37da59da
Reviewed-on: https://webrtc-review.googlesource.com/91162
Commit-Queue: Per Åhgren <peah@webrtc.org>
Reviewed-by: Sam Zackrisson <saza@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24146}
2018-07-30 22:34:19 +00:00
1899c1270f Roll chromium_revision 0e06bcbb6f..fdf31b15b2 (579047:579160)
Change log: 0e06bcbb6f..fdf31b15b2
Full diff: 0e06bcbb6f..fdf31b15b2

Changed dependencies:
* src/base: 8ea7161f59..9ca0da16a1
* src/build: 1f59651baf..b425156dd9
* src/buildtools: 0dd5c6f980..9a90d9aaad
* src/ios: f1ebe8e9da..25918ccac3
* src/testing: ab0ab287dd..c076780de8
* src/third_party: d7fdf61870..1fdaaa480a
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/dd9e5b9c8c..0f2c4fec12
* src/third_party/libvpx/source/libvpx: 3b921d49b0..2d79df4940
* src/tools: 75087970bb..22b35fcfb9
DEPS diff: 0e06bcbb6f..fdf31b15b2/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,marpan@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: I12357feb87153c19b2c0e7c49fe7e0078c39b26f
Reviewed-on: https://webrtc-review.googlesource.com/91280
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24145}
2018-07-30 21:09:35 +00:00
f452e3e588 Roll chromium_revision 2d5685c126..0e06bcbb6f (578940:579047)
Change log: 2d5685c126..0e06bcbb6f
Full diff: 2d5685c126..0e06bcbb6f

Changed dependencies:
* src/build: b222950442..1f59651baf
* src/ios: a90302ddad..f1ebe8e9da
* src/third_party: 1430c5903d..d7fdf61870
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/e05f902a69..dd9e5b9c8c
* src/tools: 95134bd3c0..75087970bb
DEPS diff: 2d5685c126..0e06bcbb6f/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: I3e7a90ecd226db5d8dc348a3ed9caf9194d5b294
Reviewed-on: https://webrtc-review.googlesource.com/91182
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24144}
2018-07-30 16:07:51 +00:00
c3d0d0d330 Delete unused RtpVideoStreamReceiver methods GetRemoteSsrc and GetCsrcs
Bug: webrtc:7135
Change-Id: I410939299f1cded9e103206b0ac883af05f360ff
Reviewed-on: https://webrtc-review.googlesource.com/91122
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24143}
2018-07-30 15:02:03 +00:00
731a2c2dc6 Convert webrtcvideoengine CVO tests away from cricket::VideoCapturer.
Bug: webrtc:6353
Change-Id: I1f4f705cda4fdf88465395898e2588b2a19eebf3
Reviewed-on: https://webrtc-review.googlesource.com/83324
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24142}
2018-07-30 14:41:23 +00:00
1bd66642c3 Set RtpReceiverAudio::telephone_event_forward_to_decoder_ true on construction.
All users call SetTelephoneEventForwardToDecoder(true). Setting the
flag to true on construction, enables deletion of those calls,
followed by deletion of the flag itself.

The unused getter method TelephoneEventForwardToDecoder() is deleted
right away.

Bug: webrtc:7135
Change-Id: I8c52c957b3f074be7ffc425b3588402d1e42b844
Reviewed-on: https://webrtc-review.googlesource.com/90402
Reviewed-by: Oskar Sundbom <ossu@webrtc.org>
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24141}
2018-07-30 12:24:49 +00:00
009d096986 Add missing dependencies to sdk/android:native_unittests_java.
Bug: None
Change-Id: I5247263804bc7eb8c2cef687adae645c24464e2f
Reviewed-on: https://webrtc-review.googlesource.com/90869
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24140}
2018-07-30 12:11:59 +00:00
d1d8f9815d Roll chromium_revision 743fc877e0..2d5685c126 (578837:578940)
Change log: 743fc877e0..2d5685c126
Full diff: 743fc877e0..2d5685c126

Changed dependencies:
* src/base: 546ed8fec4..8ea7161f59
* src/build: da2375258a..b222950442
* src/ios: 98b5db447e..a90302ddad
* src/testing: a9069c15d5..ab0ab287dd
* src/third_party: 0b736ecf97..1430c5903d
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/b9813d2fdd..e05f902a69
* src/tools: ade436a212..95134bd3c0
DEPS diff: 743fc877e0..2d5685c126/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: Iccb753b2e86116c9db07baa8d711f8e86c1b27be
Reviewed-on: https://webrtc-review.googlesource.com/91075
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24139}
2018-07-28 23:07:42 +00:00
42c75b14f4 Roll chromium_revision 8c2846e52f..743fc877e0 (578726:578837)
Change log: 8c2846e52f..743fc877e0
Full diff: 8c2846e52f..743fc877e0

Changed dependencies:
* src/base: 51d33cac77..546ed8fec4
* src/build: 2178d78be2..da2375258a
* src/testing: 6e3e0e8c03..a9069c15d5
* src/third_party: 4182cc41fe..0b736ecf97
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/d8072e683a..b9813d2fdd
* src/third_party/depot_tools: f12600a8ba..d795ab891c
* src/tools: 82bdd248f4..ade436a212
DEPS diff: 8c2846e52f..743fc877e0/DEPS

No update to Clang.

TBR=buildbot@webrtc.org,
BUG=None
CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal

Change-Id: If77c045a29eb961accd4ea8ab577e0ca36b37fb5
Reviewed-on: https://webrtc-review.googlesource.com/91000
Reviewed-by: WebRTC Buildbot <buildbot@webrtc.org>
Commit-Queue: WebRTC Buildbot <buildbot@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24138}
2018-07-27 23:04:52 +00:00