Cast: Fix two video freezing problems

media/cast/transport/rtp_sender/packet_storage/packet_storage.h

Index: media/cast/transport/rtp_sender/packet_storage/packet_storage.h
diff --git a/media/cast/transport/rtp_sender/packet_storage/packet_storage.h b/media/cast/transport/rtp_sender/packet_storage/packet_storage.h
index b5981e858463e34a22dc40e971b250bdb6020268..096186a680e35ddbf0b9edc36fc4aef764382756 100644
--- a/media/cast/transport/rtp_sender/packet_storage/packet_storage.h
+++ b/media/cast/transport/rtp_sender/packet_storage/packet_storage.h
@@ -23,14 +23,14 @@ namespace cast {
 namespace transport {
 
 class StoredPacket;
-typedef std::map<uint32, std::pair<PacketKey, PacketRef> > PacketMap;
-typedef std::multimap<base::TimeTicks, uint32> TimeToPacketMap;
+
+// StorageIndex contains {frame_id, packet_id}.
+typedef std::pair<uint32, uint16> StorageIndex;
+typedef std::map<StorageIndex, std::pair<PacketKey, PacketRef> > PacketMap;
 
 class PacketStorage {
  public:
-  static const unsigned int kMaxStoredPackets = 1000;
-
-  PacketStorage(base::TickClock* clock, int max_time_stored_ms);
+  PacketStorage(int stored_frames);
   virtual ~PacketStorage();
 
   void StorePacket(uint32 frame_id,
@@ -44,15 +44,18 @@ class PacketStorage {
       SendPacketVector* packets_to_resend);
 
   // Copies packet into the packet list.
-  bool GetPacket(uint8 frame_id, uint16 packet_id, SendPacketVector* packets);
-
+  bool GetPacket(uint8 frame_id_8bit,
+                 uint16 packet_id,
+                 SendPacketVector* packets);
+  // Same as GetPacket, but takes a 32-bit frame id.

[Alpha Left Google, 2014/04/29 19:46:48]

Make this private and use FRIEND_TEST_ALL_PREFIXES.

[hubbe, 2014/04/29 20:03:19]

Done.

+  bool GetPacket32(uint32 frame_id,
+                   uint16 packet_id,
+                   SendPacketVector* packets);
  private:
-  void CleanupOldPackets(base::TimeTicks now);
+  void CleanupOldPackets(uint32 current_frame_id);
 
-  base::TickClock* const clock_;  // Not owned by this class.
-  base::TimeDelta max_time_stored_;
   PacketMap stored_packets_;
-  TimeToPacketMap time_to_packet_map_;
+  uint32 stored_frames_;
 
   DISALLOW_COPY_AND_ASSIGN(PacketStorage);
 };