825 Commits

Author SHA1 Message Date
Jeremy Leconte
d15f3e1220 Prepare the code to run ios tests with the standalone recipe.
The flags isolated-script-test-output and isolated-script-test-perf-output need to be consumed by the tests.

The generated .app folder in added in the data list of the gni file.
This will make it available in the runtime_deps file and thus will be populated to the swarming tasks.

Bug: webrtc:13556
Change-Id: I2c75774b847d9f686c3abc00ba0400bbc3fcefae
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/240520
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#36029}
2022-02-18 12:57:42 +00:00
Christoffer Jansson
1b083a998b Encode data for compression + add initial tests
Bug: webrtc:13607
Change-Id: I3bbec5558e676ca45125fad3fdbd10cc47c84601
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251390
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#36003}
2022-02-15 14:24:11 +00:00
Jeremy Leconte
63472e5aea Replace dump_json_test_results flag by isolated-script-test-output.
This is because it's the default flag used in the recipes to dump a json output.
This CL also fixes some python3 lint issues in mb.py.

Bug: webrtc:13594
Change-Id: I9275b5da0963f801d3191703c2eb72d90befb5d7
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/248142
Reviewed-by: Christoffer Jansson <jansson@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35986}
2022-02-14 08:55:01 +00:00
Christoffer Jansson
e6aa6a8740 Set file mode in the argparse argument
Bug: webrtc:13607
Change-Id: I7943761933e0e110ff16b65284d8b36644b5c4ea
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251381
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35970}
2022-02-10 09:02:26 +00:00
Harald Alvestrand
981c572eab Updated apply-iwyu to autogenerate compile_commands.json
Also deleted iwyu script that was not maintained, and deleted
some options that made the script more complex.

Bug: none
Change-Id: I39d8eaa37f12c72ddc127ae145e6a3a80f328316
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251384
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35966}
2022-02-09 14:30:57 +00:00
Christoffer Jansson
c98fb2ca0f Use binary mode for proto ingestion
Bug: webrtc:13607
Change-Id: Id0385f74215360ff604641a50ce9f599c87abb5e
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251327
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35960}
2022-02-09 07:19:35 +00:00
Christoffer Jansson
409ac89b43 decode strings to utf-8 and use universal_newlines for subprocess
Bug: webrtc:13607
Change-Id: I4d8df7488b819b477cd0ae5553c0eb8a7adfb179
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251343
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35955}
2022-02-08 17:42:25 +00:00
Christoffer Jansson
4e8a773b4b tools_webrtc dir converted to py3 + top level PRESUBMIT script
Bug: webrtc:13607
Change-Id: Ib018e43ea977cc24dd71048e68e3343741f7f31b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/249083
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Jeremy Leconte <jleconte@google.com>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35953}
2022-02-08 14:42:26 +00:00
Jeremy Leconte
cbfa235b35 Add the possibility to output a json gtest output to the perf tests.
We use the Chromium existing flag isolated_script_test_output that we translate into gtest_output.
This is because the Chromium flag has the same purpose as gtest_output and is already provided in the recipe modules.

No-Presubmit: True
Bug: b/197492097
Change-Id: Ia432a85b0e0ab32008b39ffe751d11aefb9b24ea
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251041
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35937}
2022-02-07 20:59:33 +00:00
Harald Alvestrand
0d4af12cab Add back --no_fwd_decls to iwyu run
Applies IWYU to pc/peer_connection.cc as test.

Bug: webrtc:13532
Change-Id: Iec453fdf6525c6376cb73dd1199326a8e1a21ff6
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/249986
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35845}
2022-01-31 10:26:17 +00:00
Christoffer Jansson
a4290a7909 add m1 try bots
Any Mac builder should be able to compile ARM64 builds when using target_cpu=arm64

Bug: chromium:1238267
Change-Id: I72dac3b6f170f09d5c158ec11650e0cff7b9e638
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/249790
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35838}
2022-01-31 08:28:29 +00:00
Jeremy Leconte
e006b7d103 Fix print error when migrating to python3.
This is causing errors on the ci:
https://ci.chromium.org/p/webrtc/builders/ci/Win32%20Release%20%28Clang%29/24047

No-Presubmit: True
Bug: webrtc:13607
Change-Id: I05c85d5d67ab71ef971899a60daffa2c10b72305
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/249783
Reviewed-by: Christoffer Jansson <jansson@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35823}
2022-01-28 10:45:55 +00:00
Mirko Bonadei
5d9ae8635c Switch Swarming commands to py3.
Thanks to machenbach@ for the heads up.

