Propagate probing cluster id to SendTimeHistory.

webrtc/modules/remote_bitrate_estimator/test/estimators/send_side.cc

Index: webrtc/modules/remote_bitrate_estimator/test/estimators/send_side.cc
diff --git a/webrtc/modules/remote_bitrate_estimator/test/estimators/send_side.cc b/webrtc/modules/remote_bitrate_estimator/test/estimators/send_side.cc
index 3cf7c752a07e911bd680a3c48e0addaf57518ecb..48fd74c5e7ad715cf5ebff570b84c1821d6dbf92 100644
--- a/webrtc/modules/remote_bitrate_estimator/test/estimators/send_side.cc
+++ b/webrtc/modules/remote_bitrate_estimator/test/estimators/send_side.cc
@@ -92,9 +92,9 @@ void FullBweSender::OnPacketsSent(const Packets& packets) {
   for (Packet* packet : packets) {
     if (packet->GetPacketType() == Packet::kMedia) {
       MediaPacket* media_packet = static_cast<MediaPacket*>(packet);
-      send_time_history_.AddAndRemoveOld(media_packet->header().sequenceNumber,
-                                         media_packet->payload_size(),
-                                         packet->paced());
+      send_time_history_.AddAndRemoveOld(
+          media_packet->header().sequenceNumber, media_packet->payload_size(),
+          packet->paced(), PacketInfo::kNotAProbe);

[danilchap, 2016/05/31 13:06:50]

This look like the place to test how cluster id affects send-side BWE.
Can you add at least add a TODO comment to extend this test suit to use
propagated cluster_id.

[philipel, 2016/05/31 14:34:58]

Added a TODO.

       send_time_history_.OnSentPacket(media_packet->header().sequenceNumber,
                                       media_packet->sender_timestamp_ms());
     }