Update SendTimeHistory to store complete PacketInfo, not just send time

webrtc/modules/remote_bitrate_estimator/include/send_time_history.h

Index: webrtc/modules/remote_bitrate_estimator/include/send_time_history.h
diff --git a/webrtc/modules/remote_bitrate_estimator/include/send_time_history.h b/webrtc/modules/remote_bitrate_estimator/include/send_time_history.h
index 8835856353c285d93c7ac7cf3b031438dbf5096f..341956b527fd7515cc2e83c18d6c50e25676209a 100644
--- a/webrtc/modules/remote_bitrate_estimator/include/send_time_history.h
+++ b/webrtc/modules/remote_bitrate_estimator/include/send_time_history.h
@@ -15,6 +15,7 @@
 
 #include "webrtc/base/constructormagic.h"
 #include "webrtc/base/basictypes.h"
+#include "webrtc/modules/remote_bitrate_estimator/include/remote_bitrate_estimator.h"
 
 namespace webrtc {
 
@@ -23,8 +24,9 @@ class SendTimeHistory {
   explicit SendTimeHistory(int64_t packet_age_limit);
   virtual ~SendTimeHistory();
 
-  void AddAndRemoveOldSendTimes(uint16_t sequence_number, int64_t timestamp);
-  bool GetSendTime(uint16_t sequence_number, int64_t* timestamp, bool remove);
+  void AddAndRemoveOld(const PacketInfo& packet);
+  bool UpdateSendTime(uint16_t sequence_number, int64_t timestamp);
+  bool Populate(PacketInfo* packet, bool remove);

[stefan-webrtc, 2015/09/03 11:30:32]

GetPacketInfo would be better I think. Populate could mean that we want to
insert packet into the history.

[sprang_webrtc, 2015/09/03 13:24:09]

Updated, and added a comment about expected input/output.

   void Clear();
 
  private:
@@ -33,7 +35,7 @@ class SendTimeHistory {
 
   const int64_t packet_age_limit_;
   uint16_t oldest_sequence_number_;  // Oldest may not be lowest.
-  std::map<uint16_t, int64_t> history_;
+  std::map<uint16_t, PacketInfo> history_;
 
   DISALLOW_COPY_AND_ASSIGN(SendTimeHistory);
 };