Added ClearTo(seq_num) to RtpFrameReferenceFinder.

webrtc/modules/video_coding/rtp_frame_reference_finder.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 21 matching lines @@
 // A complete frame is a frame which has received all its packets and all its
 // references are known.
 class OnCompleteFrameCallback {
  public:
   virtual ~OnCompleteFrameCallback() {}
   virtual void OnCompleteFrame(std::unique_ptr<FrameObject> frame) = 0;
 };
 
 class RtpFrameReferenceFinder {
  public:
   explicit RtpFrameReferenceFinder(OnCompleteFrameCallback* frame_callback);

[brandtr, 2016/09/05 11:20:32]

Perhaps it would be good to comment the public interface of this class as well.
(The private interface is nicely commented.)

[philipel, 2016/09/05 12:11:01]

Done.

   void ManageFrame(std::unique_ptr<RtpFrameObject> frame);
   void PaddingReceived(uint16_t seq_num);
+  void ClearTo(uint16_t seq_num);
 
  private:
   static const uint16_t kPicIdLength = 1 << 7;
   static const uint8_t kMaxTemporalLayers = 5;
   static const int kMaxLayerInfo = 10;
   static const int kMaxStashedFrames = 10;
   static const int kMaxNotYetReceivedFrames = 20;
   static const int kMaxGofSaved = 15;
   static const int kMaxPaddingAge = 100;
 
@@ skipping 115 matching lines @@
   // Keep track of which picture id and which temporal layer that had the
   // up switch flag set.
   std::map<uint16_t, uint8_t, DescendingSeqNumComp<uint16_t, kPicIdLength>>
       up_switch_ GUARDED_BY(crit_);
 
   // For every temporal layer, keep a set of which frames that are missing.
   std::array<std::set<uint16_t, DescendingSeqNumComp<uint16_t, kPicIdLength>>,
              kMaxTemporalLayers>
       missing_frames_for_layer_ GUARDED_BY(crit_);
 
+  // How far frames have been cleared by sequence number. A frame will be
+  // cleared if it contains a packet with a sequence number older than
+  // |cleared_to_seq_num_|.
+  int cleared_to_seq_num_ GUARDED_BY(crit_);
+
   OnCompleteFrameCallback* frame_callback_;
 };
 
 }  // namespace video_coding
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_VIDEO_CODING_RTP_FRAME_REFERENCE_FINDER_H_