Commit Graph

34793 Commits

Author SHA1 Message Date
a18cad9c09 Add compile-time warning for RTC_LOG(INFO)
Bug: webrtc:13362
Change-Id: I3727891bfeb192941f249a74ead5ef38ac8acc2f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238800
Reviewed-by: Niels Moller <nisse@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35411}
2021-11-23 21:46:38 +00:00
6c3c3db184 Roll chromium_revision 866506397b..1ce7d592c3 (944390:944520)
Change log: 866506397b..1ce7d592c3
Full diff: 866506397b..1ce7d592c3

Changed dependencies
* src/base: 379395e10f..a0e30222a6
* src/build: 368880db83..749ecdaeea
* src/ios: f5aa54daec..d9a982f504
* src/testing: 83d4511de4..4e888a310b
* src/third_party: 9a1aa8fd33..7d86ceacba
* src/third_party/androidx: jXiiROMn0WffhrxtV5WAwhgkPccRtz7S0sTKt5pboZsC..ftwyhL300WgpRNfRASqCL9olp8f0SJTlzIAgKWR_lwUC
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/7ae3bc4766..4b9301e9c5
* src/tools: 91e4d58d2c..0e9c2f5abf
DEPS diff: 866506397b..1ce7d592c3/DEPS

No update to Clang.

BUG=None

Change-Id: I096f3f8ff7c09d4e187180942c2c319bc13e5725
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238820
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35410}
2021-11-23 16:35:18 +00:00
b09d87232b Reland "Add dav1d decoder to WebRTC."
This reverts commit 8498b7e7f6b90fa036de2a6887d34256f0565b4f.

Reason for revert: Updating CL to include conditional build flag.

Original change's description:
> Revert "Add dav1d decoder to WebRTC."
>
> This reverts commit 147858577d4db6d257d3cc248fe571a1bbf887e3.
>
> Reason for revert: High binary size increase
>
> Original change's description:
> > Add dav1d decoder to WebRTC.
> >
> > Bug: none
> > Change-Id: I7642f42e592dcf510679f881f118bc4dab93b31c
> > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237504
> > Commit-Queue: Philip Eliasson <philipel@webrtc.org>
> > Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> > Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
> > Reviewed-by: Magnus Flodman <mflodman@webrtc.org>
> > Reviewed-by: Sergey Silkin <ssilkin@webrtc.org>
> > Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
> > Cr-Commit-Position: refs/heads/main@{#35394}
>
> TBR=danilchap@webrtc.org,mbonadei@webrtc.org,ilnik@webrtc.org,philipel@webrtc.org,mflodman@webrtc.org,ssilkin@webrtc.org,webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com
>
> Change-Id: I00a8acd6ea94ce523c2d5ba705333c9174678180
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: none
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238560
> Reviewed-by: Artem Titov <titovartem@webrtc.org>
> Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> Owners-Override: Mirko Bonadei <mbonadei@webrtc.org>
> Owners-Override: Artem Titov <titovartem@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#35395}

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: none
Change-Id: Iff51848731646159e87e075c38af7cb6355f5b5b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238661
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Magnus Flodman <mflodman@webrtc.org>
Reviewed-by: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35409}
2021-11-23 15:59:58 +00:00
707e5a0cd7 Make test framework create portallocator with an explicit PacketSocketFactory.
Bug: webrtc:13145
Change-Id: I04575517b1e215a2204611415f728c358c8d64fe
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238660
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35408}
2021-11-23 15:00:33 +00:00
aad4ba102b Fix windows bot command line issue.
When wrapping test_env.py command on windows bot, there is an error because the python command is missing.
Adding vpython in the command like it is done on chromium mb.py script fixes the problem.