WebRTC's MB was generating a Swarming command that was using vpython
(defaulting on python2 on some platforms). This CL switches that to
vpython3 (fixing gtest-parallel-wrapper.py to be python3 compliant).

No-Presubmit: True
Bug: webrtc:13607
Change-Id: Icfa7d23b81e30cebfe8243d4ba65284955593465
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/249604
Reviewed-by: Christoffer Jansson <jansson@google.com>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35821}
2022-01-28 08:17:05 +00:00
Byoungchan Lee
e286a7bbf7 Modify apply-iwyu to utilize compile_commands.json
Bug: webrtc:13532
Change-Id: I06e5987469e732296deb1eefd317e2478add98b8
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/246040
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: (Daniel.L) Byoungchan Lee <daniel.l@hpcnt.com>
Cr-Commit-Position: refs/heads/main@{#35674}
2022-01-12 23:08:22 +00:00
Jeremy Leconte
994bf454ec Revert of flag simplification.
In order to unify WebRTC recipes with Chromium recipes this CL tries to revert the old CL https://webrtc-review.googlesource.com/c/src/+/171681.
This CL was already partially reverted (https://webrtc-review.googlesource.com/c/src/+/171809).
In upcoming CLs, the added flag dump_json_test_results will be removed in order to use isolated-script-test-output instead.

Bug: webrtc:13556
Change-Id: I3144498b9a5cbaa56c23b3b8adbac2229ad63c37
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/245602
Reviewed-by: Andrey Logvin <landrey@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35666}
2022-01-12 10:53:12 +00:00
Christoffer Jansson
cf00059db6 Remove Win8
Bug: chromium:1281336
Change-Id: I490ec704d8f006301ab41dc3380983075032285e
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/245483
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35656}
2022-01-11 10:27:54 +00:00
Jeremy Leconte
67b0a4476b Remove perf test isac_fix_test.
Bug: b/210641670
Change-Id: If562d1b2aa3486a51efffb4dec471ed7064c5261
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/241360
Reviewed-by: Christoffer Jansson <jansson@webrtc.org>
Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35533}
2021-12-14 16:52:58 +00:00
Mirko Bonadei
d80a21d2f1 Relax gn check test.
See the report at https://bugs.chromium.org/p/gn/issues/detail?id=271.

Bug: None
Change-Id: I6ddc12fc83f8f8eca985af9237e0b736814574fb
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/240522
Reviewed-by: Christoffer Jansson <jansson@google.com>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35516}
2021-12-10 09:33:01 +00:00
Jeremy Leconte
f22c78b01a Fix mb.py presubmit issues.
* Add a config file for python formatting (.style.yapf).
* Change the default indentation from 4 spaces to 2 spaces.
* Run 'git cl format --python' on a few python files.

Bug: webrtc:13413
Change-Id: Ia71135131276c2c499b00032d57ad16ee5200a5c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238982
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Reviewed-by: Henrik Andreassson <henrika@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35500}
2021-12-08 08:53:00 +00:00
Jeremy Leconte
a2e3d80cf6 Revert "Reland "Use gtest_parallel with 1 worker for webrtc_perf_tests.""
This reverts commit c31fc2a941d417286e1a56d6f18d5051c99d06d9.

Reason for revert: Fix is not working properly.

Original change's description:
> Reland "Use gtest_parallel with 1 worker for webrtc_perf_tests."
>
> This is a reland of 258ed1a38ad9d4f0da798c40b6976eff2dce864f
>
> Original change's description:
> > Use gtest_parallel with 1 worker for webrtc_perf_tests.
> >
> > This will enable test results to be uploaded to ResultDB.
> >
> > Bug: b/197492097
> > Change-Id: Iec28520c4cd8f35fcff2cbd105a4b851ef41b9fc
> > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239641
> > 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@{#35458}
>
> Bug: b/197492097
> No-Presubmit: True
> Change-Id: Iea90f5698c83791d39c0f6da666c1d1eb274edd3
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239645
> Commit-Queue: Jeremy Leconte <jleconte@google.com>
> Reviewed-by: Andrey Logvin <landrey@webrtc.org>
> Reviewed-by: Christoffer Jansson <jansson@webrtc.org>
> Reviewed-by: Jakob Ivarsson <jakobi@webrtc.org>
> Reviewed-by: Gustaf Ullberg <gustaf@webrtc.org>
> Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#35483}

TBR=mbonadei@webrtc.org,gustaf@webrtc.org,jansson@google.com,jansson@webrtc.org,jakobi@webrtc.org,landrey@webrtc.org,jleconte@google.com,jleconte@webrtc.org,webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com

Change-Id: Iee9b67db99545a1e6c707bc03faaf55afc90cbbf
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: b/197492097
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/240182
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Owners-Override: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35486}
2021-12-07 12:04:08 +00:00
Jeremy Leconte
c31fc2a941 Reland "Use gtest_parallel with 1 worker for webrtc_perf_tests."
This is a reland of 258ed1a38ad9d4f0da798c40b6976eff2dce864f

