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

Unified Diff: webrtc/modules/video_coding/rtp_frame_reference_finder.h

Issue 2304723004: Added ClearTo(seq_num) to RtpFrameReferenceFinder. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/video_coding/rtp_frame_reference_finder.h
diff --git a/webrtc/modules/video_coding/rtp_frame_reference_finder.h b/webrtc/modules/video_coding/rtp_frame_reference_finder.h
index a812303b7ea85eea4e1a02992322f22473743367..d48efc63eecdad4fe350f5ad926da83acc4ccb98 100644
--- a/webrtc/modules/video_coding/rtp_frame_reference_finder.h
+++ b/webrtc/modules/video_coding/rtp_frame_reference_finder.h
@@ -42,6 +42,7 @@ class RtpFrameReferenceFinder {
explicit RtpFrameReferenceFinder(OnCompleteFrameCallback* frame_callback);
brandtr 2016/09/05 11:20:32 Perhaps it would be good to comment the public int
philipel 2016/09/05 12:11:01 Done.
void ManageFrame(std::unique_ptr<RtpFrameObject> frame);
void PaddingReceived(uint16_t seq_num);
+ void ClearTo(uint16_t seq_num);
private:
static const uint16_t kPicIdLength = 1 << 7;
@@ -177,6 +178,11 @@ class RtpFrameReferenceFinder {
kMaxTemporalLayers>
missing_frames_for_layer_ GUARDED_BY(crit_);
+ // How far frames have been cleared by sequence number. A frame will be
+ // cleared if it contains a packet with a sequence number older than
+ // |cleared_to_seq_num_|.
+ int cleared_to_seq_num_ GUARDED_BY(crit_);
+
OnCompleteFrameCallback* frame_callback_;
};

Powered by Google App Engine
This is Rietveld 408576698