Bug: b/197492097
Change-Id: I91dbad31549fb29058424ca3b2fb8539c3e8010e
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238760
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35407}
2021-11-23 13:56:03 +00:00
45e15e3343 Prepare for migrating to new AddPeer method
Bug: webrtc:13145
Change-Id: I089d518e55cb8df32ddf3c587f82376226c18e9a
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238761
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35406}
2021-11-23 13:04:44 +00:00
646a90784e [DVQA] Remove old constructor from StatsKey
Bug: b/205824594
Change-Id: I081ca48978ce498c56f47022e28fa0a66846765f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238667
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35405}
2021-11-23 12:30:02 +00:00
6d057976e9 StatsEndToEndTests: Remove unneeded fake renderer in tests.
Bug: none
Change-Id: I1f14fa2e606dde38113560c32640fb6b2996ea14
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238428
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Åsa Persson <asapersson@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35404}
2021-11-23 11:10:04 +00:00
f47a724168 New struct PeerNetworkDependencies
Preparation to make landing of
https://webrtc-review.googlesource.com/c/src/+/238660
easier.

Bug: webrtc:13145
Change-Id: I314a53cc634f842e5df009d0802b214aa6f8728b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238663
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Gustaf Ullberg <gustaf@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35403}
2021-11-23 08:37:36 +00:00
89f1aa59b6 Roll chromium_revision 0755db8caa..866506397b (944150:944390)
Change log: 0755db8caa..866506397b
Full diff: 0755db8caa..866506397b

Changed dependencies
* src/base: e704213601..379395e10f
* src/build: 5867e704e3..368880db83
* src/ios: ee53859d89..f5aa54daec
* src/testing: 7597ad2c88..83d4511de4
* src/third_party: 55fa5116e5..9a1aa8fd33
* src/third_party/android_deps/libs/org_jetbrains_kotlin_kotlin_stdlib: version:2@1.5.31.cr0..version:2@1.6.0.cr0
* src/third_party/android_deps/libs/org_jetbrains_kotlin_kotlin_stdlib_common: version:2@1.5.31.cr0..version:2@1.6.0.cr0
* src/third_party/androidx: 5_6hGqfW9hh3pWzd73P3qsKKKFcswNcKOHp0z_Xen94C..jXiiROMn0WffhrxtV5WAwhgkPccRtz7S0sTKt5pboZsC
* src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/69030a0cea..3a667d10e9
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/8a112e2bdf..7ae3bc4766
* src/third_party/depot_tools: b6c1ed40d5..a29f589a15
* src/third_party/freetype/src: 81912a1385..3cabd142ce
* src/third_party/perfetto: 715528ea10..d5cb19a57d
* src/tools: 020ad36eaf..91e4d58d2c
DEPS diff: 0755db8caa..866506397b/DEPS

No update to Clang.

BUG=None

Change-Id: I680205721eae6b0fd3de0e34d253d6bfabea76e1
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238700
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35402}
2021-11-23 06:40:06 +00:00
f05f2823b3 Allow setting volume on unstarted AudioRtpReceivers
This was the behaviour before https://webrtc-review.googlesource.com/c/src/+/218605,
and is currently relied upon by Chrome to mute received audio tracks
by default, until they should be played out.

Bug: chromium:1272566
Change-Id: I8a288a287e7c01392f4af1db5b083e8d7ee7b2a8
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238665
Commit-Queue: Tony Herre <toprice@chromium.org>
Reviewed-by: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35401}
2021-11-22 18:58:56 +00:00
a9730e5292 Roll chromium_revision 2b49f0a621..0755db8caa (944047:944150)
Change log: 2b49f0a621..0755db8caa
Full diff: 2b49f0a621..0755db8caa

Changed dependencies
* src/base: b53fedf715..e704213601
* src/build: b8e6a02717..5867e704e3
* src/ios: ad4a582ec6..ee53859d89
* src/testing: 4f31c40149..7597ad2c88
* src/third_party: a57f1dde5f..55fa5116e5
* src/third_party/androidx: wgGrIASbzI25xEdOD1slUhYwmBa_FcdH498LBPPTwBgC..5_6hGqfW9hh3pWzd73P3qsKKKFcswNcKOHp0z_Xen94C
* src/third_party/perfetto: 03cb3fd2f9..715528ea10
* src/tools: ff95136344..020ad36eaf
DEPS diff: 2b49f0a621..0755db8caa/DEPS

