Only use payload size within the receive/send interval for bitrate probing.

webrtc/modules/congestion_controller/probe_bitrate_estimator_unittest.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 107 matching lines @@
   AddPacketFeedback(1, 1000, 70, 84);
   AddPacketFeedback(1, 1000, 75, 90);
 
   CheckResult(0, 1200000, 10, 90);
 
   AddPacketFeedback(0, 1000, 40, 60);
 
   EXPECT_EQ(1ul, results_.size());
 }
 
+TEST_F(TestProbeBitrateEstimator, IgnoreSizeLastSendPacket) {
+  AddPacketFeedback(0, 1000, 0, 10);
+  AddPacketFeedback(0, 1000, 10, 20);
+  AddPacketFeedback(0, 1000, 20, 30);
+  AddPacketFeedback(0, 1000, 30, 40);
+  AddPacketFeedback(0, 1500, 40, 50);

[danilchap, 2016/08/22 13:53:35]

May be add note that increased packet size increase received speed, but not send
speed, and since minimum of those two is returned, CheckResult validates send
bitrate is unaffected by last packet.

[philipel, 2016/08/22 14:06:40]

Acknowledged.

+
+  CheckResult(0, 800000, 10, 40);

[danilchap, 2016/08/22 13:53:35]

May be add a note that 1st CheckResult check bitrate after 4 packets, last -
after 5 packets.

[philipel, 2016/08/22 14:06:40]

Acknowledged.

+  CheckResult(1, 800000, 10, 50);
+}
+
+TEST_F(TestProbeBitrateEstimator, IgnoreSizeFirstReceivePacket) {
+  AddPacketFeedback(0, 1500, 0, 10);
+  AddPacketFeedback(0, 1000, 10, 20);
+  AddPacketFeedback(0, 1000, 20, 30);
+  AddPacketFeedback(0, 1000, 30, 40);
+  AddPacketFeedback(0, 1000, 40, 50);
+
+  CheckResult(0, 800000, 10, 40);

[danilchap, 2016/08/22 13:53:35]

does it make sense to CheckResult twice in this test?

[philipel, 2016/08/22 14:06:40]

Copy-paste ftw :)

But no, it doesn't.

+  CheckResult(1, 800000, 10, 50);
+}
+
 }  // namespace webrtc