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

Issue 2951693002: Revert of Roll WebRTC 18614:18649 (24 commits) (Closed)

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

Description

Revert of Roll WebRTC 18614:18649 (24 commits) (patchset #1 id:1 of https://codereview.chromium.org/2944823002/ ) Reason for revert: Increases DLL size on Windows, will roll in smaller steps. Original issue's description: > Roll WebRTC 18614:18649 (24 commits) > > Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/c81ec9b..e9cc1fe > > $ git log c81ec9b..e9cc1fe --date=short --no-merges --format=%ad %ae %s > 2017-06-18 charujain@webrtc.org Revert of Opus implementation of the AudioEncoderFactoryTemplate API (patchset #4 id:80001 of https://codereview.webrtc.org/2930243003/ ) > 2017-06-18 charujain@webrtc.org Revert of Opus implementation of the AudioDecoderFactoryTemplate API (patchset #1 id:1 of https://codereview.webrtc.org/2942733003/ ) > 2017-06-17 zhihuang@webrtc.org Support building WebRTC without audio and video for Android > 2017-06-17 kwiberg@webrtc.org Opus implementation of the AudioDecoderFactoryTemplate API > 2017-06-17 kwiberg@webrtc.org Opus implementation of the AudioEncoderFactoryTemplate API > 2017-06-17 kwiberg@webrtc.org G722 implementation of the AudioEncoderFactoryTemplate API > 2017-06-17 kwiberg@webrtc.org G722 implementation of the AudioDecoderFactoryTemplate API > 2017-06-17 kwiberg@webrtc.org Templated AudioDecoderFactory > 2017-06-16 deadbeef@webrtc.org Fixing incorrect use of erase/remove idiom. > 2017-06-16 emadomara@google.com Enable SNI in ssl adapter. > 2017-06-16 kwiberg@webrtc.org Templated AudioEncoderFactory > 2017-06-16 mellem@webrtc.org Create the VideoEncoderFactory and implement it. > 2017-06-16 stefan@webrtc.org Tune loss-based BWE to be more compatible with the low frequency loss reports of audio streams. > 2017-06-16 eladalon@webrtc.org Style fixes in rtcp_packet/ > 2017-06-16 ilnik@webrtc.org Add cropping to VIEEncoder to match simulcast streams resolution > 2017-06-16 terelius@webrtc.org Add has_value() and value() methods to rtc::Optional. > 2017-06-16 henrika@webrtc.org Reduces sensitivity in audio-glitch detector for iOS > 2017-06-16 erikvarga@webrtc.org Use RaceChecker instead of ThreadChecker in a few places. > 2017-06-16 eladalon@webrtc.org Remove unused #include "libyuv/compare.h" > 2017-06-15 andersc@webrtc.org Move setting switches in AppRTCMobile to Settings screen > 2017-06-16 sakal@webrtc.org Create AndroidVideoBuffer and allow renderers to consume it. > 2017-06-16 nisse@webrtc.org Delete SignalSrtpError. > 2017-06-15 glaznev@webrtc.org Support H.264 high profile encoding on Exynos devices. > 2017-06-15 zhihuang@webrtc.org Support building WebRTC without audio and video. > > TBR= > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng > BUG= > > Review-Url: https://codereview.chromium.org/2944823002 > Cr-Commit-Position: refs/heads/master@{#480386} > Committed: https://chromium.googlesource.com/chromium/src/+/ee62b31dfc9c325ce85d3eacca0a4013ba0afc3c # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=734631 Review-Url: https://codereview.chromium.org/2951693002 Cr-Commit-Position: refs/heads/master@{#480768} Committed: https://chromium.googlesource.com/chromium/src/+/c10209de836c85f0cbfda609d072ca94ceac8f09

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: 10 (5 generated)
sakal-chromium
Created Revert of Roll WebRTC 18614:18649 (24 commits)
3 years, 6 months ago (2017-06-20 08:11:19 UTC) #1
sakal-chromium
PTAL
3 years, 6 months ago (2017-06-20 08:12:11 UTC) #4
magjed_chromium
lgtm
3 years, 6 months ago (2017-06-20 08:13:18 UTC) #6
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/2951693002/1
3 years, 6 months ago (2017-06-20 08:13:28 UTC) #7
commit-bot: I haz the power
3 years, 6 months ago (2017-06-20 08:13:56 UTC) #10
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c10209de836c85f0cbfda609d072...

Powered by Google App Engine
This is Rietveld 408576698