Commit Graph

322 Commits

Author SHA1 Message Date
165148d84d Reland "Remove deprecated barcode scanning functionality"
This is a reland of ff292f30d9a4b7a56aea872fe488d342f47202a3

I'm leaving empty .py files in place in order to not break downstream client builds.

Original change's description:
> Remove deprecated barcode scanning functionality
>
> This code is not used anymore, but it's not possible to land this CL
> until issue webrtc:9665 is fixed.
>
> Bug: webrtc:9642,webrtc:9665
> Change-Id: Idb68e9bdf51b4239788cd6869dcb44dae87d7c56
> Reviewed-on: https://webrtc-review.googlesource.com/c/95951
> Reviewed-by: Paulina Hensman <phensman@webrtc.org>
> Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
> Commit-Queue: Magnus Jedvert <magjed@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#25289}

TBR=phensman@webrtc.org,phoglund@webrtc.org

Bug: webrtc:9642, webrtc:9665
Change-Id: I248f8656b14c89b0b92e777f4408ee6a6dad41f9
Reviewed-on: https://webrtc-review.googlesource.com/c/107360
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Magnus Jedvert <magjed@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25296}
2018-10-22 21:30:58 +00:00
38a34198a3 Revert "Remove deprecated barcode scanning functionality"
This reverts commit ff292f30d9a4b7a56aea872fe488d342f47202a3.

Reason for revert: issues with downstream projects

Original change's description:
> Remove deprecated barcode scanning functionality
> 
> This code is not used anymore, but it's not possible to land this CL
> until issue webrtc:9665 is fixed.
> 
> Bug: webrtc:9642,webrtc:9665
> Change-Id: Idb68e9bdf51b4239788cd6869dcb44dae87d7c56
> Reviewed-on: https://webrtc-review.googlesource.com/c/95951
> Reviewed-by: Paulina Hensman <phensman@webrtc.org>
> Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
> Commit-Queue: Magnus Jedvert <magjed@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#25289}

TBR=phoglund@webrtc.org,mbonadei@webrtc.org,magjed@webrtc.org,phensman@webrtc.org

Change-Id: I440025777a17d8580526289d4198da1fc3f7d62e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: webrtc:9642, webrtc:9665
Reviewed-on: https://webrtc-review.googlesource.com/c/107348
Reviewed-by: Alessio Bazzica <alessiob@webrtc.org>
Commit-Queue: Alessio Bazzica <alessiob@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25291}
2018-10-22 13:00:44 +00:00
ff292f30d9 Remove deprecated barcode scanning functionality
This code is not used anymore, but it's not possible to land this CL
until issue webrtc:9665 is fixed.

Bug: webrtc:9642,webrtc:9665
Change-Id: Idb68e9bdf51b4239788cd6869dcb44dae87d7c56
Reviewed-on: https://webrtc-review.googlesource.com/c/95951
Reviewed-by: Paulina Hensman <phensman@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Magnus Jedvert <magjed@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25289}
2018-10-22 11:36:40 +00:00
6347bda432 Remove expat from generate_licenses.py.
This library is not used by WebRTC anymore.

Bug: chromium:896154
Change-Id: Ifc2f30b9425ef7ca3ff665cc03d11932316df71c
Reviewed-on: https://webrtc-review.googlesource.com/c/106780
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25248}
2018-10-18 09:02:54 +00:00
9b1d67982f Remove 'iOS32 Sim Debug (iOS 9.0)' from client.webrtc.
Bug: webrtc:9867
Change-Id: I66b4a3bb30bccc08bd1bd0c077948550d6e08072
Reviewed-on: https://webrtc-review.googlesource.com/c/106344
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25216}
2018-10-16 16:10:35 +00:00
84d282711b Add generate_ios_coverage_command.py script
This script simplifies getting iOS code coverage for real devices
and simulators. Although getting coverage for real devices is not
fully automated.