No update to Clang.

BUG=None

Change-Id: I6068c4d828f4f56253b0271caa61ed8445a872f9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238682
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35400}
2021-11-22 18:31:17 +00:00
8ef7da78b3 [DVQA] Remove sender from DVQA StatsKey
Sender isn't actually require to identify the stream, so specifying it
every time is useless. This CL removes sender from StatsKey object and
introduces StreamsInfo object which contains all required metadata about
streams that are seen by DVQA.

Bug: b/205824594
Change-Id: I5b6be3865a30fd5980ff6e7e50906abe70a632ee
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238562
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Reviewed-by: Per Kjellander <perkj@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35399}
2021-11-22 14:42:13 +00:00
f002e2fc90 Roll chromium_revision 40bb827aa5..2b49f0a621 (943489:944047)
Change log: 40bb827aa5..2b49f0a621
Full diff: 40bb827aa5..2b49f0a621

Changed dependencies
* src/base: 54b32958be..b53fedf715
* src/build: 09c3e212be..b8e6a02717
* src/buildtools/third_party/libc++abi/trunk: a63bbc1389..e504863f9e
* src/ios: d9a147a20b..ad4a582ec6
* src/testing: 2f79556c57..4f31c40149
* src/third_party: 4732b817fb..a57f1dde5f
* src/third_party/androidx: ZLK16MyL6-u2bhLeLaelNDvoSpFO9qRUiGse469qKRAC..wgGrIASbzI25xEdOD1slUhYwmBa_FcdH498LBPPTwBgC
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/153af62da5..8a112e2bdf
* src/third_party/freetype/src: 9079c5d91a..81912a1385
* src/third_party/perfetto: 514e743cbf..03cb3fd2f9
* src/tools: 299bc80685..ff95136344
DEPS diff: 40bb827aa5..2b49f0a621/DEPS

No update to Clang.

BUG=None

Change-Id: I5069c2be5a9245943c40e7b6f9cdf43593df4b76
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238680
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35398}
2021-11-22 14:31:16 +00:00
520b365163 Fix WebRTC version updater bot.
Bug: webrtc:13373
Change-Id: If6e1cf9d6e62284b3ca496199b3bba5b8248302d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238561
Auto-Submit: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Björn Terelius <terelius@webrtc.org>
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35397}
2021-11-22 08:44:15 +00:00
984cf9b837 Explicitly set encoder and decoder format in codec tests.
This allows to differentiate and test codecs of the same type but
different implementations/settings.

Bug: none
Change-Id: I74f799b36411e63387513133ffc19a7f0c45d550
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238165
Commit-Queue: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35396}
2021-11-22 08:18:25 +00:00
8498b7e7f6 Revert "Add dav1d decoder to WebRTC."
This reverts commit 147858577d4db6d257d3cc248fe571a1bbf887e3.

Reason for revert: High binary size increase

Original change's description:
> Add dav1d decoder to WebRTC.
>
> Bug: none
> Change-Id: I7642f42e592dcf510679f881f118bc4dab93b31c
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237504
> Commit-Queue: Philip Eliasson <philipel@webrtc.org>
> Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
> Reviewed-by: Magnus Flodman <mflodman@webrtc.org>
> Reviewed-by: Sergey Silkin <ssilkin@webrtc.org>
> Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#35394}

TBR=danilchap@webrtc.org,mbonadei@webrtc.org,ilnik@webrtc.org,philipel@webrtc.org,mflodman@webrtc.org,ssilkin@webrtc.org,webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com

Change-Id: I00a8acd6ea94ce523c2d5ba705333c9174678180
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: none
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238560
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Owners-Override: Mirko Bonadei <mbonadei@webrtc.org>
Owners-Override: Artem Titov <titovartem@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35395}
2021-11-19 18:47:42 +00:00
147858577d Add dav1d decoder to WebRTC.
Bug: none
Change-Id: I7642f42e592dcf510679f881f118bc4dab93b31c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237504
Commit-Queue: Philip Eliasson <philipel@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Magnus Flodman <mflodman@webrtc.org>
Reviewed-by: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35394}
2021-11-19 15:03:12 +00:00
ae47cf7dc6 Optimize suppression_filter.
Reducing pointer following. This will allow the compiler to optimize more efficiently with the "-fno-strict-aliasing" flag.

