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

Issue 2899713002: Break backwards traversal loop if we have looped around all packets in the PacketBuffer for H264 fr… (Closed)

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

Description

Break backwards traversal loop if we have looped around all packets in the PacketBuffer for H264 frames. BUG=webrtc:7532, chromium:724031 R=holmer@google.com Review-Url: https://codereview.webrtc.org/2868723003 Cr-Original-Commit-Position: refs/heads/master@{#18191} Review-Url: https://codereview.webrtc.org/2899713002 . Cr-Commit-Position: refs/branch-heads/59@{#12} Cr-Branched-From: 10d095d4f743bc16f8e486e156c48a6d023b32c5-refs/heads/master@{#17657} Committed: https://chromium.googlesource.com/external/webrtc/+/2996dc5a520cbe2051d401dfce5ee4800162007f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -10 lines) Patch
M webrtc/modules/video_coding/packet_buffer.cc View 4 chunks +6 lines, -10 lines 0 comments Download
M webrtc/modules/video_coding/video_packet_buffer_unittest.cc View 1 chunk +24 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (4 generated)
philipel
This is the merge to M59, PTAL
3 years, 7 months ago (2017-05-22 11:43:25 UTC) #2
holmer
lgtm
3 years, 7 months ago (2017-05-22 11:52:31 UTC) #4
holmer
On 2017/05/22 11:52:31, holmer wrote: > lgtm Please also add BUG=chromium:724031
3 years, 7 months ago (2017-05-22 11:53:22 UTC) #5
philipel
3 years, 7 months ago (2017-05-22 12:13:27 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
2996dc5a520cbe2051d401dfce5ee4800162007f.

Powered by Google App Engine
This is Rietveld 408576698