Bug: chromium:844647
Change-Id: Ib58173a9a6f61408ac1f8c7eaea712517b78e0ea
Reviewed-on: https://webrtc-review.googlesource.com/c/105663
Commit-Queue: Artem Titarenko <artit@webrtc.org>
Reviewed-by: Artem Titarenko <artit@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25202}
2018-10-16 10:26:11 +00:00
781b2bde69 Restore "device type" for iOS internal.client.webrtc
This partially reverts a94ad2920c7aa645e6d81bc17f7a1be17fef4be7
because of failures in iOS Perf trybots.

Bug: webrtc:7156
Change-Id: Icaf10ffc2941419bd89481f1d5c8e377801b24eb
Reviewed-on: https://webrtc-review.googlesource.com/c/105626
Reviewed-by: Artem Titarenko <artit@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Artem Titarenko <artit@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25142}
2018-10-12 12:55:03 +00:00
2558c4e938 Remove ortc folder.
Since it is currently unused and not actively maintained, code under
ortc/ will be deleted by this CL.

Bug: webrtc:9824
Change-Id: I20f890b1a1e5e1dbd2b3949af916ae0a6bc8a032
Reviewed-on: https://webrtc-review.googlesource.com/c/102601
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25082}
2018-10-10 09:06:57 +00:00
ab09039d2a Add comment that xcode version needs to be updated in two places
Bug: None
Change-Id: I6876c06079a06241fe8756941f1822d99c15fdcc
Reviewed-on: https://webrtc-review.googlesource.com/c/104483
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25051}
2018-10-08 13:42:33 +00:00
d2fb1bfeba Generate module.modulemap file when building Mac Framework
Without this file, the Framework can't be used by Swift projects.

Bug: webrtc:9142
Change-Id: I8803ec8b194dc116e133257e205f4620bb34a692
Reviewed-on: https://webrtc-review.googlesource.com/c/103340
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Anders Carlsson <andersc@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25047}
2018-10-08 12:32:45 +00:00
e6708f31a0 Notify a rotation about autoroll CLs
Also don't keep dry-run-only CLs as work-in-progress.

No-Try: True
Bug: chromium:836566
Change-Id: I3008845396d09c751443755f96a5d571389e1d2e
Reviewed-on: https://webrtc-review.googlesource.com/c/104620
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25046}
2018-10-08 12:20:41 +00:00
e9b74a77b1 Autoroller: don't run presubmit hooks
The presubmit lint checks require GN but it is not downloaded because runhooks is avoided on the autoroller bot.
The trybots in Gerrit UI should catch the same errors anyway, if they somehow happen.

Also minor cleanup of obsolete flag

No-Try: True
Bug: chromium:836566
Change-Id: I8bf03b8e155343f723c6fdda37210d9161da984c
Reviewed-on: https://webrtc-review.googlesource.com/c/103620
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24967}
2018-10-04 08:21:44 +00:00
dca5a2ca73 Autoroller: switch back to old-style "=" tags for TBR to work
This partially revers commit 1ee9160a2e0bc6381caca2b8c42f7ce5507619bc

No-Try: True
Bug: chromium:888417
Change-Id: I72b4f95235d5132e8e82065ce2a78329d2f42f52
Reviewed-on: https://webrtc-review.googlesource.com/102621
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24893}
2018-09-28 13:20:43 +00:00
d9664b8249 Whitespace change 2 to kick bots.
TBR=oprypin@webrtc.org

