Propagate probing cluster id to SendTimeHistory.

webrtc/video/rtp_stream_receiver.cc

Index: webrtc/video/rtp_stream_receiver.cc
diff --git a/webrtc/video/rtp_stream_receiver.cc b/webrtc/video/rtp_stream_receiver.cc
index 98b46831c53894ae655928156babf8120a45fff7..4694f3d48a8912a6925a826f0345f49f9ebfaf73 100644
--- a/webrtc/video/rtp_stream_receiver.cc
+++ b/webrtc/video/rtp_stream_receiver.cc
@@ -309,7 +309,7 @@ bool RtpStreamReceiver::DeliverRtp(const uint8_t* rtp_packet,
   }
 
   remote_bitrate_estimator_->IncomingPacket(arrival_time_ms, payload_length,
-                                            header, true);

[danilchap, 2016/05/26 11:22:42]

this change looks even stranger than similar code for incoming audio packet:
true (that means 'many be a probe') replaced with value that tells 'not a
probe'. Again, look like it could be simply removed.

[stefan-webrtc, 2016/05/26 16:29:50]

Yes, this looks wrong. If anything it should be assumed to always be a probe as
before. Not sure how that would work though since we don't know what probe
cluster it belongs to at the receiver.

+                                            header, PacketInfo::kNotAProbe);
   header.payload_type_frequency = kVideoPayloadTypeFrequency;
 
   bool in_order = IsPacketInOrder(header);