Original change's description:
> Use gtest_parallel with 1 worker for webrtc_perf_tests.
>
> This will enable test results to be uploaded to ResultDB.
>
> Bug: b/197492097
> Change-Id: Iec28520c4cd8f35fcff2cbd105a4b851ef41b9fc
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239641
> 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@{#35458}

Bug: b/197492097
No-Presubmit: True
Change-Id: Iea90f5698c83791d39c0f6da666c1d1eb274edd3
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239645
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Reviewed-by: Andrey Logvin <landrey@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@webrtc.org>
Reviewed-by: Jakob Ivarsson <jakobi@webrtc.org>
Reviewed-by: Gustaf Ullberg <gustaf@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35483}
2021-12-07 10:08:16 +00:00
Christoffer Jansson
833b095694 add more owners to mb
Bug: None
Change-Id: I24616c391b61264e4509f20ffd6d1accd7f3bab9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239843
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35480}
2021-12-06 16:46:16 +00:00
Nidhi Jaju
2394977f74 CodeHealth: Python3 migration for tools_webrtc/mb/
This CL includes:
* set Pylint to 2.7 in tools_webrtc/mb/PRESUBMIT.py
* fix some style warnings caused by scripts in tools_webrtc/mb/
* pass skip_shebang_check=True within tools_webrtc/mb/PRESUBMIT.py

Bug: chromium:1262287, chromium:1262352
Change-Id: Iae4f111942d724db6a7f4585c97a7c26c1e6ccc0
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239660
Commit-Queue: Nidhi Jaju <nidhijaju@chromium.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35477}
2021-12-06 12:59:57 +00:00
Jeremy Leconte
a642df2598 Revert "Use gtest_parallel with 1 worker for webrtc_perf_tests."
This reverts commit 258ed1a38ad9d4f0da798c40b6976eff2dce864f.

Reason for revert: This cl breaks the dumping of the perf proto histogram.

Original change's description:
> Use gtest_parallel with 1 worker for webrtc_perf_tests.
>
> This will enable test results to be uploaded to ResultDB.
>
> Bug: b/197492097
> Change-Id: Iec28520c4cd8f35fcff2cbd105a4b851ef41b9fc
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239641
> 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@{#35458}

TBR=mbonadei@webrtc.org,jansson@google.com,webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com,jleconte@webrtc.org

Change-Id: Ic4ab03d0e7f8bc1ce799d30e74420200d2f8f224
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: b/197492097
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239644
Reviewed-by: Jeremy Leconte <jleconte@google.com>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35463}
2021-12-02 13:16:00 +00:00
Jeremy Leconte
258ed1a38a Use gtest_parallel with 1 worker for webrtc_perf_tests.
This will enable test results to be uploaded to ResultDB.