Bug: None
Change-Id: I7cde835161e2d3e85fc7c919556fa9a9e87ef6df
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238169
Reviewed-by: Sam Zackrisson <saza@webrtc.org>
Reviewed-by: Per Åhgren <peah@webrtc.org>
Commit-Queue: Christian Schuldt <cschuldt@google.com>
Cr-Commit-Position: refs/heads/main@{#35393}
2021-11-19 14:17:31 +00:00
ad3551af7a Roll chromium_revision 20e23fa31a..40bb827aa5 (943065:943489)
Change log: 20e23fa31a..40bb827aa5
Full diff: 20e23fa31a..40bb827aa5

Changed dependencies
* src/base: 4f53b8db94..54b32958be
* src/build: 70b834d169..09c3e212be
* src/buildtools/third_party/libc++abi/trunk: 81212d29a2..a63bbc1389
* src/buildtools/third_party/libunwind/trunk: 7f068e554e..038090f742
* src/ios: 519eeb6520..d9a147a20b
* src/testing: 62352386ee..2f79556c57
* src/third_party: 5f7bdd99df..4732b817fb
* src/third_party/androidx: B7B3qXoWKXASrnI5M2ySp9skL0Ozjnw8ViPqwmmTlfEC..ZLK16MyL6-u2bhLeLaelNDvoSpFO9qRUiGse469qKRAC
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/e6e1eb6895..153af62da5
* src/third_party/depot_tools: 1c7dec337d..b6c1ed40d5
* src/third_party/freetype/src: e4f7673e46..9079c5d91a
* src/third_party/googletest/src: 1a3e2a265f..3e0e32ba30
* src/third_party/icu: d3ba6ae0c0..bf66d373ae
* src/third_party/perfetto: 19fb767c32..514e743cbf
* src/tools: 215804b665..299bc80685
DEPS diff: 20e23fa31a..40bb827aa5/DEPS

No update to Clang.

BUG=None

Change-Id: If6f557a958902f9581567fd3ec38d35db09aa765
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238429
Auto-Submit: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35392}
2021-11-19 14:01:43 +00:00
524a422ecd Modify global variables to not call the destructor on exit.
In logging.cc, use the pointer of the static variable so that
it doesn't need a global constructor/exit time destructor.

In RTCFieldTrials.mm, store the field trial string as a char pointer
instead of a std::unique_ptr to ensure that it is never freed.

LSAN will be unhappy with this fix, but WebRTC itself hasn't been
tested with LSAN enabled, and any code changed in this CL does not
build with build_with_chromium=true, so it shouldn't be a problem.

Bug: webrtc:9693, webrtc:11665
Change-Id: Ia28e3534170e0817b815717f6efe862f7b51ef62
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237320
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Byoungchan Lee <daniel.l@hpcnt.com>
Cr-Commit-Position: refs/heads/main@{#35391}
2021-11-19 13:32:21 +00:00
9c4e89d302 Fixed CFBundleVersion version in Python 3.
Bug: None
Change-Id: I69343e26aee74189b192b1de40bfaa5fcac093f3
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238426
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Yura Yaroshevich <yura.yaroshevich@gmail.com>
Cr-Commit-Position: refs/heads/main@{#35390}
2021-11-19 13:01:25 +00:00
68f06af6f6 WebRtcVideoChannelBaseTest.InvalidRecvBufferSize: fix UAF.
The test could cause a UAF as the test exits while the lambda is
still running. Only seems to happen on Linux for some reason.

Bug: webrtc:12854
Change-Id: Ie0c0de09b675ef93dc195a6470752a772083029e
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238425
Auto-Submit: Markus Handell <handellm@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Reviewed-by: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35389}
2021-11-19 12:53:33 +00:00
4be518d1fb Update autoroll script to support py3 (only)
Bug: chromium:1256460
Change-Id: I41e7b0b302733748c1e6b20b7f496e07cc5e3b24
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238424
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35388}
2021-11-19 12:10:40 +00:00
d7eef66a39 VideoStreamEncoder: move PostTasks to WebRtcVideoChannel.
This change moves the responsibility of posting
EncoderSwitchRequestCallback calls closer to the top-level
users which has a better idea about threading requirements.

