commit 92e211d05eb1e9f48fb3d6a8803d7007ab122160
author wub <wub@google.com> Tue Jun 11 04:55:16 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Jun 11 11:36:00 2019 -0700
tree 839d807ba5e5f7c8e1261e6863c12444881862ce
parent b3241165f4242a5fe2eed0ceb169334d043af1d0

gfe-relnote: (n/a) Fix QuicBandwidth::(ToBytesPerPeriod|ToKBytesPerPeriod) by doing multiplications before divisions. Not protected.

PiperOrigin-RevId: 252595070
Change-Id: Iced39e2225f45576a23a484dd43b231fff9f3636

quic/core/quic_bandwidth_test.cc

diff --git a/quic/core/quic_bandwidth_test.cc b/quic/core/quic_bandwidth_test.cc
index 22a5107..005eef0 100644
--- a/quic/core/quic_bandwidth_test.cc
+++ b/quic/core/quic_bandwidth_test.cc
@@ -84,6 +84,13 @@
                          QuicTime::Delta::FromMilliseconds(100)));
   EXPECT_EQ(200u, QuicBandwidth::FromKBytesPerSecond(2000).ToKBytesPerPeriod(
                       QuicTime::Delta::FromMilliseconds(100)));
+
+  // 1599 * 1001 = 1600599 bits/ms = 200.074875 bytes/s.
+  EXPECT_EQ(200u, QuicBandwidth::FromBitsPerSecond(1599).ToBytesPerPeriod(
+                      QuicTime::Delta::FromMilliseconds(1001)));
+
+  EXPECT_EQ(200u, QuicBandwidth::FromBitsPerSecond(1599).ToKBytesPerPeriod(
+                      QuicTime::Delta::FromSeconds(1001)));
 }
 
 TEST_F(QuicBandwidthTest, TransferTime) {