Experiment for the nack module.

webrtc/modules/video_coding/nack_module.h

 /*
  *  Copyright (c) 2016 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 17 matching lines @@
 
 class NackModule : public Module {
  public:
   NackModule(Clock* clock,
              NackSender* nack_sender,
              KeyFrameRequestSender* keyframe_request_sender);
 
   void OnReceivedPacket(const VCMPacket& packet);
   void ClearUpTo(uint16_t seq_num);
   void UpdateRtt(int64_t rtt_ms);
+  void Flush();

[stefan-webrtc, 2016/03/10 17:06:30]

Can we call this Clear() instead? Flush is a bad name in my opinion since it
sounds like "send all buffered nacks" (e.g., fflush()).

[philipel, 2016/03/11 09:24:14]

Done.

   void Stop();
 
   // Module implementation
   int64_t TimeUntilNextProcess() override;
   void Process() override;
 
  private:
   // Which fields to consider when deciding which packet to nack in
   // GetNackBatch.
   enum NackFilterOptions { kSeqNumOnly, kTimeOnly, kSeqNumAndTime };
@@ skipping 44 matching lines @@
   bool running_ GUARDED_BY(crit_);
   bool initialized_ GUARDED_BY(crit_);
   int64_t rtt_ms_ GUARDED_BY(crit_);
   uint16_t last_seq_num_ GUARDED_BY(crit_);
   int64_t next_process_time_ms_ GUARDED_BY(crit_);
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_VIDEO_CODING_NACK_MODULE_H_