The change is planned to be followed-up with more changes removing
the need for VSE to post to the worker thread.

Bug: webrtc:13414, chromium:1255737
Change-Id: I57a2962a70e9f245460c59c0d61824371394b952
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238420
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Commit-Queue: Markus Handell <handellm@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35387}
2021-11-19 11:25:11 +00:00
20d0323e57 Dump json output for webrtc tests using gtest_parallel.
This is required step in order to upload tests results to ResultDB.

Bug: b/197492097
Change-Id: Ia25e63c649361c9e09d0710bd582d9511348fc90
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238423
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35386}
2021-11-19 10:49:51 +00:00
dc3a4c47d2 Dump json output for webrtc tests using gtest_parallel.
This is required step in order to upload tests results to ResultDB.

Bug: b/197492097
Change-Id: Ifff898997f6c9a3385659be34e7545217bb08f88
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238000
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35385}
2021-11-19 10:14:32 +00:00
a83f874d03 AGC2 limiter: faster recovery
New limiter tuning to more quickly go back to 0 dB after the limiter
kicks in and the input peak level goes back to normal.

Bug: webrtc:7494
Change-Id: I1050957ca4caf12c4562b899b16c306957dce169
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237701
Reviewed-by: Hanna Silen <silen@webrtc.org>
Commit-Queue: Alessio Bazzica <alessiob@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35384}
2021-11-19 10:00:21 +00:00
1a75d5e67b Remove mentions to xcode_clang which has been removed.
It has been removed upstream by
https://chromium-review.googlesource.com/c/chromium/src/+/3268669.

Bug: None
Change-Id: Ib4dca3322d0a526fea739c7dc3dccd2fc132ec17
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238422
Reviewed-by: Christoffer Jansson <jansson@google.com>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35383}
2021-11-19 09:03:10 +00:00
36cbcf1be8 Remove TBR from version updater script.
No-Presubmit: True
Bug: webrtc:13373
Change-Id: I9c81b670c0456730495e6ec048a9dadfd4fb6f04
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238421
Auto-Submit: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35382}
2021-11-19 07:20:48 +00:00
6d2be9251a Fix mb.py presubmit.
Basically switch the indentation from 2 spaces to 4 spaces.

Bug: webrtc:13413
Change-Id: Iaafdec12715b2e9bfcb0743cf53b960fdf7f371a
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238380
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35381}
2021-11-18 17:20:05 +00:00
9a478b527d VideoStreamEncoder: expect frame entry on the encoder queue.
This change switches the sequence used by the FrameCadenceAdapter
to be the encoder_queue, enabling VideoStreamEncoder::OnFrame to be
invoked directly on the encoder_queue and eliminates the contained
PostTasks.

Bug: chromium:1255737
Change-Id: Ib86fc96ad2be9a38585fef2535855e3f9cc7e57c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238171
Commit-Queue: Markus Handell <handellm@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35380}
2021-11-18 15:35:17 +00:00
7ec990de37 Fix flakky test: ensure that DVQA was executed for some time.
It will ensure that clock moved forward, so clock related metrics as
CPU usage will be also calculated correctly. We should dig into it deeper
to fix the root cause later.

Example failure:
https://ci.chromium.org/ui/p/webrtc/builders/try/win_asan/44610/overview

