Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(448)

Side by Side Diff: webrtc/modules/pacing/bitrate_prober_unittest.cc

Issue 2235373004: Probing: Add support for exponential startup probing (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@fix_probing2
Patch Set: Updates Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 #include <limits> 11 #include <limits>
12 12
13 #include "testing/gtest/include/gtest/gtest.h" 13 #include "testing/gtest/include/gtest/gtest.h"
14 #include "webrtc/modules/pacing/bitrate_prober.h" 14 #include "webrtc/modules/pacing/bitrate_prober.h"
15 15
16 namespace webrtc { 16 namespace webrtc {
17 17
18 namespace {
19
20 constexpr int probe_multipliers[2] = {3, 6};
stefan-webrtc 2016/08/16 11:27:04 kProbeMultipliers
Irfan 2016/08/16 18:12:47 Done.
21
22 } // namespace
23
18 TEST(BitrateProberTest, VerifyStatesAndTimeBetweenProbes) { 24 TEST(BitrateProberTest, VerifyStatesAndTimeBetweenProbes) {
19 BitrateProber prober; 25 BitrateProber prober;
20 EXPECT_FALSE(prober.IsProbing()); 26 EXPECT_FALSE(prober.IsProbing());
21 int64_t now_ms = 0; 27 int64_t now_ms = 0;
22 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms)); 28 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms));
23 29
24 prober.SetEnabled(true); 30 prober.SetEnabled(true);
25 EXPECT_FALSE(prober.IsProbing()); 31 EXPECT_FALSE(prober.IsProbing());
26 32
27 prober.OnIncomingPacket(300000, 1000, now_ms); 33 prober.InitiateProbing(300000, probe_multipliers, 2);
28 EXPECT_TRUE(prober.IsProbing()); 34 EXPECT_TRUE(prober.IsProbing());
29 EXPECT_EQ(0, prober.CurrentClusterId()); 35 EXPECT_EQ(0, prober.CurrentClusterId());
30 36
31 // First packet should probe as soon as possible. 37 // First packet should probe as soon as possible.
32 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms)); 38 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms));
33 prober.PacketSent(now_ms, 1000); 39 prober.PacketSent(now_ms, 1000);
34 40
35 for (int i = 0; i < 5; ++i) { 41 for (int i = 0; i < 5; ++i) {
36 EXPECT_EQ(8, prober.TimeUntilNextProbe(now_ms)); 42 EXPECT_EQ(8, prober.TimeUntilNextProbe(now_ms));
37 now_ms += 4; 43 now_ms += 4;
(...skipping 17 matching lines...) Expand all
55 61
56 TEST(BitrateProberTest, DoesntProbeWithoutRecentPackets) { 62 TEST(BitrateProberTest, DoesntProbeWithoutRecentPackets) {
57 BitrateProber prober; 63 BitrateProber prober;
58 EXPECT_FALSE(prober.IsProbing()); 64 EXPECT_FALSE(prober.IsProbing());
59 int64_t now_ms = 0; 65 int64_t now_ms = 0;
60 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms)); 66 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms));
61 67
62 prober.SetEnabled(true); 68 prober.SetEnabled(true);
63 EXPECT_FALSE(prober.IsProbing()); 69 EXPECT_FALSE(prober.IsProbing());
64 70
65 prober.OnIncomingPacket(300000, 1000, now_ms); 71 prober.InitiateProbing(300000, probe_multipliers, 2);
66 EXPECT_TRUE(prober.IsProbing()); 72 EXPECT_TRUE(prober.IsProbing());
67 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms)); 73 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms));
68 prober.PacketSent(now_ms, 1000); 74 prober.PacketSent(now_ms, 1000);
69 // Let time pass, no large enough packets put into prober. 75 // Let time pass, no large enough packets put into prober.
70 now_ms += 6000; 76 now_ms += 6000;
71 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms)); 77 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms));
72 // Insert a small packet, not a candidate for probing.
73 prober.OnIncomingPacket(300000, 100, now_ms);
74 prober.PacketSent(now_ms, 100);
75 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms));
76 // Insert a large-enough packet after downtime while probing should reset to 78 // Insert a large-enough packet after downtime while probing should reset to
77 // perform a new probe since the requested one didn't finish. 79 // perform a new probe since the requested one didn't finish.
78 prober.OnIncomingPacket(300000, 1000, now_ms); 80 prober.InitiateProbing(300000, probe_multipliers, 2);
79 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms)); 81 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms));
80 prober.PacketSent(now_ms, 1000); 82 prober.PacketSent(now_ms, 1000);
81 // Next packet should be part of new probe and be sent with non-zero delay. 83 // Next packet should be part of new probe and be sent with non-zero delay.
82 prober.OnIncomingPacket(300000, 1000, now_ms); 84 prober.InitiateProbing(300000, probe_multipliers, 2);
83 EXPECT_GT(prober.TimeUntilNextProbe(now_ms), 0); 85 EXPECT_GT(prober.TimeUntilNextProbe(now_ms), 0);
84 } 86 }
85 87
86 TEST(BitrateProberTest, DoesntInitializeProbingForSmallPackets) {
87 BitrateProber prober;
88 prober.SetEnabled(true);
89 EXPECT_FALSE(prober.IsProbing());
90
91 prober.OnIncomingPacket(300000, 100, 0);
92 EXPECT_FALSE(prober.IsProbing());
93 }
94
95 } // namespace webrtc 88 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698