Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(365)

Issue 2326953004: Roll WebRTC 14139:14157 (15 commits) (Closed)

Created:
4 years, 3 months ago by magjed_chromium
Modified:
4 years, 1 month ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll WebRTC 14139:14157 (15 commits) Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/3d67882..c46742b $ git log 3d67882..c46742b --date=short --no-merges --format=%ad %ae %s 2016-09-09 kwiberg@webrtc.org rtc::Optional: Tell sanitizers that unset values aren't OK to access 2016-09-09 philipel@webrtc.org Added ClearTo(seq_num) to RtpFrameReferenceFinder. 2016-09-09 kthelgason@webrtc.org Reland of move all reference to carbon api (patchset #1 id:1 of https://codereview.webrtc.org/2317343003/ ) 2016-09-09 kwiberg@webrtc.org VadCore: Allow signed multiplication overflow that we don't know how to fix 2016-09-09 kjellander@webrtc.org MB: Disable more parts of the GYP build. 2016-09-09 kjellander@webrtc.org Remove references to .isolate files that are no longer needed. 2016-09-09 hbos@webrtc.org Renamed RTCStatsReport to RTCLegacyStatsReport in objc files. 2016-09-09 sakal@webrtc.org Add sakal as an OWNER to some Android files. 2016-09-09 asapersson@webrtc.org Update AvgCounter to have the ability to include last period metric for subsequent intervals without samples (e.g. for non-periodic updated stats). 2016-09-09 sakal@webrtc.org Make UMA stats creation available in the Java interface. 2016-09-08 magjed@webrtc.org Add magjed@ as owner of webrtc/common_video 2016-09-08 kjellander@webrtc.org Revert of Add a DEPS gclient hook to prune corrupt mockito remote. (patchset #1 id:1 of https://codereview.webrtc.org/2326523002/ ) 2016-09-08 kjellander@webrtc.org Revert of Separating video settings in VideoQualityTest. (patchset #2 id:20001 of https://codereview.webrtc.org/2312613003/ ) 2016-09-08 peah@webrtc.org This CL refactors the buffering of the incoming near-end signal inside the AEC. This solves the following issues: -Even though the buffering was previously done using ringbuffers, those were inefficiently used which caused a lot of hidden memcopys. -The ringbuffers wasted a lot of space in the AEC state as they were too long. -The lowest and two upper bands were decoupled in the buffering, which required extra code to handle. -On top of the ringbuffers there was a second linear buffer that was stored in the state which caused even more data to be stored on the state. -The incoming nearend frames were passed to the functions in the form of buffers on the state, which made the code harder to read as it was not immediately clear where the nearend signal was used, and when it was modified. 2016-09-08 henrikg@webrtc.org Change owner of webrtc/test/channel_transport to solenberg@. TBR= CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng BUG= Committed: https://crrev.com/802aefedad11037059a1afcd0bd37614ba783e8d Cr-Commit-Position: refs/heads/master@{#417564}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2326953004/1
4 years, 3 months ago (2016-09-09 11:13:28 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-09 12:24:56 UTC) #3
commit-bot: I haz the power
4 years, 3 months ago (2016-09-09 12:26:51 UTC) #5
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/802aefedad11037059a1afcd0bd37614ba783e8d
Cr-Commit-Position: refs/heads/master@{#417564}

Powered by Google App Engine
This is Rietveld 408576698