Bug: b/205824594
Change-Id: If1ebcf6a2b88ba0054479be292cca8f50506e741
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238381
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35379}
2021-11-18 15:05:05 +00:00
50b95525c7 Reintroduce enable_dtls_srtp option
This is a partial revert of commit f9e502d935fa9ca8d9cc7540680021b93d285f28.

Reason for revert: Functionality turns out to be needed by some partners for some months more.

Original change's description:
> Remove enable_dtls_srtp option
>
> This is part of the removal of support for SDES.
>
> Bug: webrtc:11066
> Change-Id: I448d0e0032672c04c87b00550ab4b9d792071a0b
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/234864
> Reviewed-by: Henrik Boström <hbos@webrtc.org>
> Commit-Queue: Harald Alvestrand <hta@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#35262}

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: webrtc:11066, chromium:1271469
Change-Id: I79a90f025e53816789b391bc52a0e896b9be87e1
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238170
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35378}
2021-11-18 14:42:35 +00:00
1dfce5d413 Roll chromium_revision ca9862fcfe..20e23fa31a (942948:943065)
Change log: ca9862fcfe..20e23fa31a
Full diff: ca9862fcfe..20e23fa31a

Changed dependencies
* src/base: 4ed31fedf0..4f53b8db94
* src/build: 35b3ca69be..70b834d169
* src/ios: a11b9c4ac0..519eeb6520
* src/testing: ea0d39ed61..62352386ee
* src/third_party: 5783a5da1a..5f7bdd99df
* src/third_party/android_sdk/public: AuYa11pULKT8AI14_owabJrkZoRGuovL-nvwmiONlYEC..Ez2NWws2SJYCF6qw2O-mSCqK6424l3ZdSTpppLyVR_cC
* src/third_party/androidx: KaJVIPBjh2qbCob6hsVzLk1brSl4iuZVYNpUH0sgg20C..B7B3qXoWKXASrnI5M2ySp9skL0Ozjnw8ViPqwmmTlfEC
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/f61cb71c3c..e6e1eb6895
* src/third_party/depot_tools: 176a9e8764..1c7dec337d
* src/third_party/perfetto: f32b6ac458..19fb767c32
* src/tools: eedc9a77f2..215804b665
DEPS diff: ca9862fcfe..20e23fa31a/DEPS

No update to Clang.

BUG=None

Change-Id: I808517b0f6384efc9585383ba7c2283d26e1e859
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238363
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35377}
2021-11-18 14:38:35 +00:00
cabc3e50dd Delete obsolete method QueryVideoEncoder
Bug: webrtc:12875
Change-Id: Icc2f3ceb9814292755b9c382186e27f3131b64a2
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237280
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35376}
2021-11-18 11:17:35 +00:00
0a7a8e0aba Remove DEPRECATED_AsyncInvoker::Flush.
The method is not used so can be safely deleted before the full
(and eventual) removal of the implementation.

Bug: webrtc:12339
Change-Id: I7726313c46562041f670c3baec2db955de0b4298
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238141
Auto-Submit: Tommi <tommi@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35375}
2021-11-18 09:08:55 +00:00
6002b15cd1 Optimize ComputeFrequencyResponse().
Reducing pointer following. This will allow the compiler to optimize more efficiently with the "-fno-strict-aliasing" flag.

Bug: None
Change-Id: Ib1fd3a1cf3f89471b0ec87404650a6061eec5e2d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237782
Reviewed-by: Sam Zackrisson <saza@webrtc.org>
Commit-Queue: Christian Schuldt <cschuldt@google.com>
Cr-Commit-Position: refs/heads/main@{#35374}
2021-11-18 08:49:25 +00:00
7500552a8f Fix WebRTC version updater bot.
No-Presubmit: True
Bug: webrtc:13373
Change-Id: If668e01f426d19d8d90328cb358a131425b23047
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238168
Auto-Submit: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35373}
2021-11-18 08:45:36 +00:00
7b0c6a3ae5 Roll chromium_revision ed9be3a428..ca9862fcfe (942828:942948)
Change log: ed9be3a428..ca9862fcfe
Full diff: ed9be3a428..ca9862fcfe