Change-Id: I3e2fb1278d4729c8419a29f9516c2f064696f29f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: chromium:877018
Reviewed-on: https://webrtc-review.googlesource.com/102562
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24887}
2018-09-28 11:06:55 +00:00
3f939bf215 Whitespace change to kick bots.
Tbr: oprypin@webrtc.org
Bug: chromium:877018
Change-Id: I29a619de34fa299753b856e0f813d314c5a8cba6
Reviewed-on: https://webrtc-review.googlesource.com/102542
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24885}
2018-09-28 09:57:36 +00:00
1ee9160a2e Don't use nonstandard tryjobs for autoroller
Bug: chromium:888417
Change-Id: I47b0f4e3ef19d88a906e52d0e994ea377a565afc
No-Try: True
Reviewed-on: https://webrtc-review.googlesource.com/102420
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24875}
2018-09-28 06:57:53 +00:00
d5bc865f13 [ios] Update swarming dimensions for Xcode 10
Bug: chromium:888513
Change-Id: I55fb5522cc4beee8e657d63bd2a166bce8b7291e
Reviewed-on: https://webrtc-review.googlesource.com/101944
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24844}
2018-09-26 09:50:35 +00:00
ea766fa7a2 Support e.g. --workers=2x to use two workers per core.
This is mostly useful for tests performing a lot of I/O and sleeping,
when you don't know on which architecture they end up running.

The syntax can also be used to reduce CPU load (e.g. --workers=0.5x).

Bug: webrtc:9717
Change-Id: I26b4552576b1dd56a69c2223da39f4bb1115bbf6
Reviewed-on: https://webrtc-review.googlesource.com/101643
Commit-Queue: Yves Gerey <yvesg@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24830}
2018-09-25 15:09:29 +00:00
4b9f3908ed Auto roller: [unittest cleanup] Proper patching mechanism.
Use mock.patch instead of setattr, deemed hackish and less robust.
As an additional benefit, mock is explictly activated and precisely scoped.

Bug: chromium:855108
Change-Id: I3644bb6773a4b95e50aa5b671292e108af1fd2e9
Reviewed-on: https://webrtc-review.googlesource.com/101660
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Yves Gerey <yvesg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24804}
2018-09-24 15:30:39 +00:00
3ee3c40f95 Add extra logging to roll_deps.py
This is to help debug a depot_tools auth problem and can be reverted
once it is solved.

Bug: skia:8394
Change-Id: I3c713fce6c6ba6edbd2498d95938b48a28eff588
Reviewed-on: https://webrtc-review.googlesource.com/101160
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Yves Gerey <yvesg@webrtc.org>
Commit-Queue: Eric Boren <borenet@google.com>
Cr-Commit-Position: refs/heads/master@{#24781}
2018-09-20 18:30:39 +00:00
3606cab8da Roll chromium_revision e4b02117a9..cc7b9c6822 (592264:592452)
Change log: e4b02117a9..cc7b9c6822
Full diff: e4b02117a9..cc7b9c6822

Changed dependencies
* src/base: 8130b147de..84eacf48e2
* src/ios: 7d51fa5227..e1bcf04272
* src/testing: fe4e1f210c..b1fa2ea487
* src/third_party: f29c2448d2..e49700f62c
* src/third_party/depot_tools: 07b5283a4e..69f640ec09
* src/tools: ae28316058..846c5a40f9
Added dependencies
* src/third_party/android_deps/libs/com_google_code_findbugs_jsr305
* src/third_party/android_deps/libs/com_google_j2objc_j2objc_annotations
* src/third_party/android_deps/libs/com_google_errorprone_error_prone_annotations
* src/third_party/android_deps/libs/org_codehaus_mojo_animal_sniffer_annotations
* src/third_party/android_deps/libs/org_checkerframework_checker_compat_qual
DEPS diff: e4b02117a9..cc7b9c6822/DEPS

No update to Clang.

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

Partial revert of 1548e9a2959360b327f57d2b2851995b9d1a397f.


Fix ANDROID_DEPS tags in DEPS file.

We need the same tags than Chromium's DEPS for detection purpose.

Also, fix format (will reduce noise in subsequent diff).

Change-Id: I8607fd00be55680d65757c1e17ab08de42b2143d
Reviewed-on: https://webrtc-review.googlesource.com/101062
Reviewed-by: Yves Gerey <yvesg@webrtc.org>
Commit-Queue: Yves Gerey <yvesg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24777}
2018-09-19 17:46:30 +00:00
31eb01faa5 Auto roller: Fix GenerateCommitMessage signature.
TODO: Add integration test.

