Simplify SetFecParameters signature.

webrtc/modules/video_coding/include/video_coding_defines.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 96 matching lines @@
                                int render_delay_ms) = 0;
 
  protected:
   virtual ~VCMDecoderTimingCallback() {}
 };
 
 // Callback class used for telling the user about how to configure the FEC,
 // and the rates sent the last second is returned to the VCM.
 class VCMProtectionCallback {
  public:
-  virtual int ProtectionRequest(const FecProtectionParams* delta_params,
-                                const FecProtectionParams* key_params,
-                                uint32_t* sent_video_rate_bps,
-                                uint32_t* sent_nack_rate_bps,
-                                uint32_t* sent_fec_rate_bps) = 0;
+  virtual void ProtectionRequest(const FecProtectionParams& delta_params,

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

changing this interface will break dependent projects.
change it in backward compatible way (may be in a dedicated CL)

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

I see. I'll leave it unchanged for now.

+                                 const FecProtectionParams& key_params,
+                                 uint32_t* sent_video_rate_bps,
+                                 uint32_t* sent_nack_rate_bps,
+                                 uint32_t* sent_fec_rate_bps) = 0;
 
  protected:
   virtual ~VCMProtectionCallback() {}
 };
 
 // Callback class used for telling the user about what frame type needed to
 // continue decoding.
 // Typically a key frame when the stream has been corrupted in some way.
 class VCMFrameTypeCallback {
  public:
@@ skipping 32 matching lines @@
  public:
   virtual void RequestKeyFrame() = 0;
 
  protected:
   virtual ~KeyFrameRequestSender() {}
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_VIDEO_CODING_INCLUDE_VIDEO_CODING_DEFINES_H_