Changed dependencies
* src/build: 91e430766b..35b3ca69be
* src/buildtools/third_party/libunwind/trunk: 51ffc5ed5c..7f068e554e
* src/ios: 31651ec3f0..a11b9c4ac0
* src/testing: c204cbf41a..ea0d39ed61
* src/third_party: 8ee22ff845..5783a5da1a
* src/third_party/androidx: wnPOPNFDdq5pVsm7Lj0o7NWefxqLo5x-WUBlmrrO9_0C..KaJVIPBjh2qbCob6hsVzLk1brSl4iuZVYNpUH0sgg20C
* src/third_party/depot_tools: 64cb3ee4cd..176a9e8764
* src/third_party/googletest/src: 1b18723e87..1a3e2a265f
* src/tools: fdbb4310c2..eedc9a77f2
DEPS diff: ed9be3a428..ca9862fcfe/DEPS

No update to Clang.

BUG=None

Change-Id: I8f96020fb88a319a47ce9eb750574455958f7ffe
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238321
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35372}
2021-11-18 04:44:50 +00:00
be1a170435 Roll chromium_revision 35d97f39f3..ed9be3a428 (942706:942828)
Change log: 35d97f39f3..ed9be3a428
Full diff: 35d97f39f3..ed9be3a428

Changed dependencies
* src/base: 6a20a89ab6..4ed31fedf0
* src/build: b9feec1b66..91e430766b
* src/buildtools: b138e6ce86..075dd7e228
* src/ios: 08b95d6c3c..31651ec3f0
* src/testing: ba85cc96b4..c204cbf41a
* src/third_party: 97df40dd1f..8ee22ff845
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/ccdf6bde5f..f61cb71c3c
* src/third_party/perfetto: 406e7c3674..f32b6ac458
* src/tools: 2f46b72b92..fdbb4310c2
DEPS diff: 35d97f39f3..ed9be3a428/DEPS

No update to Clang.

BUG=None

Change-Id: I962fab411f8c500ba7553813cd0a10d54241b992
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238260
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35371}
2021-11-17 22:56:44 +00:00
9e677f0072 Roll chromium_revision f8776f0d78..35d97f39f3 (942535:942706)
Change log: f8776f0d78..35d97f39f3
Full diff: f8776f0d78..35d97f39f3

Changed dependencies
* src/base: fad8a0b16e..6a20a89ab6
* src/build: 829065270e..b9feec1b66
* src/buildtools/third_party/libc++abi/trunk: 707d75f53e..81212d29a2
* src/ios: d5ffd86948..08b95d6c3c
* src/testing: 6713c8c181..ba85cc96b4
* src/third_party: 9e360d6249..97df40dd1f
* src/third_party/androidx: GpsjLtmFq0wH_145hryWddxYQ-dHC-2cy5x8AoOf7pIC..wnPOPNFDdq5pVsm7Lj0o7NWefxqLo5x-WUBlmrrO9_0C
* src/third_party/harfbuzz-ng/src: 9bd7ba5019..64b29dbd59
* src/third_party/perfetto: f890ec27af..406e7c3674
* src/third_party/r8: 9q0e-VLY3WNUFn48GeKkiokWeK656oswOvlPUuQbSM8C..nqWomZTwNDoogX26WeCSoFGg6aQN1FrwzoU4hCS0duEC
* src/tools: 6e1149d9e5..2f46b72b92
DEPS diff: f8776f0d78..35d97f39f3/DEPS

No update to Clang.

BUG=None

Change-Id: I308a32c1bb33b97dbfe9a68f1150c3106a7c2eb0
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238220
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Commit-Queue: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#35370}
2021-11-17 18:53:24 +00:00
4de99443dd Delete memory allocated by GetStreamCaps
Bug: webrtc:13260
Change-Id: I18c23e2c3aad7c711c33c8cc381d46275473b4a0
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237344
Reviewed-by: Per Kjellander <perkj@webrtc.org>
Commit-Queue: Sergey Silkin <ssilkin@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35369}
2021-11-17 15:06:31 +00:00
9ebe6d7c88 Remove the AsyncInvoker alias.
This emphasizes the "hint" to potential external users that the
class has been deprecated.

