Refactor NACK bitrate allocation

webrtc/modules/rtp_rtcp/include/rtp_rtcp_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 369 matching lines @@
   // while we are paused.
 
   // Returns true if we send the packet now, else it will add the packet
   // information to the queue and call TimeToSendPacket when it's time to send.
   virtual void InsertPacket(Priority priority,
                             uint32_t ssrc,
                             uint16_t sequence_number,
                             int64_t capture_time_ms,
                             size_t bytes,
                             bool retransmission) = 0;
+
+  // Try to allocate bitrate for sending a retransmission. This may fail if too
+  // much retransmission is requested in relation to payload.
+  virtual bool AllocateRetransmissionBitrate(size_t bytes) = 0;
+
+  virtual int CurrentRetransmissionBitrate() = 0;

[danilchap, 2016/06/23 12:46:12]

maybe call it CurrentRetransmisisonBitrateBps to put units directly into
function name instead of comment in a derived class.

[sprang_webrtc, 2016/06/28 09:12:33]

Done. And removed. :)

 };
 
 class TransportSequenceNumberAllocator {
  public:
   TransportSequenceNumberAllocator() {}
   virtual ~TransportSequenceNumberAllocator() {}
 
   virtual uint16_t AllocateSequenceNumber() = 0;
 };
 
 }  // namespace webrtc
 #endif  // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_DEFINES_H_