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

Unified Diff: webrtc/modules/rtp_rtcp/source/ulpfec_receiver_unittest.cc

Issue 2918333002: Reland of Only compare sequence numbers from the same SSRC in ForwardErrorCorrection. (Closed)
Patch Set: Let ForwardErrorCorrection be aware of its SSRCs. Created 3 years, 6 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/rtp_rtcp/source/ulpfec_receiver_unittest.cc
diff --git a/webrtc/modules/rtp_rtcp/source/ulpfec_receiver_unittest.cc b/webrtc/modules/rtp_rtcp/source/ulpfec_receiver_unittest.cc
index 6f9243429d0e55355c93cbc9de27a56918073ddf..7a9c446a22209917a18087589d34f3ddcc697072 100644
--- a/webrtc/modules/rtp_rtcp/source/ulpfec_receiver_unittest.cc
+++ b/webrtc/modules/rtp_rtcp/source/ulpfec_receiver_unittest.cc
@@ -48,8 +48,9 @@ class NullRecoveredPacketReceiver : public RecoveredPacketReceiver {
class UlpfecReceiverTest : public ::testing::Test {
protected:
UlpfecReceiverTest()
- : fec_(ForwardErrorCorrection::CreateUlpfec()),
- receiver_fec_(UlpfecReceiver::Create(&recovered_packet_receiver_)),
+ : fec_(ForwardErrorCorrection::CreateUlpfec(kMediaSsrc)),
+ receiver_fec_(
+ UlpfecReceiver::Create(kMediaSsrc, &recovered_packet_receiver_)),
packet_generator_(kMediaSsrc) {}
// Generates |num_fec_packets| FEC packets, given |media_packets|.
@@ -179,7 +180,7 @@ void UlpfecReceiverTest::SurvivesMaliciousPacket(const uint8_t* data,
NullRecoveredPacketReceiver null_callback;
std::unique_ptr<UlpfecReceiver> receiver_fec(
- UlpfecReceiver::Create(&null_callback));
+ UlpfecReceiver::Create(kMediaSsrc, &null_callback));
receiver_fec->AddReceivedRedPacket(header, data, length, ulpfec_payload_type);
}

Powered by Google App Engine
This is Rietveld 408576698