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

Issue 2449483002: Roll WebRTC 14722:14746 (23 commits) (Closed)

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

Description

Roll WebRTC 14722:14746 (23 commits) Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/5a1b3fe..7938471 $ git log 5a1b3fe..7938471 --date=short --no-merges --format=%ad %ae %s 2016-10-24 ehmaldonado@google.com Roll chromium_revision 1362287708..9b5bb47fa0 (426760:426837) + roll Android SDK to N 2016-10-24 philipel@webrtc.org Drop VP8 frames older than the last sync frame in the RtpFrameReferenceFinder. 2016-10-24 aleloi@webrtc.org Elimiteted race condition in the AudioMixer. 2016-10-24 terelius@webrtc.org Revert of Fix some chromium style warnings in remote_bitrate_estimator.h (patchset #1 id:1 of https://codereview.webrtc.org/2387113008/ ) 2016-10-24 danilchap@webrtc.org Add tests and fix thread annotations 2016-10-24 kwiberg@webrtc.org Eliminate left shift of negative value by using multiplication instead 2016-10-24 hbos@webrtc.org RTCTransportStats[1] added, supporting all members. 2016-10-24 terelius@webrtc.org Remove unused variable from delay based BWE. 2016-10-24 terelius@webrtc.org Fix chromium-style warnings. 2016-10-24 terelius@webrtc.org Fix some chromium style warnings in remote_bitrate_estimator.h 2016-10-24 kjellander@webrtc.org GN: move webrtc/video/ targets from webrtc/BUILD.gn into webrtc/video/BUILD.gn 2016-10-24 sakal@webrtc.org Implement framesEncoded stat in video send ssrc stats. 2016-10-24 perkj@webrtc.org Remove cricket::VideoCodec with, height and framerate properties Since WebRtcVideoSendStream have reconfigures the send codec to match the incoming captured frames widht and height they have not been used. Framerate has just been set when parsing sdp to 60fps and not changed elsewhere. 2016-10-24 nisse@webrtc.org Delete always-zero ByteBufferWriter::start_. 2016-10-24 nisse@webrtc.org Reland of Delete webrtc::VideoFrame::CopyFrame. (patchset #1 id:1 of https://codereview.webrtc.org/2397943003/ ) 2016-10-24 sakal@webrtc.org Allow using Java classes that don't require JNI in Chromium. 2016-10-24 nisse@webrtc.org Revert of Add method cricket::VideoCapturer::NeedsDenoising, use in VideoCapturerTrackSource. (patchset #5 id:80001 of https://codereview.webrtc.org/2334683002/ ) 2016-10-24 nisse@webrtc.org Delete unused class AudioSourceWithMixStatus. 2016-10-23 brandtr@webrtc.org Integrate FlexfecReceiveStream with Call. 2016-10-22 peah@webrtc.org Several subcomponents inside APM copy render audio from the render side to the capture side using the same pattern. Currently this is done independently for the submodules. 2016-10-22 peah@webrtc.org Added the missing ReadQueuedRenderData() call to the AECM bitexactness test 2016-10-22 peah@webrtc.org Added the missing ReadQueuedRenderData() call to the gain controller bitexactness test 2016-10-22 peah@webrtc.org Added the missing ReadQueuedRenderData() call to the AEC bitexactness test 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/7bda5b2db9ce07f1b2d62f576f7517fe1885fb21 Cr-Commit-Position: refs/heads/master@{#427301}

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: 15 (9 generated)
sakal-chromium
PTAL
4 years, 1 month ago (2016-10-24 14:56:04 UTC) #3
kjellander_chromium
lgtm, although notice that revision hasn't yet been fully built and tested in https://build.chromium.org/p/chromium.webrtc.fyi/console
4 years, 1 month ago (2016-10-24 17:03:22 UTC) #6
kjellander_chromium
On 2016/10/24 17:03:22, kjellander_chromium wrote: > lgtm, although notice that revision hasn't yet been fully ...
4 years, 1 month ago (2016-10-24 19:23:20 UTC) #9
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/2449483002/1
4 years, 1 month ago (2016-10-25 07:40:41 UTC) #11
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-10-25 08:38:01 UTC) #13
commit-bot: I haz the power
4 years, 1 month ago (2016-10-25 08:40:31 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/7bda5b2db9ce07f1b2d62f576f7517fe1885fb21
Cr-Commit-Position: refs/heads/master@{#427301}

Powered by Google App Engine
This is Rietveld 408576698