Rename {,Set}GenericFECStatus to {,Set}UlpfecConfig.

webrtc/video/video_send_stream.cc

 /*
  *  Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 #include "webrtc/video/video_send_stream.h"
@@ skipping 918 matching lines @@
   // that this is not the case with FLEXFEC.
   if (enable_protection_nack &&
       !PayloadTypeSupportsSkippingFecPackets(
           config_->encoder_settings.payload_name)) {
     LOG(LS_WARNING) << "Transmitting payload type without picture ID using"
                        "NACK+FEC is a waste of bandwidth since FEC packets "
                        "also have to be retransmitted. Disabling FEC.";
     enable_protection_fec = false;
   }
 
-  // Set to valid uint8_ts to be castable later without signed overflows.
-  uint8_t payload_type_red = 0;
-  uint8_t payload_type_fec = 0;
-
   // TODO(changbin): Should set RTX for RED mapping in RTP sender in future.
   // Validate payload types. If either RED or FEC payload types are set then
   // both should be. If FEC is enabled then they both have to be set.
   if (config_->rtp.ulpfec.red_payload_type != -1) {

[danilchap, 2016/10/26 14:18:08]

these two if block can be now reduced to
RTC_DCHECK_GE(...payload_type, -1);
RTC_DCHECK_LE(...payload_type, 127);
or may be just drop them (payload type dchecked internally anyway,
here it was necessary to ensure cast will not turn invalid value into valid.)

[brandtr, 2016/10/27 09:47:42]

Right. I will make some specific changes to this function in the next (or second
next) CL, so I won't make more changes right now.

     RTC_DCHECK_GE(config_->rtp.ulpfec.red_payload_type, 0);
     RTC_DCHECK_LE(config_->rtp.ulpfec.red_payload_type, 127);
     // TODO(holmer): We should only enable red if ulpfec is also enabled, but
     // but due to an incompatibility issue with previous versions the receiver
     // assumes rtx packets are containing red if it has been configured to
     // receive red. Remove this in a few versions once the incompatibility
     // issue is resolved (M53 timeframe).
-    payload_type_red =
-        static_cast<uint8_t>(config_->rtp.ulpfec.red_payload_type);
   }
   if (config_->rtp.ulpfec.ulpfec_payload_type != -1) {
     RTC_DCHECK_GE(config_->rtp.ulpfec.ulpfec_payload_type, 0);
     RTC_DCHECK_LE(config_->rtp.ulpfec.ulpfec_payload_type, 127);
-    payload_type_fec =
-        static_cast<uint8_t>(config_->rtp.ulpfec.ulpfec_payload_type);
   }
 
   for (RtpRtcp* rtp_rtcp : rtp_rtcp_modules_) {
     // Set NACK.
     rtp_rtcp->SetStorePacketsStatus(
         enable_protection_nack || congestion_controller_->pacer(),
         kMinSendSidePacketHistorySize);
     // Set FEC.
     for (RtpRtcp* rtp_rtcp : rtp_rtcp_modules_) {
-      rtp_rtcp->SetGenericFECStatus(enable_protection_fec, payload_type_red,
-                                    payload_type_fec);
+      rtp_rtcp->SetUlpfecConfig(enable_protection_fec,
+                                config_->rtp.ulpfec.red_payload_type,
+                                config_->rtp.ulpfec.ulpfec_payload_type);
     }
   }
 
   protection_bitrate_calculator_.SetProtectionMethod(enable_protection_fec,
                                                      enable_protection_nack);
 }
 
 void VideoSendStreamImpl::ConfigureSsrcs() {
   RTC_DCHECK_RUN_ON(worker_queue_);
   // Configure regular SSRCs.
@@ skipping 124 matching lines @@
                           &module_nack_rate);
     *sent_video_rate_bps += module_video_rate;
     *sent_nack_rate_bps += module_nack_rate;
     *sent_fec_rate_bps += module_fec_rate;
   }
   return 0;
 }
 
 }  // namespace internal
 }  // namespace webrtc