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

Unified Diff: webrtc/call/flexfec_receive_stream_unittest.cc

Issue 2499963002: Make configuration logic harsher in FlexfecReceiveStream. (Closed)
Patch Set: Feedback response 1. Created 4 years, 1 month 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/call/flexfec_receive_stream_unittest.cc
diff --git a/webrtc/call/flexfec_receive_stream_unittest.cc b/webrtc/call/flexfec_receive_stream_unittest.cc
index a7a3158ee555e9ab00019216dd5f2add83b3137b..a41b2a54e8a30ff12303bd8aa9e4d77967ae9ef0 100644
--- a/webrtc/call/flexfec_receive_stream_unittest.cc
+++ b/webrtc/call/flexfec_receive_stream_unittest.cc
@@ -54,20 +54,6 @@ TEST(FlexfecReceiveStreamTest, DoesNotProcessPacketWhenNoMediaSsrcGiven) {
receive_stream.AddAndProcessReceivedPacket(packet, packet_length));
}
-// TODO(brandtr): Remove when we support multistream protection.
-TEST(FlexfecReceiveStreamTest, CannotProtectMultipleMediaStreams) {
- FlexfecReceiveStream::Config config;
- config.flexfec_payload_type = 118;
- config.flexfec_ssrc = 424223;
- config.protected_media_ssrcs = {123, 456};
- MockRecoveredPacketReceiver callback;
- internal::FlexfecReceiveStream receive_stream(config, &callback);
-
- ASSERT_EQ(1U, receive_stream.config().protected_media_ssrcs.size());
- EXPECT_EQ(config.protected_media_ssrcs[0],
- receive_stream.config().protected_media_ssrcs[0]);
-}
-
// Create a FlexFEC packet that protects a single media packet and ensure
// that the callback is called. Correctness of recovery is checked in the
// FlexfecReceiver unit tests.
« webrtc/call/flexfec_receive_stream.cc ('K') | « webrtc/call/flexfec_receive_stream.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698