Simplify SetFecParameters signature.

webrtc/modules/rtp_rtcp/include/rtp_rtcp.h

Index: webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
diff --git a/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h b/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
index 4dda74d3432b21b751fe497aae44f08e94955cb5..95cf65bb7c85d7bcbf640ebddf017a43e09ef65f 100644
--- a/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
+++ b/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
@@ -459,8 +459,10 @@ class RtpRtcp : public Module {
   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 FEC rates, max frames before FEC is sent, and type of FEC masks.
+  // Returns false on failure.
+  virtual bool SetFecParameters(const FecProtectionParams& delta_params,

[danilchap, 2016/10/28 10:16:41]

to avoid breaking dependant projects, keep the old signature too for a while:
RTC_DEPRECATED
inline int32_t SetFecParameters(* delta, *key) {
  return SetFecParameters(*delta, *key) ? 0 : -1;
}

[brandtr, 2016/10/28 11:06:10]

Done.

Added implementation in rtp_rtcp_impl.cc, to avoid having to include checks.h in
this file.

+                                const FecProtectionParams& key_params) = 0;
 
   // Set method for requestion a new key frame.
   // Returns -1 on failure else 0.