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

Issue 2992123002: Merge WebRTC r18998 and r19018 to M60 branch (Closed)

Created:
3 years, 4 months ago by sprang_webrtc
Modified:
3 years, 4 months ago
Reviewers:
philipel
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, zhengzhonghou_agora.io, video-team_agora.io, stefan-webrtc, mflodman
Target Ref:
refs/branch-heads/60
Project:
webrtc
Visibility:
Public.

Description

Merge WebRTC r18998 and r19018 to M60 branch This CL is cherry pick of the following two commits: 1) RtpFrameReferenceFinder VP8 reference calculation fix. Review-Url: https://codereview.webrtc.org/2980943003 Cr-Commit-Position: refs/heads/master@{#19018} 2) Fix potential incorrect sync flags used with screenshare TL stream. Review-Url: https://codereview.webrtc.org/2978743002 Cr-Commit-Position: refs/heads/master@{#18999} TEST=webrtc unit tests plus manual verification. BUG=chromium:748417, webrtc:7768, webrtc:7980 R=philipel@webrtc.org Review-Url: https://codereview.webrtc.org/2992123002 . Cr-Commit-Position: refs/branch-heads/60@{#11} Cr-Branched-From: c61bf947b4ac31f3500858ffcae6fee39d799930-refs/heads/master@{#18252} Committed: https://chromium.googlesource.com/external/webrtc/+/6294a7eb71c891e9ea41273a7a94113f6802d0da

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+97 lines, -16 lines) Patch
M webrtc/modules/video_coding/codecs/vp8/default_temporal_layers.cc View 2 chunks +5 lines, -2 lines 0 comments Download
M webrtc/modules/video_coding/codecs/vp8/screenshare_layers.cc View 1 chunk +10 lines, -4 lines 0 comments Download
M webrtc/modules/video_coding/codecs/vp8/screenshare_layers_unittest.cc View 2 chunks +44 lines, -6 lines 0 comments Download
M webrtc/modules/video_coding/codecs/vp8/temporal_layers.h View 1 chunk +10 lines, -0 lines 0 comments Download
M webrtc/modules/video_coding/rtp_frame_reference_finder.cc View 1 chunk +3 lines, -4 lines 0 comments Download
M webrtc/modules/video_coding/rtp_frame_reference_finder_unittest.cc View 1 chunk +25 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
sprang_webrtc
3 years, 4 months ago (2017-08-01 08:17:44 UTC) #2
philipel
lgtm
3 years, 4 months ago (2017-08-01 08:19:17 UTC) #3
sprang_webrtc
3 years, 4 months ago (2017-08-01 09:33:45 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
6294a7eb71c891e9ea41273a7a94113f6802d0da.

Powered by Google App Engine
This is Rietveld 408576698