PacketBuffer now can save how many times a packet has been nacked.

webrtc/modules/video_coding/frame_object.h

Index: webrtc/modules/video_coding/frame_object.h
diff --git a/webrtc/modules/video_coding/frame_object.h b/webrtc/modules/video_coding/frame_object.h
index 80ab0ec4ff7daa234328f5c8b4d8997bfa3e7ba6..57ec7dfe2656829d748c0f8eeaa3d71394732c62 100644
--- a/webrtc/modules/video_coding/frame_object.h
+++ b/webrtc/modules/video_coding/frame_object.h
@@ -35,6 +35,8 @@ class FrameObject {
   size_t num_references;
   uint16_t references[kMaxFrameReferences];
   bool inter_layer_predicted;
+
+  size_t size;
 };
 
 class PacketBuffer;
@@ -43,11 +45,14 @@ class RtpFrameObject : public FrameObject {
  public:
   RtpFrameObject(PacketBuffer* packet_buffer,
                  uint16_t first_seq_num,
-                 uint16_t last_seq_num);
+                 uint16_t last_seq_num,
+                 size_t combined_size,
+                 int8_t max_nack_count);
 
   ~RtpFrameObject();
   uint16_t first_seq_num() const;
   uint16_t last_seq_num() const;
+  int8_t max_nack_count() const;
   FrameType frame_type() const;
   VideoCodecType codec_type() const;
   bool GetBitstream(uint8_t* destination) const override;
@@ -59,6 +64,7 @@ class RtpFrameObject : public FrameObject {
   VideoCodecType codec_type_;
   uint16_t first_seq_num_;
   uint16_t last_seq_num_;
+  int8_t max_nack_count_;

[stefan-webrtc, 2016/05/20 13:10:23]

Why int8_t? int would be nicer

[philipel, 2016/05/23 09:19:22]

Changed to int.

The reason I use int8_t here is because a FrameObject is something we might
potentially have many instances of and therefore size might make a difference.

[terelius, 2016/05/23 11:49:55]

If you keep this as an int8_t and you are going to (semi-implicitly) cast the
ints to int8_t, I think would be better to have int8_t everywhere.

 };
 
 }  // namespace video_coding