Bug: b/197492097
Change-Id: Iec28520c4cd8f35fcff2cbd105a4b851ef41b9fc
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239641
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@{#35458}
2021-12-02 10:40:46 +00:00
Jeremy Leconte
59b8a46d3f Fix low_bandwidth_audio_test on Windows bots.
Bug: b/197492097
Change-Id: I29ae4d699c3c928f9d22da8cf9026c462f3afa7e
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239540
Reviewed-by: Christoffer Jansson <jansson@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#35452}
2021-12-01 13:04:52 +00:00
Junji Watanabe
8d89f461c9 Remove --isolated from mb.py
This option is not supported now.

Bug: chromium:1271841
Change-Id: I27dd1531589ac497ef345fff41897a087ace4362
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239461
Reviewed-by: Takuto Ikuta <tikuta@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Auto-Submit: Junji Watanabe <jwata@google.com>
Cr-Commit-Position: refs/heads/main@{#35449}
2021-12-01 06:15:41 +00:00
Mirko Bonadei
b55c299f4d Avoid TBR in DEPS autoroller (use CC instead).
No-Presubmit: True
No-Try: True
Bug: None
Change-Id: I71c71de4ff9cfb1b9e59620b56a13ff4e94639eb
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239361
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Andrey Logvin <landrey@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35430}
2021-11-29 12:54:01 +00:00
Takuto Ikuta
9345bee860 DEPS: use python3
Bug: chromium:1208028
Change-Id: I1117855556e4a9e7cbaf54764c8a23c5b8581ded
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/239160
Auto-Submit: Takuto Ikuta <tikuta@google.com>
Reviewed-by: Jeremy Leconte <jleconte@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35426}
2021-11-29 07:27:52 +00:00
Mirko Bonadei
4e8e36c745 Try to fix the version updated bot.
No-Presubmit: True
Bug: webrtc:13373
Change-Id: I30418f760cda6be31c75ff7e59311cb59c1874c9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238805
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35415}
2021-11-24 13:04:06 +00:00
Jeremy Leconte
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
Yura Yaroshevich
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
Christoffer Jansson
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
Jeremy Leconte
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
Jeremy Leconte
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
Mirko Bonadei
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
Jeremy Leconte
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
Mirko Bonadei
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
Mirko Bonadei
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
Mirko Bonadei
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
Mirko Bonadei
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
Florent Castelli
607513e598 Add more usrsctp TSAN suppressions
Also remove some older suppressions that are not needed anymore.

Bug: webrtc:9850
Change-Id: I5cb6822ba0880671efbdb89d439cffd5ff71883f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238001
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Florent Castelli <orphis@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35360}
2021-11-16 17:05:54 +00:00
Evan Shrubsole
45448e9c1c Remove slow_tests
repeating_task_unittests no longer relies on sleeps, so slow_tests
is no longer needed.

Bug: webrtc:12808
Change-Id: Ie9ad23e065074638fe3770e51fb7be83344160e7
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237804
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Evan Shrubsole <eshr@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35351}
2021-11-16 11:11:20 +00:00
Mirko Bonadei
f5ea3b96cd Roll chromium_revision 867b87eb29..b56d8868f3 (933713:941610)
Change log: 867b87eb29..b56d8868f3
Full diff: 867b87eb29..b56d8868f3

Changed dependencies
* src/base: 14c89123e6..b367343820
* src/build: a0368daa25..d134d68f00
* src/buildtools: aa2fb0187c..b138e6ce86
* src/buildtools/linux64: git_revision:693f9fb87e4febdd4299db9f73d8d2c958e63148..git_revision:90294ccdcf9334ed25a76ac9b67689468e506342
* src/buildtools/mac: git_revision:693f9fb87e4febdd4299db9f73d8d2c958e63148..git_revision:90294ccdcf9334ed25a76ac9b67689468e506342
* src/buildtools/third_party/libc++abi/trunk: 4c834abe6f..707d75f53e
* src/buildtools/third_party/libunwind/trunk: 10f4e4a5b5..51ffc5ed5c
* src/buildtools/win: git_revision:693f9fb87e4febdd4299db9f73d8d2c958e63148..git_revision:90294ccdcf9334ed25a76ac9b67689468e506342
* src/ios: eb1001b206..af72ede971
* src/testing: 650a2f9467..619bca0ef9
* src/third_party: bc52b47596..5f94f37f8c
* src/third_party/android_build_tools/aapt2: oJ_fhfmT6sLorimH6Eo6hwWS2R2gU0c4ZUQfXvzae1UC..GlCdqxHxlg-8YkDGgj5cie-6COsEAZga9jyq-LAYxY4C
* src/third_party/android_build_tools/bundletool: OZ4is33usuZy-deP3FpWCtbKXaIY0qBLsElWte7556YC..2ZcLVDxyRwp8FzpeYLtLT0TfSRweZxvwh1-Kx1jZ_FoC
* src/third_party/android_deps/libs/com_google_android_material_material: version:2@1.5.0-alpha02.cr0..version:2@1.5.0-alpha05.cr0
* src/third_party/android_deps/libs/com_google_errorprone_error_prone_annotation: version:2@2.9.0.cr0..version:2@2.10.0.cr0
* src/third_party/android_deps/libs/com_google_errorprone_error_prone_annotations: version:2@2.9.0.cr0..version:2@2.10.0.cr0
* src/third_party/android_deps/libs/com_google_errorprone_error_prone_check_api: version:2@2.9.0.cr0..version:2@2.10.0.cr0
* src/third_party/android_deps/libs/com_google_errorprone_error_prone_core: version:2@2.9.0.cr0..version:2@2.10.0.cr0
* src/third_party/android_deps/libs/com_google_errorprone_error_prone_type_annotations: version:2@2.9.0.cr0..version:2@2.10.0.cr0
* src/third_party/androidx: Dewv4glpYqPtnvaOzHgVF6074yDsYzzDFiv0dO1ijwgC..nCX115noKlFOaRMhmFKd1v01hnVpUAaKg2SyLFZaYZgC
* src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/45c8be91f3..69030a0cea
* src/third_party/breakpad/breakpad: ff5892c5da..ee2ad61263
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/2d036344bd..a7ad5b5c96
* src/third_party/depot_tools: 5cffc195c9..9fcfde2499
* src/third_party/freetype/src: fde91ab8f1..d31bafcb9c
* src/third_party/googletest/src: 16f637fbf4..1b18723e87
* src/third_party/icu: eedbaf76e4..3e05d9daa9
* src/third_party/libvpx/source/libvpx: 7aabd69682..e259e6951d
* src/third_party/perfetto: 844b8662e9..5ed467a290
* src/third_party/r8: EU82Aqeu2B1PZzAmebHXfyOgeL2UZifcIJ1LPyoThw0C..nqWomZTwNDoogX26WeCSoFGg6aQN1FrwzoU4hCS0duEC
* src/third_party/turbine: 6QYCyqU5yXkAT7T-_K3sCPvfISI8ACXF_cW4OM5v9BgC..KbLQUR_KFiUEzVt1lMTORY96bz_PbMwC3GwIb1oGJmAC
* src/tools: f4333a2a39..a182115705
* src/tools/luci-go: git_revision:d1c03082ecda0148d8096f1fd8bf5491eafc7323..git_revision:0e76392e6557cc3ff8d95c3bc012540e0dbc128a
* src/tools/luci-go: git_revision:d1c03082ecda0148d8096f1fd8bf5491eafc7323..git_revision:0e76392e6557cc3ff8d95c3bc012540e0dbc128a
Added dependency
* src/third_party/android_deps/libs/com_google_flatbuffers_flatbuffers_java
DEPS diff: 867b87eb29..b56d8868f3/DEPS

