Make |rtcp_send_transport| mandatory in FlexfecReceiveStream::Config.

webrtc/video/end_to_end_tests.cc

Index: webrtc/video/end_to_end_tests.cc
diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc
index b7ae199de0651dab7ed8a8f9c0ecb36305f5e0f8..e20803716d32267c03b84694f7cb7238bf3cc21b 100644
--- a/webrtc/video/end_to_end_tests.cc
+++ b/webrtc/video/end_to_end_tests.cc
@@ -3888,7 +3888,7 @@ TEST_P(EndToEndTest, VerifyDefaultVideoReceiveConfigParameters) {
 }
 
 TEST_P(EndToEndTest, VerifyDefaultFlexfecReceiveConfigParameters) {
-  FlexfecReceiveStream::Config default_receive_config;
+  FlexfecReceiveStream::Config default_receive_config(nullptr);

[stefan-webrtc, 2016/12/22 09:14:54]

I guess it's questionable if it should be OK to pass nullptr as the transport if
you've made it required? Maybe you should DCHECK for it in the constructor of
the config?

[brandtr, 2017/01/03 10:01:05]

Done.

   EXPECT_EQ(-1, default_receive_config.payload_type)
       << "Enabling FlexFEC requires rtpmap: flexfec negotiation.";
   EXPECT_EQ(0U, default_receive_config.remote_ssrc)