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

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

Issue 2613833003: Don't detect a new frame if a previous packet is used in a previous frame. (Closed)
Patch Set: Created 3 years, 11 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/video_packet_buffer_unittest.cc
diff --git a/webrtc/modules/video_coding/video_packet_buffer_unittest.cc b/webrtc/modules/video_coding/video_packet_buffer_unittest.cc
index 1e2ef3ec2aca42965bd53b4f335c091b62f396ca..6cc496a9196889d95927babd3a33d02367295acd 100644
--- a/webrtc/modules/video_coding/video_packet_buffer_unittest.cc
+++ b/webrtc/modules/video_coding/video_packet_buffer_unittest.cc
@@ -460,5 +460,14 @@ TEST_F(TestPacketBuffer, DontLeakPayloadData) {
EXPECT_FALSE(Insert(2 + kMaxSize, kKeyFrame, kFirst, kNotLast, 5, data4));
}
+TEST_F(TestPacketBuffer, ContinuousSeqNumNoMarkerBit) {
+ Insert(2, kKeyFrame, kNotFirst, kNotLast);
+ Insert(1, kKeyFrame, kFirst, kLast);
+ frames_from_callback_.clear();
+ Insert(3, kKeyFrame, kNotFirst, kLast);
stefan-webrtc 2017/01/10 11:50:00 Doesn't this mean you have received the marker bit
philipel 2017/01/10 12:03:36 Renamed it to ContinuousSeqNumDoubleMarkerBit.
+
+ ASSERT_EQ(0UL, frames_from_callback_.size());
stefan-webrtc 2017/01/10 11:50:00 EXPECT_EQ
philipel 2017/01/10 12:03:36 Done.
+}
+
} // namespace video_coding
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698