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

webrtc/modules/video_coding/packet.cc

Index: webrtc/modules/video_coding/packet.cc
diff --git a/webrtc/modules/video_coding/packet.cc b/webrtc/modules/video_coding/packet.cc
index e25de2ed6c6f4fb45361b0642f431b642131a438..e8c06a69771e86aaaa57aacc8e9237ba455927ec 100644
--- a/webrtc/modules/video_coding/packet.cc
+++ b/webrtc/modules/video_coding/packet.cc
@@ -24,6 +24,7 @@ VCMPacket::VCMPacket()
       dataPtr(NULL),
       sizeBytes(0),
       markerBit(false),
+      timesNacked(-1),

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

Why does a newly constructed (that has not been nacked yet) have timesNacked =
-1? Wouldn't zero be more natural?

[philipel, 2016/05/23 13:34:17]

-1 mean we don't know, so 0 can mean 0.

       frameType(kEmptyFrame),
       codec(kVideoCodecUnknown),
       isFirstPacket(false),
@@ -43,6 +44,7 @@ VCMPacket::VCMPacket(const uint8_t* ptr,
       dataPtr(ptr),
       sizeBytes(size),
       markerBit(rtpHeader.header.markerBit),
+      timesNacked(-1),
 
       frameType(rtpHeader.frameType),
       codec(kVideoCodecUnknown),
@@ -67,6 +69,7 @@ VCMPacket::VCMPacket(const uint8_t* ptr,
       dataPtr(ptr),
       sizeBytes(size),
       markerBit(mBit),
+      timesNacked(-1),
 
       frameType(kVideoFrameDelta),
       codec(kVideoCodecUnknown),
@@ -85,6 +88,7 @@ void VCMPacket::Reset() {
   dataPtr = NULL;
   sizeBytes = 0;
   markerBit = false;
+  timesNacked = -1;
   frameType = kEmptyFrame;
   codec = kVideoCodecUnknown;
   isFirstPacket = false;