Experiment for the nack module.

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.
  */
 
 #ifndef WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_H_
 #define WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_H_
 
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "webrtc/modules/include/module.h"
 #include "webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h"
+#include "webrtc/modules/video_coding/include/video_coding_defines.h"
 
 namespace webrtc {
 // Forward declarations.
 class ReceiveStatistics;
 class RemoteBitrateEstimator;
 class RtpReceiver;
 class Transport;
 class RtcEventLog;
 
 RTPExtensionType StringToRtpExtensionType(const std::string& extension);
 
 namespace rtcp {
 class TransportFeedback;
 }
 
-class RtpRtcp : public Module {
+class RtpRtcp : public Module, public NackSender {
  public:
   struct Configuration {
     Configuration();
 
    /*  id                   - Unique identifier of this RTP/RTCP module object
     *  audio                - True for a audio version of the RTP/RTCP module
     *                         object false will create a video version
     *  clock                - The clock to use to read time. If NULL object
     *                         will be using the system clock.
     *  incoming_data        - Callback object that will receive the incoming
@@ skipping 480 matching lines @@
      *
      *  Returns -1 on failure, otherwise 0.
      */
     virtual int SetSelectiveRetransmissions(uint8_t settings) = 0;
 
     /*
     *   Send a Negative acknowledgement packet
     *
     *   return -1 on failure else 0
     */
+    // TODO(philipel): Deprecate this and start using SendNack instead,
+    //                 mostly because we want a function that actually send
+    //                 NACK for the specified packets.
     virtual int32_t SendNACK(const uint16_t* nackList, uint16_t size) = 0;
 
     /*
+    *   Implements NackSender
+    *   Send NACK for the packets specified.
+    */
+    virtual void SendNack(const std::vector<uint16_t>& sequence_numbers) = 0;
+
+    /*
     *   Store the sent packets, needed to answer to a Negative acknowledgement
     *   requests
     */
     virtual void SetStorePacketsStatus(bool enable, uint16_t numberToStore) = 0;
 
     // Returns true if the module is configured to store packets.
     virtual bool StorePackets() const = 0;
 
     // Called on receipt of RTCP report block from remote side.
     virtual void RegisterRtcpStatisticsCallback(
@@ skipping 101 matching lines @@
 
     /*
     *   send a request for a keyframe
     *
     *   return -1 on failure else 0
     */
     virtual int32_t RequestKeyFrame() = 0;
 };
 }  // namespace webrtc
 #endif  // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_H_