Clang version changed llvmorg-14-init-6722-g0fbd3aad:llvmorg-14-init-8564-g34b903d8
Details: 867b87eb29..b56d8868f3/tools/clang/scripts/update.py

TBR=chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com,marpan@webrtc.org, jianj@chromium.org,
BUG=None

No-Presubmit: True
Change-Id: I721d681c6c0a687c782fa1f34ea6cea3fba6ce02
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/237761
Reviewed-by: Xavier Lepaul‎ <xalep@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Autoroller <chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com>
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35347}
2021-11-15 17:34:08 +00:00
Mirko Bonadei
7b95fec6c9 Disable DCHECKs on sanitizer builds.
Sanitizer builds are already slower than release builds, so removing
DCHECKs might allow for more coverage (less tests skipped because
of timing issues).

Bug: webrtc:13329
Change-Id: I5433f0e520b3ad3e463dea019f3b524a6034f1ed
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/236583
Reviewed-by: Tommi <tommi@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35285}
2021-10-31 13:17:38 +00:00
Sylvain Defresne
2110f7d664 [ios] Only build ios_remoting_unittests
The ios bots are only interested in running ios_remoting_unittests,
so in order to avoid breaking when //ios/{web,chrome} requires new
version of Xcode, set `ios_build_chrome=false` to stop building
those targets (as they are not run/tested since they don't depend
on WebRTC).

Bug: webrtc:13222
Change-Id: Ib08044157d7ee9ea44a3c608310609cad99665b1
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/233620
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/main@{#35256}
2021-10-21 08:59:55 +00:00
Jerome Jiang
e88d06e9b7 Add marpan and jianj to av1 email
Bug: None
Change-Id: Ia58bfa5fd85c0202d72f6275f6bc6afef7a4cd22
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/235841
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35255}
2021-10-21 06:48:11 +00:00
Mirko Bonadei
d91edc2244 Remove MSVC bots configs from MB.
Bug: webrtc:13232
Change-Id: If348dcf644cefb9b804dc4922f809c9d5135b38c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/234084
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35223}
2021-10-18 07:38:27 +00:00
Christoffer Jansson
294f5bcaa8 testing code-owners plugin.
Bug: NONE
Change-Id: I935b413f1707de11593ff02df294afeefb64f60d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/234585
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#35173}
2021-10-11 07:39:05 +00:00
Björn Terelius
88ae708d21 Roll chromium_revision e2c8b6114c..2895dbef35 (925965:929709)
Change log: e2c8b6114c..2895dbef35
Full diff: e2c8b6114c..2895dbef35

