Adding reduced size RTCP configuration down to the video stream level.

talk/session/media/mediasession.cc

Index: talk/session/media/mediasession.cc
diff --git a/talk/session/media/mediasession.cc b/talk/session/media/mediasession.cc
index 2cd2d46d9ca5adbf28f61a83b7b501354a99be75..4ef846fcd7d4246e11c9e02308b4e3969a4fa675 100644
--- a/talk/session/media/mediasession.cc
+++ b/talk/session/media/mediasession.cc
@@ -726,6 +726,8 @@ static bool CreateMediaContentOffer(
     offer->set_crypto_required(CT_SDES);
   }
   offer->set_rtcp_mux(options.rtcp_mux_enabled);
+  // TODO(pthatcher): Once we support this, enable it in CreateOffer.
+  // offer->set_rtcp_reduced_size(true);

[pthatcher1, 2015/10/23 20:40:01]

If we fix VideoSendStream::SignalNetworkState, I think we can offer it.  But
someone will need to be on the hook for making sure that it works correctly.  So
it might make sense to submit this CL without it offering and then ask someone
else to enable it when they feel like supporting it (pbos might be the right
person for that).

[Taylor Brandstetter, 2015/11/11 19:42:40]

I'll go with the latter option and change this to TODO(pbos). When I uncomment
these lines, I see the "reduced" configuration propagated to the RTCPSender, but
I don't see any reduced size packets. So maybe there's still work to be done?

   offer->set_multistream(options.is_muc);
   offer->set_rtp_header_extensions(rtp_extensions);
 
@@ -1004,6 +1006,8 @@ static bool CreateMediaContentAnswer(
   answer->set_rtp_header_extensions(negotiated_rtp_extensions);
 
   answer->set_rtcp_mux(options.rtcp_mux_enabled && offer->rtcp_mux());
+  // TODO(pthatcher): Once we support this, enable it in CreateOffer.
+  // answer->set_rtcp_reduced_size(offer->rtcp_reduced_size());
 
   if (sdes_policy != SEC_DISABLED) {
     CryptoParams crypto;