Bug: webrtc:12339
Change-Id: Iab83481af69a505059297cce959f02b5ab649f2f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237805
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35368}
2021-11-17 13:37:06 +00:00
9456501f19 Try to avoid to set Code-Review+1
No-Presubmit: True
Bug: chromium:1260482
Change-Id: I9db4b1291217031ff5107668318edfef7c33ddfb
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238163
Auto-Submit: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35367}
2021-11-17 11:58:46 +00:00
96ef2bd293 Roll chromium_revision f1aab63b8b..f8776f0d78 (942294:942535)
Change log: f1aab63b8b..f8776f0d78
Full diff: f1aab63b8b..f8776f0d78

Changed dependencies
* src/base: 783320b7cf..fad8a0b16e
* src/build: 220a7fc489..829065270e
* src/buildtools/linux64: git_revision:185124551408e7a5349c2aa31051b5a629dc3a5e..git_revision:4aa9bdfa05b688c58d3d7d3e496f3f18cbb3d89e
* src/buildtools/mac: git_revision:185124551408e7a5349c2aa31051b5a629dc3a5e..git_revision:4aa9bdfa05b688c58d3d7d3e496f3f18cbb3d89e
* src/buildtools/win: git_revision:185124551408e7a5349c2aa31051b5a629dc3a5e..git_revision:4aa9bdfa05b688c58d3d7d3e496f3f18cbb3d89e
* src/ios: 39acfe80c3..d5ffd86948
* src/testing: 4eee891603..6713c8c181
* src/third_party: fc7888c598..9e360d6249
* src/third_party/androidx: eenVsfeHLKnm1MmzR1_7Zz7iUfRKwHG4j5jTupRbzrcC..GpsjLtmFq0wH_145hryWddxYQ-dHC-2cy5x8AoOf7pIC
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/1a5ffbc5f1..ccdf6bde5f
* src/third_party/depot_tools: 067f0e55d3..64cb3ee4cd
* src/third_party/freetype/src: d31bafcb9c..e4f7673e46
* src/third_party/icu: 3e05d9daa9..d3ba6ae0c0
* src/third_party/libvpx/source/libvpx: e259e6951d..ec80f88c5d
* src/third_party/perfetto: 4d59d2eea6..f890ec27af
* src/tools: 149b7515cb..6e1149d9e5
DEPS diff: f1aab63b8b..f8776f0d78/DEPS

No update to Clang.

TBR=marpan@webrtc.org, jianj@chromium.org,
BUG=None

Change-Id: I402163c72be42c1d7c00c58f01ca545c222092e5
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238103
Bot-Commit: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Reviewed-by: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35366}
2021-11-17 11:44:55 +00:00
be9c40f0b4 Fix documentation for VideoQualityAnalyzerInterface::GetStreamLabel
Bug: b/205824594
Change-Id: I76eff28984446ed94d701129d63f2a1643f9d983
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238161
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35365}
2021-11-17 11:38:33 +00:00
9c14573d4e Add method to return frames history from DVQA.
Bug: b/205824594
Change-Id: Ie388aa4b61538cf97e714b1c5d28b01c72739ebd
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238162
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35364}
2021-11-17 11:12:05 +00:00
5a9be308e3 Remove self TBR to avoid setting Code-Review+1.
No-Presubmit: True
Bug: chromium:1260482
Change-Id: I988b4056ff2425f8ec764ab816a7966368b5c65a
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238160
Reviewed-by: Christoffer Jansson <jansson@google.com>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35363}
2021-11-17 10:26:46 +00:00
2ae54b921b Make Chromium DEPS autoroller set Bot-Commit+1
Bug: chromium:1260482
Change-Id: I2bc1a2ccab3d07588d3d965f3c24e60ae0dba41b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238140
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Auto-Submit: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35362}
2021-11-17 07:24:34 +00:00