Changed dependencies
* src/base: f03432908f..74160bef85
* src/build: b2dbce6eed..6eb2bf24e4
* src/buildtools/third_party/libc++abi/trunk: 50e90b8650..9eb0245224
* src/buildtools/third_party/libunwind/trunk: a002c725cf..557b51a0cc
* src/ios: 3c447291ef..135b00add1
* src/testing: 65fe96e361..99cbff10f8
* src/third_party: 11fcbd5f43..7ecb34e56f
* src/third_party/android_build_tools/bundletool: nK0LSzIJHh_OqyIEMh3zye_Ad54jw57inqQF5jFJzdYC..OZ4is33usuZy-deP3FpWCtbKXaIY0qBLsElWte7556YC
* src/third_party/android_deps/libs/com_google_guava_guava: version:2@30.1.1-jre.cr0..version:2@31.0-jre.cr0
* src/third_party/android_deps/libs/com_google_guava_guava_android: version:2@30.1-android.cr0..version:2@31.0-android.cr0
* src/third_party/android_deps/libs/org_checkerframework_checker_qual: version:2@3.8.0.cr0..version:2@3.12.0.cr0
* src/third_party/android_deps/libs/org_jetbrains_kotlin_kotlin_stdlib: version:2@1.5.30.cr0..version:2@1.5.31.cr0
* src/third_party/android_deps/libs/org_jetbrains_kotlin_kotlin_stdlib_common: version:2@1.5.30.cr0..version:2@1.5.31.cr0
* src/third_party/androidx: 5ogjCeEw_SblgjIEGQ90Mr2ImySIcVxVCx6ve5lMk5YC..Dewv4glpYqPtnvaOzHgVF6074yDsYzzDFiv0dO1ijwgC
* src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/27a3328a37..87f316d774
* src/third_party/breakpad/breakpad: 1147c2fcf0..ff5892c5da
* src/third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/2146b04faf..ee3f2f49c1
* src/third_party/depot_tools: d2fe58c115..7cdf1424b9
* src/third_party/freetype/src: 12ef831fc3..946df22165
* src/third_party/googletest/src: e4717df71a..075810f7a2
* src/third_party/harfbuzz-ng/src: a52c6df38a..d58f1685ce
* src/third_party/icu: 3f443830bd..eedbaf76e4
* src/third_party/libaom/source/libaom: https://aomedia.googlesource.com/aom.git/+log/da0b537ee1..d50b2b81ca
* src/third_party/libvpx/source/libvpx: 65a1751e5b..7aabd69682
* src/third_party/libyuv: b92a60320f..d13d9d5972
* src/third_party/nasm: 4e6fe9d154..335fe83e13
* src/third_party/perfetto: 76a8111c0c..51d6baaff7
* src/third_party/turbine: OxN4BOHObt6tbi4fpsSFeUAxSiaf2Cto-QlpFzGzC_IC..EbT5V0csSPtrqE7MUYHXHhKwiBh8znOrUP46FVGfb9gC
* src/tools: 7e14c9353a..03ea394946
* src/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:413d434bd4eee1130614494dfb19f1eba03d71af
* src/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:413d434bd4eee1130614494dfb19f1eba03d71af
* src/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:413d434bd4eee1130614494dfb19f1eba03d71af
DEPS diff: e2c8b6114c..2895dbef35/DEPS

Clang version changed llvmorg-14-init-4918-ge787678c:llvmorg-14-init-5759-g02895eed
Details: e2c8b6114c..2895dbef35/tools/clang/scripts/update.py

Additionally,
- manually change instrumented libraries from Trusty to Xenial,
- add bazel to WEBRTC_ONLY_DEPS
- change the Info.plist file,
- manually roll libaom (https://aomedia.googlesource.com/aom.git/+log/d50b2b81ca06..552a0398f47d)

TBR=terelius@webrtc.org, landrey@webrtc.org, marpan@webrtc.org, jianj@chromium.org,
BUG=b/201970606

Change-Id: Ie048e6baf7c1d1f5d2557a36d216ed9cb0401925
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/234588
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Reviewed-by: Andrey Logvin <landrey@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Andrey Logvin <landrey@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35171}
2021-10-09 16:54:09 +00:00