Simplify {,Set}UlpfecConfig interface.

webrtc/modules/rtp_rtcp/include/rtp_rtcp.h

 /*
  *  Copyright (c) 2012 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.
  */
 
@@ skipping 435 matching lines @@
   // indication.
   // This API shall be called before transmision of an RTP packet to ensure
   // that the |level| part of the extended RTP header is updated.
   // return -1 on failure else 0.
   virtual int32_t SetAudioLevel(uint8_t level_dbov) = 0;
 
   // **************************************************************************
   // Video
   // **************************************************************************
 
-  // Turn on/off ULPFEC.
-  virtual void SetUlpfecConfig(bool enabled,
-                               int red_payload_type,
+  // Set RED and ULPFEC payload types. A payload type of -1

[danilchap, 2016/10/27 11:53:40]

which of the two payload types should be -1 to turn ulpfec off?

[brandtr, 2016/10/28 07:46:31]

ulpfec_payload_type == -1 means that ULPFEC is off and red_payload_type == -1
means that RED is off. I have expanded the comment to clarify this.

In a consistent system, these should always be enabled/disabled together.
However, due to a workaround involving RTX (see
https://codereview.webrtc.org/1649493004), for now we will always send RED if it
has been negotiated in the SDP, regardless if we are using ULPFEC or not. I will
most likely remove this workaround, but for now we need the ability to turn
these off individually.

+  // means that the functionality is turned off.
+  virtual void SetUlpfecConfig(int red_payload_type,
                                int ulpfec_payload_type) = 0;
 
   virtual int32_t SetFecParameters(const FecProtectionParams* delta_params,
                                    const FecProtectionParams* key_params) = 0;
 
   // Set method for requestion a new key frame.
   // Returns -1 on failure else 0.
   virtual int32_t SetKeyFrameRequestMethod(KeyFrameRequestMethod method) = 0;
 
   // Sends a request for a keyframe.
   // Returns -1 on failure else 0.
   virtual int32_t RequestKeyFrame() = 0;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_H_