Commit Graph

10736 Commits

Author SHA1 Message Date
53c317c4b9 Roll chromium_revision 4688e75..8a1fcdb (366364:366529)
Change log: 4688e75..8a1fcdb
Full diff: 4688e75..8a1fcdb

No dependencies changed.
No update to Clang.

TBR=

Review URL: https://codereview.webrtc.org/1543753003

Cr-Commit-Position: refs/heads/master@{#11110}
2015-12-22 04:00:10 +00:00
cfb7f01fd6 Disable VideoSendStreamTest.ReconfigureBitratesSetsEncoderBitratesCorrectly
due to flakiness on LinuxAsan.

BUG=webrtc:5382
TBR=kjellander@webrtc.org

Review URL: https://codereview.webrtc.org/1541923003

Cr-Commit-Position: refs/heads/master@{#11109}
2015-12-21 21:35:00 +00:00
e6bf587259 Deleted VideoCapturer::screencast_max_pixels, together with
VideoChannel::GetScreencastMaxPixels and VideoChannel::GetScreencastFps.

Unused in webrtc, also unused in everything indexed by google and chromium code search. With the exception of the magicflute plugin, which I'm told doesn't matter.

Review URL: https://codereview.webrtc.org/1532133002

Cr-Commit-Position: refs/heads/master@{#11108}
2015-12-21 21:18:18 +00:00
db8cf50c59 Fix two problems in network.cc:
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}
2015-12-21 21:08:54 +00:00
1227e8b345 [rtp_rtcp] time helper functions
RTP timestams helper functions moved from rtp_utility
  added functions to deal with CompactNtp timestamps

R=åsapersson
BUG=webrtc:5260

Review URL: https://codereview.webrtc.org/1535113002

Cr-Commit-Position: refs/heads/master@{#11106}
2015-12-21 19:06:56 +00:00
5908c71128 Lint fix for webrtc/modules/video_coding PART 3!
Trying to submit all changes at once proved impossible since there were
too many changes in too many files. The changes to PRESUBMIT.py
will be uploaded in the last CL.
(original CL: https://codereview.webrtc.org/1528503003/)

BUG=webrtc:5309
TBR=mflodman@webrtc.org

Review URL: https://codereview.webrtc.org/1540243002

Cr-Commit-Position: refs/heads/master@{#11105}
2015-12-21 16:23:29 +00:00
f5b1abf5b0 Roll chromium_revision c844be9..4688e75 (366322:366364)
Change log: c844be9..4688e75
Full diff: c844be9..4688e75

No dependencies changed.
No update to Clang.

TBR=

Review URL: https://codereview.webrtc.org/1533303002

Cr-Commit-Position: refs/heads/master@{#11104}
2015-12-21 14:51:27 +00:00
de94c132e0 Add webrtc/audio and webrtc/call to WATCHLISTS.
Also adds current OWNERS to the corresponding watchlists.

BUG=
R=mflodman@webrtc.org

Review URL: https://codereview.webrtc.org/1533093002 .

Cr-Commit-Position: refs/heads/master@{#11103}
2015-12-21 13:30:01 +00:00
9d3ab61325 Lint fix for webrtc/modules/video_coding PART 2!
Trying to submit all changes at once proved impossible since there were
too many changes in too many files. The changes to PRESUBMIT.py
will be uploaded in the last CL.
(original CL: https://codereview.webrtc.org/1528503003/)

BUG=webrtc:5309
TBR=mflodman@webrtc.org

Review URL: https://codereview.webrtc.org/1543503002

Cr-Commit-Position: refs/heads/master@{#11102}
2015-12-21 12:12:45 +00:00
ff483617a4 Step 1 to prepare call_test.* for combined audio/video tests.
Also move (and clean up includes) rampup_tests.* to webrtc/call in preparation for combined audio/video ramp-up tests.

No functional changes.

BUG=webrtc:5263

Review URL: https://codereview.webrtc.org/1537273003

Cr-Commit-Position: refs/heads/master@{#11101}
2015-12-21 11:14:05 +00:00
cce46fc108 Lint fix for webrtc/modules/video_coding PART 1!
Trying to submit all changes at once proved impossible since there were
too many changes in too many files. The changes to PRESUBMIT.py
will be uploaded in the last CL.
(original CL: https://codereview.webrtc.org/1528503003/)

BUG=webrtc:5309
TBR=mflodman@webrtc.org

Review URL: https://codereview.webrtc.org/1541803002

Cr-Commit-Position: refs/heads/master@{#11100}
2015-12-21 11:04:57 +00:00
53805324c0 Rename RTC_HISTOGRAM_* macros to RTC_HISTOGRAM_*_SPARSE_* to indicate that these are for infrequent updates.
This implementation will be replaced by a faster one and sparse will be removed.

BUG=webrtc:5283

Review URL: https://codereview.webrtc.org/1530913002

Cr-Commit-Position: refs/heads/master@{#11099}
2015-12-21 09:46:25 +00:00
9fca7e18c3 A unittest that reports the statistics for the duration of an APM stream processing API call.
BUG=webrtc:5099

Committed: https://crrev.com/880896ab0976bbf86a6753d0c900c70e51f421cb
Cr-Commit-Position: refs/heads/master@{#10786}

Review URL: https://codereview.webrtc.org/1436553004

Cr-Commit-Position: refs/heads/master@{#11098}
2015-12-21 07:13:46 +00:00
c693820f7f CQ: Add linux_libfuzzer_rel trybot as default.
NOTRY=True
BUG=chromium:570439
TBR=pbos@webrtc.org

Review URL: https://codereview.webrtc.org/1540773002

Cr-Commit-Position: refs/heads/master@{#11097}
2015-12-21 06:15:17 +00:00
54bab12e1f Roll chromium_revision db567a8..c844be9 (366304:366322)
Change log: db567a8..c844be9
Full diff: db567a8..c844be9

No dependencies changed.
No update to Clang.

TBR=

Review URL: https://codereview.webrtc.org/1540813002

Cr-Commit-Position: refs/heads/master@{#11096}
2015-12-21 03:59:08 +00:00
2f042f26a3 Roll chromium_revision 1b6c421..db567a8 (365999:366304)
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: 1b6c421..db567a8
Full diff: 1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv: 1019e45..1ccbf8f
* src/third_party/nss: a676aa0..aee1b12
DEPS diff: 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}
2015-12-20 20:25:17 +00:00
a4df27b671 Revert of Reland "Added option to specify a maximum file size when recording an AEC dump." (patchset #2 id:20001 of https://codereview.webrtc.org/1541633002/ )
Reason for revert:
Compile error on Android needs to be fixed before relanding.

Original issue's description:
> Reland "Added option to specify a maximum file size when recording an AEC dump.", commit ae2c5ad12afc8cc29fe9c59dea432b697b871a87.
>
> The revert of the original CL was commit 36d4c545007129446e551c45c17b25377dce89a4.
> Original review: https://codereview.webrtc.org/1413483003/
>
> The original CL changes a function on audio_processing.h that is used by Chrome, this CL adds back the old function.
>
> NOTRY=true
> TBR=glaznev@webrtc.org, henrik.lundin@webrtc.org, solenberg@google.com, henrikg@webrtc.org, perkj@webrtc.org
> BUG=webrtc:4741
>
> Committed: https://crrev.com/f4f5cb09277d5ef6aeac8341e5f54a055867803a
> Cr-Commit-Position: refs/heads/master@{#11093}

TBR=glaznev@webrtc.org,henrik.lundin@webrtc.org,solenberg@google.com,henrikg@webrtc.org,perkj@webrtc.org,kwiberg@webrtc.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:4741

Review URL: https://codereview.webrtc.org/1537213002

Cr-Commit-Position: refs/heads/master@{#11094}
2015-12-19 18:14:18 +00:00
f4f5cb0927 Reland "Added option to specify a maximum file size when recording an AEC dump.", commit ae2c5ad12afc8cc29fe9c59dea432b697b871a87.
The revert of the original CL was commit 36d4c545007129446e551c45c17b25377dce89a4.
Original review: https://codereview.webrtc.org/1413483003/

The original CL changes a function on audio_processing.h that is used by Chrome, this CL adds back the old function.

NOTRY=true
TBR=glaznev@webrtc.org, henrik.lundin@webrtc.org, solenberg@google.com, henrikg@webrtc.org, perkj@webrtc.org
BUG=webrtc:4741

Review URL: https://codereview.webrtc.org/1541633002

Cr-Commit-Position: refs/heads/master@{#11093}
2015-12-19 18:02:39 +00:00
bd7d8f7e2b Adding a MediaStream parameter to createSender.
This will allow an app to create senders with the same stream id,
without SDP munging.

Review URL: https://codereview.webrtc.org/1538673002

Cr-Commit-Position: refs/heads/master@{#11092}
2015-12-19 00:58:51 +00:00
92594a30ce Moving FFT on farend signal to where it is used in AEC (bit exact).
Currently, FFT is performance when AEC buffers farend signal. This has some drawbacks
1. memory inefficiency: two ring buffers are needed;
2. computation inefficiency: if ringbuffer gets wrapped around, some FFT computation will be wasted;
3. accessibility: the main AEC function looses accessibility to the time-domain signal.

Therefore, this CL tries to buffer time domain data, which is buffered any way if a debugging macro is defined, and calculate the FFTs where they are actually used.

BUG=

Review URL: https://codereview.webrtc.org/1512573003

Cr-Commit-Position: refs/heads/master@{#11091}
2015-12-18 23:31:19 +00:00
4ff818e614 Make download_from_google_storage print less during runhooks.
Add the --quiet flag to the download_from_google_storage runhooks
step to prevent it from spamming the console when all the files
are already downloaded.

NOTRY=True

Review URL: https://codereview.webrtc.org/1527713003

Cr-Commit-Position: refs/heads/master@{#11090}
2015-12-18 20:29:33 +00:00
740c367af3 iSAC: Remove unnecessary WEBRTC_LINUX define.
I can only find one use in iSAC codebase:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc/modules/audio_coding/test/iSACTest.cc&l=19

It's the prime suspect for causing a compilation error for iOS failing to
include linux/net.h which is being included in
webrtc/voice_engine/voice_engine_defines.h

NOTRY=True

Review URL: https://codereview.webrtc.org/1539883002

Cr-Commit-Position: refs/heads/master@{#11089}
2015-12-18 20:28:28 +00:00
c155b16b22 remove deprecated StringToIP() methods from SocketAddress API
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}
2015-12-18 16:13:16 +00:00
36d4c54500 Revert of Added option to specify a maximum file size when recording an AEC dump. (patchset #5 id:120001 of https://codereview.webrtc.org/1413483003/ )
Reason for revert:
Breaks Chrome-FYI bots because of a change in the StartDebugRecording function in audio_processing.h, that is called from Chrome.

Original issue's description:
> Added option to specify a maximum file size when recording an AEC dump.
>
> For applications with a strict filesize limit for debug files,
> I added an option to specify a maximum filesize for AEC dumps. An
> existing unit test is extended to check that the feature works as
> advertised.
>
> BUG=webrtc:4741
> TBR=glaznev@webrtc.org
>
> Committed: https://crrev.com/ae2c5ad12afc8cc29fe9c59dea432b697b871a87
> Cr-Commit-Position: refs/heads/master@{#11081}

TBR=pthatcher@webrtc.org,henrik.lundin@webrtc.org,henrikg@webrtc.org,solenberg@webrtc.org,andrew@webrtc.org,kwiberg@webrtc.org,perkj@webrtc.org,glaznev@webrtc.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:4741

Review URL: https://codereview.webrtc.org/1533913004

Cr-Commit-Position: refs/heads/master@{#11087}
2015-12-18 16:05:21 +00:00
455a252453 Fix pointer compare-and-swap on Windows.
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}
2015-12-18 16:00:35 +00:00
c1cd566cc6 delete basictypes.h header
We have updated the uses of ARRAY_SIZE to arraysize in past patches:

5237aaf243
fa5d0dbd1e

BUG=None
R=tommi@webrtc.org

Review URL: https://codereview.webrtc.org/1537943002

Cr-Commit-Position: refs/heads/master@{#11085}
2015-12-18 15:33:09 +00:00
b7d9a97ce4 Expose codec implementation names in stats.
Used to distinguish between software/hardware encoders/decoders and
other implementation differences. Useful for tracking quality
regressions related to specific implementations.

BUG=webrtc:4897
R=hta@webrtc.org, mflodman@webrtc.org, stefan@webrtc.org

Review URL: https://codereview.webrtc.org/1406903002 .

Cr-Commit-Position: refs/heads/master@{#11084}
2015-12-18 15:01:23 +00:00
6c6510afad audio_device: Move sources into platform-conditions.
This should solve a problem discovered when converting from GYP to
other project formats, where the source files weren't included correctly
for each platform.

Two other targets in WebRTC have similar source files, which are correctly
generated for each platform:
* video_render_module_internal_impl
* video_capture_module_internal_impl
They both list the sources as it's changed to in this CL.

NOTRY=True

Review URL: https://codereview.webrtc.org/1536923003

Cr-Commit-Position: refs/heads/master@{#11083}
2015-12-18 12:33:34 +00:00
9b7fc7f25d Defines for ARM and MIPS CPU types.
This removes a dependency on Chromium's build/build_config.h
(which is not allowed).
The added defines are identical to the ones in build/build_config.h.

NOTRY=True

Review URL: https://codereview.webrtc.org/1532333002

Cr-Commit-Position: refs/heads/master@{#11082}
2015-12-18 12:28:49 +00:00
ae2c5ad12a Added option to specify a maximum file size when recording an AEC dump.
For applications with a strict filesize limit for debug files,
I added an option to specify a maximum filesize for AEC dumps. An
existing unit test is extended to check that the feature works as
advertised.

BUG=webrtc:4741
TBR=glaznev@webrtc.org

Review URL: https://codereview.webrtc.org/1413483003

Cr-Commit-Position: refs/heads/master@{#11081}
2015-12-18 11:53:42 +00:00
095ae15d6b Keep listening if "accept" returns an invalid socket.
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}
2015-12-18 09:40:03 +00:00
88518a22c6 Use NV21 instead of YUV12 and clean up.
BUG=webrtc:5375

Review URL: https://codereview.webrtc.org/1530843002

Cr-Commit-Position: refs/heads/master@{#11079}
2015-12-18 08:37:10 +00:00
48477c1c6a MediaCodecVideoEncoder, set timestamp on the encoder surface when drawing a texture.
BUG=webrtc:4993

Review URL: https://codereview.webrtc.org/1523843006

Cr-Commit-Position: refs/heads/master@{#11078}
2015-12-18 08:34:44 +00:00
fc96bd1f8b Roll chromium_revision e78bc2f..1b6c421 (365856:365999)
Change log: e78bc2f..1b6c421
Full diff: e78bc2f..1b6c421

Changed dependencies:
* src/third_party/openmax_dl: 7a179b9..ff8766d
DEPS diff: e78bc2f..1b6c421/DEPS

No update to Clang.

TBR=

Review URL: https://codereview.webrtc.org/1534883003

Cr-Commit-Position: refs/heads/master@{#11077}
2015-12-18 04:01:07 +00:00
77fa59d789 Fix build break in google3 import caused by https://codereview.webrtc.org/1532543003
TBR=pthatcher@webrtc.org
BUG=

Review URL: https://codereview.webrtc.org/1537683003

Cr-Commit-Position: refs/heads/master@{#11076}
2015-12-18 02:02:35 +00:00
4638331fd8 DTLS-SRTP set up is bypassed when the channel has been writable.
This regression was introduced by CL 1505573002 to support remote fingerprint update. What happened is that during PrAnswer, we incorrectly do not apply bundle. However, the channel has become writable at that time. When Answer comes, we still reset the srtp_filter but since the channel has been writable, the new SRTP context has never been applied.

We're making sure that we could always apply SRTP context even when channel has been writable. We'll address the issue that bundle should apply even in PrAnswer in a different CL.

BUG=568734

Review URL: https://codereview.webrtc.org/1532543003

Cr-Commit-Position: refs/heads/master@{#11075}
2015-12-18 00:46:04 +00:00
efb047d2dd Compilation failed with openssl.
Missing a cast.

BUG=webrtc:5365

Review URL: https://codereview.webrtc.org/1529043003

Cr-Commit-Position: refs/heads/master@{#11074}
2015-12-17 21:45:03 +00:00
933f3ec924 Roll chromium_revision ddfc1fe..e78bc2f (365801:365856)
Change log: ddfc1fe..e78bc2f
Full diff: ddfc1fe..e78bc2f

No dependencies changed.
No update to Clang.

TBR=

Review URL: https://codereview.webrtc.org/1534863002

Cr-Commit-Position: refs/heads/master@{#11073}
2015-12-17 20:04:03 +00:00
002f0d09c9 VP9: Set speed setting to 8 for ARM.
At speed 8, vp9 on ARM is currently ~2x times slower than vp8 on ARM (speed -12).

Update some parameters in videoprocessor_integrationtest.cc
to make tests pass on android (which uses the new speed setting).

TBR=stefan@webrtc.org
BUG=

Review URL: https://codereview.webrtc.org/1526973004 .

Cr-Commit-Position: refs/heads/master@{#11072}
2015-12-17 17:49:39 +00:00
5a4ce2fd33 Deleted declaration of VideoCaptureInput::DeliverI420Frame
It appears unimplemented and unused.

BUG=

Review URL: https://codereview.webrtc.org/1539513002

Cr-Commit-Position: refs/heads/master@{#11071}
2015-12-17 17:37:26 +00:00
a0b9549b88 Roll gtest-parallel.
BUG=
R=kwiberg@webrtc.org

Review URL: https://codereview.webrtc.org/1534773002

Cr-Commit-Position: refs/heads/master@{#11070}
2015-12-17 15:15:41 +00:00
369f828bfe Adding trace events for the APM render and capture stream processing functions.
BUG=webrtc:5099

Review URL: https://codereview.webrtc.org/1536613002

Cr-Commit-Position: refs/heads/master@{#11069}
2015-12-17 14:42:42 +00:00
9390f84a4a Use std::nullptr_t instead of decltype(nullptr)
Review URL: https://codereview.webrtc.org/1531173003

Cr-Commit-Position: refs/heads/master@{#11068}
2015-12-17 14:20:32 +00:00
1e0cfd9a46 Add VP8 and H264 depacketizer fuzzers.
Also removes listing of targets in webrtc_fuzzers which is very prone to
not being up to date. They're not required for ClusterFuzz integration
or building locally. This also means that adding fuzzers won't require
approval outside the fuzzers directory.

BUG=webrtc:4771
R=kjellander@webrtc.org, stefan@webrtc.org

Review URL: https://codereview.webrtc.org/1518973003 .

Cr-Commit-Position: refs/heads/master@{#11067}
2015-12-17 13:28:28 +00:00
9d98f217c3 Roll chromium_revision 68898fb..ddfc1fe (365698:365801)
Change log: 68898fb..ddfc1fe
Full diff: 68898fb..ddfc1fe

Changed dependencies:
* src/buildtools: 68e3c23..fee7f1e
DEPS diff: 68898fb..ddfc1fe/DEPS

No update to Clang.

TBR=

Review URL: https://codereview.webrtc.org/1525073006

Cr-Commit-Position: refs/heads/master@{#11066}
2015-12-17 12:32:49 +00:00
a689b44c17 Add tracing to NetEqImpl::InsertPacket
BUG=webrtc:5167
R=pbos@webrtc.org

Review URL: https://codereview.webrtc.org/1525423004

Cr-Commit-Position: refs/heads/master@{#11065}
2015-12-17 11:50:11 +00:00
0eb15ed7b8 Don't call the Pass methods of rtc::Buffer, rtc::scoped_ptr, and 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}
2015-12-17 11:04:24 +00:00
e376f0f2df Add Windows Clang trybots to the default set.
BUG=chromium:568952
NOTRY=True

Review URL: https://codereview.webrtc.org/1528303002

Cr-Commit-Position: refs/heads/master@{#11063}
2015-12-17 06:54:57 +00:00
e40dedb03f Roll chromium_revision 004c7b4..68898fb (365580:365698)
Change log: 004c7b4..68898fb
Full diff: 004c7b4..68898fb

No dependencies changed.
No update to Clang.

TBR=

Review URL: https://codereview.webrtc.org/1533523005

Cr-Commit-Position: refs/heads/master@{#11062}
2015-12-17 04:14:46 +00:00
a08925791c Cleanup use of "do { ... } while (0)".
BUG=

Review URL: https://codereview.webrtc.org/1530003004

Cr-Commit-Position: refs/heads/master@{#11061}
2015-12-17 02:38:34 +00:00