Age | Commit message (Collapse) | Author |
|
am: cd56b9ddae
Change-Id: I7b1ed3c35ee9adfec375979ef07bd0c62adc01a9
|
|
* We should enable those warnings after fixing the upstream source code.
Bug: 64487164
Test: build with WITH_TIDY=1
Change-Id: I306b4eb26fc8729f21cab4d5b82c9e3b59d8db3e
|
|
am: f08d623d4e
Change-Id: I1e376149ca4c357b34538e21b4673ca4d788d876
|
|
am: 83cb4ee1d7
Change-Id: Ia8eae124a9a4a6d13740f8ceb4d32198dbfbb296
|
|
Fix a missing dependency.
Test: mma -j
Test: m -j -k BOARD_VNDK_VERSION=current has no error for webrtc
Bug: 62489821
Merged-In: I479a4addb59b42dd86cc5dfa02cd4ff6d1b6d626
Change-Id: I479a4addb59b42dd86cc5dfa02cd4ff6d1b6d626
(cherry picked from commit c6f833ab1ab6ccb6e0401fa61a182c516b849d87)
|
|
Native tests are not moved. Tests include:
webrtc_apm_process_test
webrtc_isac_test
webrtc_apm_unit_test
Test: mma -j
Test: m -j -k BOARD_VNDK_VERSION=current has no errors for webrtc
Bug: 62489821
Merged-In: I013303de263866cbf368f3f89327c5357f9cecdb
Change-Id: I013303de263866cbf368f3f89327c5357f9cecdb
(cherry picked from commit 8df7e85368569b7cd0afc1ce231b8b3a0ab49333)
|
|
SSL_CTX_set_max_version." am: 0d85cc3c3c am: 1335ab5cae
am: c0c6db4af3
Change-Id: Ie40178c8e1e8c2c9e13401142a202d9c61e8cbc1
|
|
These functions are identical. BoringSSL added these APIs, then OpenSSL
1.1.0 added similar ones but with slightly longer names. We're
standardizing on the OpenSSL names to avoid API skew.
This is a cherry-pick of https://codereview.webrtc.org/2550423004.
Test: mma
Change-Id: Iffe4416bd300d71bbaa94c7c127ae89d6286cd7b
|
|
am: d1d58ca589
Change-Id: Ib8e42e572f0c64557b4412b18584260be647f55f
|
|
We should not copy OWNERS files from upstream,
or the owners should be registered in Gerrit Code Review.
Bug: 33166666
Test: default build targets
Change-Id: Ibfd47e643f03678bb65880653383adb84809169d
|
|
Test: builds without warnings
Change-Id: Ic88c90167584c97689ac32e0c03d692560e14570
|
|
Suppress warnings until upstream can fix them.
BUG: 27074506
Change-Id: If7e6f190100fba025d25d2634d1c9a657cc24854
|
|
* git merge 04cb763
* See all upstream changes since the previous merge in branch aosp/upstream-master: git diff cb3f9bd..04cb763
* Modify webrtc/.gitignore to keep *.mk files.
* Removed old files from *.mk files:
- thread.cc
- thread_posix.cc
* Add new files to *.mk files:
- event_tracer.cc
* Android relevant upstream changes:
- Make Beamforming dynamically settable for Android platform builds
- Remove additional channel constraints when Beamforming is enabled in AudioProcessing
- Use an explicit identifier in Config
Change-Id: I384a4e8f6982c31c5bc70eef521bb2d90800b9a4
|
|
Replace LINUX, OSX and IOS defines with WEBRTC_ prefixed versions.
Remove no longer used defines from talk/build/common.gypi due to
previously migrated sources (into webrtc/p2p and webrtc/libjingle).
When this is rolled into Chromium, we can also clean up the platform
defines in
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjingle/libjingle.gyp
NOTRY=True
BUG=webrtc:5420
TESTED=Ran all compile trybots with --clobber flag.
TBR=tommi@webrtc.org
Review URL: https://codereview.webrtc.org/1588453005
Cr-Commit-Position: refs/heads/master@{#11254}
|
|
tests. (patchset #3 id:40001 of https://codereview.webrtc.org/1550773002/ )
Reason for revert:
We're getting boringssl version conflicts. Reverting for now.
Original issue's description:
> Update with new default boringssl no-aes cipher suites. Re-enable tests.
>
> This undoes https://codereview.webrtc.org/1533253002 (except the DEPS part).
>
> BUG=webrtc:5381
> R=davidben@webrtc.org, henrika@webrtc.org
>
> Committed: https://crrev.com/31c8d2eac5aec977f584ab0ae5a1d457d674f101
> Cr-Commit-Position: refs/heads/master@{#11250}
TBR=davidben@webrtc.org,henrika@webrtc.org,torbjorng@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5381
Review URL: https://codereview.webrtc.org/1586183002
Cr-Commit-Position: refs/heads/master@{#11253}
|
|
This undoes https://codereview.webrtc.org/1533253002 (except the DEPS part).
BUG=webrtc:5381
R=davidben@webrtc.org, henrika@webrtc.org
Review URL: https://codereview.webrtc.org/1550773002 .
Cr-Commit-Position: refs/heads/master@{#11250}
|
|
Previosly ToSesnsetiveString() wasn't working witn some implementations
of inet_ntop(). Rewrote it to avoid that dependency.
BUG=chromium:577344
R=pthatcher@webrtc.org, tommi@webrtc.org
Review URL: https://codereview.webrtc.org/1584793004 .
Cr-Commit-Position: refs/heads/master@{#11242}
|
|
It looks to me like targets :rtc_base_approved is logically a subset of
:rtc_base, and so any targets depending on :rtc_base expect to also get
access to the headers in :rtc_base_approved.
Thus I think it's appropriate for :rtc_base to have :rtc_base_approved in
public_deps, so that `gn check` will permit this without clients having to
explicitly depend on both.
NOTRY=True
Review URL: https://codereview.webrtc.org/1578833002
Cr-Commit-Position: refs/heads/master@{#11227}
|
|
There's no reason not to use std::move instead now that we can use the
C++11 standard library.
BUG=webrtc:5373
Review URL: https://codereview.webrtc.org/1531013003
Cr-Commit-Position: refs/heads/master@{#11225}
|
|
This changes for standalone webrtc applications.
BUG=
R=pthatcher@webrtc.org
Review URL: https://codereview.webrtc.org/1548733002 .
Cr-Commit-Position: refs/heads/master@{#11211}
|
|
TBR=guidou@webrtc.org
BUG=
Review URL: https://codereview.webrtc.org/1576723002 .
Cr-Commit-Position: refs/heads/master@{#11196}
|
|
To ease use of WebRTC in other codebases, update some macros
to match glibc's ansidecl.h, which uses double-underscores for attributes.
NOTRY=True
Review URL: https://codereview.webrtc.org/1571653002
Cr-Commit-Position: refs/heads/master@{#11185}
|
|
Review URL: https://codereview.webrtc.org/1570473002
Cr-Commit-Position: refs/heads/master@{#11163}
|
|
This CL is a follow-up on https://codereview.webrtc.org/1452903006/ which
moved the definition of SocketDispatcher to physicalsocketserver.h.
Here the duplicate implementations are merged with only some #ifdef parts.
BUG=
Review URL: https://codereview.webrtc.org/1537273002
Cr-Commit-Position: refs/heads/master@{#11160}
|
|
BUG=
R=tkchin@webrtc.org
Review URL: https://codereview.webrtc.org/1517253005 .
Patch from Jon Hjelle <hjon@andyet.net>.
Cr-Commit-Position: refs/heads/master@{#11156}
|
|
Macro incorrectly displays DISABLED_ON_ANDROID in test names for
parameterized tests under --gtest_list_tests, causing tests to be
disabled on all platforms since they contain the DISABLED_ prefix rather
than their expanded variants.
This expands the macro variants to inline if they're disabled or not,
and removes building some tests under configurations where they should
fail, instead of building them but disabling them by default.
The change also removes gtest_disable.h as an unused include from many
other files.
BUG=webrtc:5387, webrtc:5400
R=kjellander@webrtc.org, phoglund@webrtc.org
TBR=henrik.lundin@webrtc.org
Review URL: https://codereview.webrtc.org/1547343002 .
Cr-Commit-Position: refs/heads/master@{#11150}
|
|
The changes in https://chromium.googlesource.com/chromium/src/+/d66326c..4df108a/build/common.gypi
enables a lot more warnings, which have been disabled/fixed in this CL.
See tracking bugs for remaining work.
Change log: https://chromium.googlesource.com/chromium/src/+log/d66326c..4df108a
Full diff: https://chromium.googlesource.com/chromium/src/+/d66326c..4df108a
Changed dependencies:
* src/buildtools: https://chromium.googlesource.com/chromium/buildtools.git/+log/fee7f1e..6d0c448
* src/third_party/libsrtp: https://chromium.googlesource.com/chromium/deps/libsrtp.git/+log/b8dd754..8a7662a
DEPS diff: https://chromium.googlesource.com/chromium/src/+/d66326c..4df108a/DEPS
No update to Clang.
BUG=webrtc:5397, webrtc:5398, webrtc:5399
TBR=hta@webrtc.org, perkj@webrtc.org
NOTRY=True
Review URL: https://codereview.webrtc.org/1553033002
Cr-Commit-Position: refs/heads/master@{#11147}
|
|
BUG=chromium:573250
Review URL: https://codereview.webrtc.org/1552863002
Cr-Commit-Position: refs/heads/master@{#11145}
|
|
ICE servers are now passed directly into PortAllocator,
making PortAllocatorFactoryInterface redundant. This CL also
moves SetNetworkIgnoreMask to PortAllocator.
R=phoglund@webrtc.org, pthatcher@webrtc.org, tkchin@webrtc.org
Review URL: https://codereview.webrtc.org/1520963002 .
Cr-Commit-Position: refs/heads/master@{#11139}
|
|
Enable IPv6 temporary address filtering on iOS.
We'll only use temporary address for IPv6. However, due to a bug in iOS sdk, the necessary headers are not included. This change copies the minimum necessary definitions such that we could retrieve the ip attributes.
BUG=webrtc:4343
Committed: https://crrev.com/29488c23644721c10a45eba74c67602b0262e582
Cr-Commit-Position: refs/heads/master@{#11114}
patch from issue 1531763006 at patchset 200001 (http://crrev.com/1531763006#ps200001)
TBR=pthatcher@webrtc.org
Review URL: https://codereview.webrtc.org/1551703002 .
Cr-Commit-Position: refs/heads/master@{#11133}
|
|
This reverts commit 29488c23644721c10a45eba74c67602b0262e582.
This broke chromium.fyi bot.
TBR=pthatcher@webrtc.org
BUG=
Review URL: https://codereview.webrtc.org/1547883002 .
Cr-Commit-Position: refs/heads/master@{#11115}
|
|
We'll only use temporary address for IPv6. However, due to a bug in iOS sdk, the necessary headers are not included. This change copies the minimum necessary definitions such that we could retrieve the ip attributes.
BUG=webrtc:4343
Review URL: https://codereview.webrtc.org/1531763006
Cr-Commit-Position: refs/heads/master@{#11114}
|
|
1. It signals network changed events whenever there are more than one IP address in a network.
2. It does not signal network changed events if a network disconnects and connects again.
Also changed DumpNetworks for better debugging.
BUG=webrtc:5096
Review URL: https://codereview.webrtc.org/1421433003
Cr-Commit-Position: refs/heads/master@{#11107}
|
|
I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same problem).
Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8
Changed dependencies:
* src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv: https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss: https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS
No update to Clang.
NOTRY=True
BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org
Review URL: https://codereview.webrtc.org/1533253002
Cr-Commit-Position: refs/heads/master@{#11095}
|
|
This patch removes StringToIP() methods as fixes the TODO there and
there are no callers at the moment for these methods.
BUG=None
R=perkj@webrtc.org
Review URL: https://codereview.webrtc.org/1535993002
Cr-Commit-Position: refs/heads/master@{#11088}
|
|
Incorrect argument order, also added unittest which should've been there
in the first place.
Also renames AtomicLoadPtr to AcquireLoadPtr to match non-ptr version.
BUG=
R=tommi@webrtc.org
Review URL: https://codereview.webrtc.org/1537923003 .
Cr-Commit-Position: refs/heads/master@{#11086}
|
|
We have updated the uses of ARRAY_SIZE to arraysize in past patches:
https://chromium.googlesource.com/external/webrtc/+/5237aaf243d29732f59557361b7a993c0a18cf0e
https://chromium.googlesource.com/external/webrtc/+/fa5d0dbd1e4de062d105b065fc2e40da47ef827a
BUG=None
R=tommi@webrtc.org
Review URL: https://codereview.webrtc.org/1537943002
Cr-Commit-Position: refs/heads/master@{#11085}
|
|
There is an issue in PhysicalSocket::Accept where the flag to continue
listening is not set in "enabled_events_" if "accept" returns an error.
This CL fixes this (initial idea by silviu.cpp@gmail.com).
BUG=webrtc:2030
Review URL: https://codereview.webrtc.org/1452903006
Cr-Commit-Position: refs/heads/master@{#11080}
|
|
Missing a cast.
BUG=webrtc:5365
Review URL: https://codereview.webrtc.org/1529043003
Cr-Commit-Position: refs/heads/master@{#11074}
|
|
Review URL: https://codereview.webrtc.org/1531173003
Cr-Commit-Position: refs/heads/master@{#11068}
|
|
rtc::ScopedVector
We can now use std::move instead!
This CL leaves the Pass methods in place; a follow-up CL will add deprecation annotations to them.
Review URL: https://codereview.webrtc.org/1460043002
Cr-Commit-Position: refs/heads/master@{#11064}
|
|
BUG=
Review URL: https://codereview.webrtc.org/1530003004
Cr-Commit-Position: refs/heads/master@{#11061}
|
|
This makes it possible to build WebRTC using Clang on Windows.
Depends on https://codereview.webrtc.org/1524703006/
BUG=webrtc:5360, webrtc:5366
NOTRY=True
Review URL: https://codereview.webrtc.org/1522223002
Cr-Commit-Position: refs/heads/master@{#11058}
|
|
Also convert DOS->Unix line endings in two of the OWNERS files.
NOTRY=True
NOPRESUBMIT=True
R=niklas.enbom@webrtc.org
Review URL: https://codereview.webrtc.org/1530003003 .
Cr-Commit-Position: refs/heads/master@{#11056}
|
|
Review URL: https://codereview.webrtc.org/1494133003
Cr-Commit-Position: refs/heads/master@{#11042}
|
|
This makes it clearer that the IP address has been stripped.
BUG=chromium:254329
Review URL: https://codereview.webrtc.org/1516163003
Cr-Commit-Position: refs/heads/master@{#11005}
|
|
BUG=
Review URL: https://codereview.webrtc.org/1499653003
Cr-Commit-Position: refs/heads/master@{#11000}
|
|
BUG=chromium:121673
R=solenberg@webrtc.org
Review URL: https://codereview.webrtc.org/1505253004 .
Cr-Commit-Position: refs/heads/master@{#10995}
|
|
Prints tid as a DWORD on Windows and an int on other systems.
BUG=
R=thakis@chromium.org, tommi@webrtc.org
Review URL: https://codereview.webrtc.org/1522483002 .
Cr-Commit-Position: refs/heads/master@{#10992}
|
|
I managed to cast the wrong parameter in #10982 :-(
BUG=82385
Review URL: https://codereview.webrtc.org/1520493003
Cr-Commit-Position: refs/heads/master@{#10983}
|