Bug: chromium:855108
Change-Id: Ic892cd09e6712e9b7304e8b10b5fdc147b38a6bd
Reviewed-on: https://webrtc-review.googlesource.com/101040
Commit-Queue: Yves Gerey <yvesg@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24776}
2018-09-19 15:51:20 +00:00
26a6b5899f Auto roller: fix list of WEBRTC_ONLY_DEPS.
Very unfortunate typo.

Bug: chromium:855108
Change-Id: I236e8537537e7dad805d58c92f804ef021981574
Reviewed-on: https://webrtc-review.googlesource.com/101021
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Yves Gerey <yvesg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24775}
2018-09-19 14:33:40 +00:00
1548e9a295 Auto roller: don't complain about expected dependencies.
It's ok for some WebRTC dependencies not to be Chromium dependencies.
Explicitly list them instead of relying of CIDP discrimination.

Bug: chromium:855108
Change-Id: I2dafce488b28409cbce7e0c3167d92f48859084f
Reviewed-on: https://webrtc-review.googlesource.com/101000
Commit-Queue: Yves Gerey <yvesg@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24774}
2018-09-19 12:25:56 +00:00
401afd51bb Auto roller: improved tracking of android dependencies.
Until now, only revision changes were automatically rolled.
This CL detect new and removed dependencies in third_party/android_deps.

Change-Id: I4f83b7308be577115cc3ed57edd9881496428173
Bug: chromium:855108
Reviewed-on: https://webrtc-review.googlesource.com/100021
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24773}
2018-09-19 09:05:20 +00:00
b56706fcd9 Reland "Compile frame analyzer for the host machine on perf tests."
This is a reland of d8ff3f29ce92e27529e100ecf71afbae6334419f.

See https://webrtc-review.googlesource.com/c/src/+/100681/1..4 for
the fix. Error "Failed to open video file for emulated camera" should
be addressed by that change.

Original change's description:
> Compile frame analyzer for the host machine on perf tests.
>
> Bug: webrtc:9665
> Change-Id: I05c01ee4bef0995556b1a679498b3d9132de7c26
> Reviewed-on: https://webrtc-review.googlesource.com/100360
> Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
> Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#24756}

TBR=phoglund@webrtc.org, oprypin@webrtc.org

Bug: webrtc:9665
Change-Id: If6a4f2259dabf50718abf47c9cf303d143a1895a
Reviewed-on: https://webrtc-review.googlesource.com/100681
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24762}
2018-09-18 09:51:19 +00:00
6d800030ab Revert "Compile frame analyzer for the host machine on perf tests."
This reverts commit d8ff3f29ce92e27529e100ecf71afbae6334419f.

Reason for revert: It breaks perf tests.

Original change's description:
> Compile frame analyzer for the host machine on perf tests.
> 
> Bug: webrtc:9665
> Change-Id: I05c01ee4bef0995556b1a679498b3d9132de7c26
> Reviewed-on: https://webrtc-review.googlesource.com/100360
> Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
> Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#24756}

TBR=phoglund@webrtc.org,mbonadei@webrtc.org,oprypin@webrtc.org

Change-Id: I9d75dee68ef9257c707fe547ec32a22572ff582c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: webrtc:9665
Reviewed-on: https://webrtc-review.googlesource.com/100680
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24758}
2018-09-17 12:45:24 +00:00
d8ff3f29ce Compile frame analyzer for the host machine on perf tests.
Bug: webrtc:9665
Change-Id: I05c01ee4bef0995556b1a679498b3d9132de7c26
Reviewed-on: https://webrtc-review.googlesource.com/100360
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24756}
2018-09-17 11:23:40 +00:00
dc899dce9e Revert "Compile frame_analyzer instead of using a prebuilt version."
This reverts commit abac56b866efbfbdfef50e7a892e8c1c91ad5fbc.

