zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Stash non layer-sync frames if we have not yet received an earlier frame for this layer.

Browse Source 
In VP8 the assumption is that a non layer-sync frame depends on all previous
frames on temporal layers less or equal to this frames temporal layer.

BUG=webrtc:5514
R=stefan@webrtc.org

Review URL: https://codereview.webrtc.org/2306513003 .

Cr-Commit-Position: refs/heads/master@{#14229}
This commit is contained in:
philipel
2016-09-15 13:43:13 +02:00
parent ebc34e78db
commit d268d6ffbe
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
webrtc/modules/video_coding/rtp_frame_reference_finder.cc
View File

@ -291,7 +291,12 @@ void RtpFrameReferenceFinder::ManageFrameVp8(
// Find all references for this frame. // Find all references for this frame.
frame->num_references = 0; frame->num_references = 0;
for (uint8_t layer = 0; layer <= codec_header.temporalIdx; ++layer) { for (uint8_t layer = 0; layer <= codec_header.temporalIdx; ++layer) {
RTC_DCHECK_NE(-1, layer_info_it->second[layer]); // If we have not yet received a previous frame on this temporal layer,
// stash this frame.
if (layer_info_it->second[layer] == -1) {
stashed_frames_.push_back(std::move(frame));
return;
}
// If we have not yet received a frame between this frame and the referenced // If we have not yet received a frame between this frame and the referenced
// frame then we have to wait for that frame to be completed first. // frame then we have to wait for that frame to be completed first.