Change how "all frames missing" is encoded.

(Used to be encoded as an empty set, now uses kRtcpCastAllPacketsLost)

Add kRtcpCastLastPacket to let the sender re-transmit the last packet of a frame. (Sender doesn't actually know how many packets there are in a frame.)

Fix a bug in retransmit cancellation.

Cancel retransmits for a frame when it is ACKed.

Review URL: https://codereview.chromium.org/325263004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277580 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 0 deletions

diff --git a/media/cast/cast_defines.h b/media/cast/cast_defines.h
index bf6df39..64b20c9 100644
--- a/media/cast/cast_defines.h
+++ b/media/cast/cast_defines.h
@@ -68,14 +68,22 @@ enum PacketType {
   kTooOldPacket,
 };
 
+// kRtcpCastAllPacketsLost is used in PacketIDSet and
+// on the wire to mean that ALL packets for a particular
+// frame are lost.
 const uint16 kRtcpCastAllPacketsLost = 0xffff;
 
+// kRtcpCastLastPacket is used in PacketIDSet to ask for
+// the last packet of a frame to be retransmitted.
+const uint16 kRtcpCastLastPacket = 0xfffe;
+
 const size_t kMinLengthOfRtcp = 8;
 
 // Basic RTP header + cast header.
 const size_t kMinLengthOfRtp = 12 + 6;
 
 // Each uint16 represents one packet id within a cast frame.
+// Can also contain kRtcpCastAllPacketsLost and kRtcpCastLastPacket.
 typedef std::set<uint16> PacketIdSet;
 // Each uint8 represents one cast frame.
 typedef std::map<uint8, PacketIdSet> MissingFramesAndPacketsMap;