Reason for revert: Breaks perf tests.

Original change's description:
> Compile frame_analyzer instead of using a prebuilt version.
> 
> Bug: webrtc:9665
> Change-Id: I589128d3f18a68a42094dacd910cd614a075a460
> Reviewed-on: https://webrtc-review.googlesource.com/99823
> Reviewed-by: Kári Helgason <kthelgason@webrtc.org>
> Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#24717}

TBR=phoglund@webrtc.org,mbonadei@webrtc.org,magjed@webrtc.org,oprypin@webrtc.org,kthelgason@webrtc.org

Change-Id: Ic0aab0a7aea18efbe802b9fca51a2b95533237c3
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: webrtc:9665
Reviewed-on: https://webrtc-review.googlesource.com/100105
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24721}
2018-09-13 10:29:26 +00:00
abac56b866 Compile frame_analyzer instead of using a prebuilt version.
Bug: webrtc:9665
Change-Id: I589128d3f18a68a42094dacd910cd614a075a460
Reviewed-on: https://webrtc-review.googlesource.com/99823
Reviewed-by: Kári Helgason <kthelgason@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24717}
2018-09-13 08:59:45 +00:00
ea5b76f7f2 Excluded tests and test utils from code coverage by generate_coverage_command.py
Bug: chromium:844647
Change-Id: I3b99cfcbeae99794f9600f232b560c47efeebc57
Reviewed-on: https://webrtc-review.googlesource.com/99682
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24692}
2018-09-11 15:06:50 +00:00
f2582da04e Revert "Partial revert of: 'Bump xcode versions for WebRTC bots.'"
This reverts commit 47c48b803202ec0132df06f20c7f16267524ab1a.

Reason for revert: Machines have been upgraded.

Original change's description:
> Partial revert of: 'Bump xcode versions for WebRTC bots.'
> 
> Partial revert of https://webrtc-review.googlesource.com/c/src/+/97060.
> These machines need to be updated to a newer OS version.
> 
> Bug: None
> Change-Id: Ice30ff9125eb366a6d6f93080ae7d0bceba1fe8b
> Reviewed-on: https://webrtc-review.googlesource.com/98400
> Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#24603}

TBR=phoglund@webrtc.org,mbonadei@webrtc.org,oprypin@webrtc.org

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

Bug: None
Change-Id: I7de961d6b6d3b472b0e6bbae545c1902e9a4d0c3
Reviewed-on: https://webrtc-review.googlesource.com/99223
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24663}
2018-09-10 19:25:56 +00:00
cf64a04acd Bump tryserver mac versions and xcode versions.
Patch by justincohen@, uploaded by phoglund@.

This moves our trybots to 10.13+, which is required for xcode 10,
which is now required by the build scripts.

Bug: None
Change-Id: I77d47bcb6696d290a397b098966ecc4ea1c0aeb9
Reviewed-on: https://webrtc-review.googlesource.com/97301
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24661}
2018-09-10 19:06:12 +00:00
a94ad2920c Bump iOS bots to iOS 11.
This should also solve the trigger problem on the iOS perf bot.

Remove device_type which appears to be ignored anyway. For instance,
device_type said iphone 6s but we got iPhone 8 when I actually looked
in the swarming dimension.

Bug: webrtc:7156
Change-Id: I1aa22e7f217deebf9eeee18363622e37ecc2a40e
Reviewed-on: https://webrtc-review.googlesource.com/99060
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24648}
2018-09-10 12:00:26 +00:00
47c48b8032 Partial revert of: 'Bump xcode versions for WebRTC bots.'
Partial revert of https://webrtc-review.googlesource.com/c/src/+/97060.
These machines need to be updated to a newer OS version.

