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

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

Issue 2199133004: PacketBuffer is now ref counted. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: PacketBuffer/RtpFrameReferenceFinder can now be stopped. Created 4 years, 4 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.cc
diff --git a/webrtc/modules/video_coding/rtp_frame_reference_finder.cc b/webrtc/modules/video_coding/rtp_frame_reference_finder.cc
index 998d757a7bc47089a9d52a32b4d38cbf474993ab..933086b340353174e04b069a35fc6a994e837392 100644
--- a/webrtc/modules/video_coding/rtp_frame_reference_finder.cc
+++ b/webrtc/modules/video_coding/rtp_frame_reference_finder.cc
@@ -61,6 +61,12 @@ void RtpFrameReferenceFinder::PaddingReceived(uint16_t seq_num) {
RetryStashedFrames();
}
+void RtpFrameReferenceFinder::Stop() {
+ rtc::CritScope lock(&crit_);
+ while (!stashed_frames_.empty())
danilchap 2016/08/02 15:29:34 stashed_frames_.clear() instead? or order of destr
+ stashed_frames_.pop();
+}
+
void RtpFrameReferenceFinder::UpdateLastPictureIdWithPadding(uint16_t seq_num) {
auto gop_seq_num_it = last_seq_num_gop_.upper_bound(seq_num);

Powered by Google App Engine
This is Rietveld 408576698