Bug: None
Change-Id: Ice30ff9125eb366a6d6f93080ae7d0bceba1fe8b
Reviewed-on: https://webrtc-review.googlesource.com/98400
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24603}
2018-09-06 11:40:28 +00:00
d866544578 Upload WebRTC CLs from Chromium.
This CL removes some assumptions that were making it difficult to
upload a patch from the directory //third_party/webrtc in a
Chromium checkout.

Bug: webrtc:9705
Change-Id: I227ca492d5cf03875474ffd4d31abf387f947e5e
Reviewed-on: https://webrtc-review.googlesource.com/97600
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24549}
2018-09-04 11:14:07 +00:00
d05916dccf Bump xcode versions for WebRTC bots.
The build toolchain now requires xcode 10, so we have to roll.

Bug: None
Change-Id: Iafec62e7927ca8a81117710d09e2c42bcf18c0d1
Reviewed-on: https://webrtc-review.googlesource.com/97060
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24510}
2018-08-31 15:32:10 +00:00
cc22f51988 Removing the intelligibility enhancer.
The intelligibility enhancer is always disabled and it is the only non-test
target using the lapped transform in common_audio (which we planned to remove).

Bug: webrtc:9689, webrtc:5298
Change-Id: Ida65d3aa11ac366471e7e5cbc053108b376c67d8
Reviewed-on: https://webrtc-review.googlesource.com/96460
Commit-Queue: Alessio Bazzica <alessiob@webrtc.org>
Reviewed-by: Alex Loiko <aleloi@webrtc.org>
Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Per Åhgren <peah@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24504}
2018-08-30 21:29:57 +00:00
7bca8ca4e2 Obj-C SDK Cleanup
This CL separates the files under sdk/objc into logical directories, replacing
the previous file layout under Framework/.

A long term goal is to have some system set up to generate the files under
sdk/objc/api (the PeerConnection API wrappers) from the C++ code. In the shorter
term the goal is to abstract out shared concepts from these classes in order to
make them as uniform as possible.

The separation into base/, components/, and helpers/ are to differentiate between
the base layer's common protocols, various utilities and the actual platform
specific components.

The old directory layout that resembled a framework's internal layout is not
necessary, since it is generated by the framework target when building it.

Bug: webrtc:9627
Change-Id: Ib084fd83f050ae980649ca99e841f4fb0580bd8f
Reviewed-on: https://webrtc-review.googlesource.com/94142
Reviewed-by: Kári Helgason <kthelgason@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Reviewed-by: Henrik Andreassson <henrika@webrtc.org>
Commit-Queue: Anders Carlsson <andersc@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24493}
2018-08-30 10:42:41 +00:00
40f876d33e Simplify assertion.
Bug: None
Change-Id: I23830c5fe91af2e6ae736176c75b6d64270111d5
No-Try: True
Reviewed-on: https://webrtc-review.googlesource.com/96241
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Henrik Andreassson <henrika@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24455}
2018-08-27 14:36:40 +00:00
cd634ce933 Remove client.webrtc.branches.
The associated master has been turned down.

Bug: chromium:877279
Change-Id: I39c3a43680288b17094d1300926ec7a8bd427509
Reviewed-on: https://webrtc-review.googlesource.com/95882
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24421}
2018-08-24 08:19:37 +00:00
c7ca7b19c7 Revert "Temporarily skip linux_internal on autoroller tryjobs."
This reverts commit ae09d3966ce24f692be8adeaaed708819cef1df7.

Reason for revert: The bot is now fixed.

Original change's description:
> Temporarily skip linux_internal on autoroller tryjobs.
> 
> Run compile_lite instead, which affords some protection
> at least.
> 
> Bug: webrtc:9615
> Change-Id: I348d535ba2c0dd6cd438be1a90da5b3f64c17c93
> Reviewed-on: https://webrtc-review.googlesource.com/93467
> Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
> Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#24262}

TBR=phoglund@webrtc.org,oprypin@webrtc.org

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

Bug: webrtc:9615
Change-Id: Ib3bc539fb49cac1c118592ebc66fe82346d70f28
Reviewed-on: https://webrtc-review.googlesource.com/94860
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24335}
2018-08-20 06:40:12 +00:00
9923353600 Make ensure webcam script do nothing on linux.
Linux has all hw webcams now, but it's tricky to stop invoking this
script just on Linux, so just make it do nothing for now.

It should be safe to turn on video_capture_tests on linux
after this lands.

Bug: webrtc:9292, webrtc:9636
Change-Id: I6e86716b4c7ca43244596f806ff904b7fdf9201a
Reviewed-on: https://webrtc-review.googlesource.com/94769
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24334}
2018-08-20 06:20:09 +00:00
adc4879909 Make sure video_capture_tests run sequentially.
Making video_capture_tests have the non_parallel_console_test_launcher
type will pass --workers=1 to gtest-parallel, which ensures the tests
execute sequentially. This is necessary now that we're accessing a
real physical webcam, which is a system-wide resource that doesn't
work well if several processes access it concurrently.

Follow-ups:
1) get video_capture_test back up on Linux
2) drop sw webcams for Mac and Win
3) remove ensure_webcam_is_running.pu and surrounding machinery

Bug: webrtc:9292
Change-Id: I5e3347ad234f6b942de736513075097d79c0fd36
Reviewed-on: https://webrtc-review.googlesource.com/94761
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24325}
2018-08-17 09:54:52 +00:00
ae09d3966c Temporarily skip linux_internal on autoroller tryjobs.
Run compile_lite instead, which affords some protection
at least.

Bug: webrtc:9615
Change-Id: I348d535ba2c0dd6cd438be1a90da5b3f64c17c93
Reviewed-on: https://webrtc-review.googlesource.com/93467
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24262}
2018-08-10 12:55:08 +00:00
151c0cddd7 Add post-submit builders without dcheck_always_on
and remove dcheck_always_on from compile-only bots.

Bug: webrtc:9415
Change-Id: I7f2dc3a5443e5a5658c248be72bec79f3e3c3cca
Reviewed-on: https://webrtc-review.googlesource.com/93027
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24244}
2018-08-09 11:29:15 +00:00
29835996e9 Move spl_sqrt_floor dep to proper third_party directory
Bug: webrtc:8366
Change-Id: I326af5251dd88136dcc722e0ba1a2f9a8aebcf89
Reviewed-on: https://webrtc-review.googlesource.com/90405
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24205}
2018-08-07 10:33:13 +00:00
264bee8bab Remove memcheck.
Since the linux_memcheck trybot is no more, this CL removes all the
code needed to make it work.

Bug: webrtc:7737, webrtc:8356, webrtc:9570
Change-Id: I09a9467b8bf895146a3384c2c915b54662721af6
Reviewed-on: https://webrtc-review.googlesource.com/90863
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24201}
2018-08-07 07:40:08 +00:00
e22a26f6f1 Add 2 more OWNERS to tools_webrtc.
Bug: None
Change-Id: I3550652ac111363d2f0e29fb97e3804c8b5d92af
No-Try: True
Reviewed-on: https://webrtc-review.googlesource.com/90409
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24200}
2018-08-07 06:06:19 +00:00
3dc427fe34 Fix remaining target name for AppRTCMobile_stubbed_video_io_test_apk.
Fixes a target name that was missed in the last CL.

TBR=phoglund

Bug: webrtc:9588
No-Try: True
Change-Id: I704325666b758cee7eb080f8628fc839ab89831d
Reviewed-on: https://webrtc-review.googlesource.com/92389
Commit-Queue: Sami Kalliomäki <sakal@webrtc.org>
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24188}
2018-08-03 14:17:55 +00:00