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

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

Issue 1946173002: Bitrate prober now keep track of probing cluster id. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase Created 4 years, 7 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
« no previous file with comments | « webrtc/modules/pacing/bitrate_prober.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 TEST(BitrateProberTest, VerifyStatesAndTimeBetweenProbes) { 18 TEST(BitrateProberTest, VerifyStatesAndTimeBetweenProbes) {
19 BitrateProber prober; 19 BitrateProber prober;
20 EXPECT_FALSE(prober.IsProbing()); 20 EXPECT_FALSE(prober.IsProbing());
21 int64_t now_ms = 0; 21 int64_t now_ms = 0;
22 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms)); 22 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms));
23 23
24 prober.SetEnabled(true); 24 prober.SetEnabled(true);
25 EXPECT_FALSE(prober.IsProbing()); 25 EXPECT_FALSE(prober.IsProbing());
26 26
27 prober.OnIncomingPacket(300000, 1000, now_ms); 27 prober.OnIncomingPacket(300000, 1000, now_ms);
28 EXPECT_TRUE(prober.IsProbing()); 28 EXPECT_TRUE(prober.IsProbing());
29 EXPECT_EQ(0, prober.CurrentClusterId());
29 30
30 // First packet should probe as soon as possible. 31 // First packet should probe as soon as possible.
31 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms)); 32 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms));
32 prober.PacketSent(now_ms, 1000); 33 prober.PacketSent(now_ms, 1000);
33 34
34 for (int i = 0; i < 5; ++i) { 35 for (int i = 0; i < 5; ++i) {
35 EXPECT_EQ(8, prober.TimeUntilNextProbe(now_ms)); 36 EXPECT_EQ(8, prober.TimeUntilNextProbe(now_ms));
36 now_ms += 4; 37 now_ms += 4;
37 EXPECT_EQ(4, prober.TimeUntilNextProbe(now_ms)); 38 EXPECT_EQ(4, prober.TimeUntilNextProbe(now_ms));
38 now_ms += 4; 39 now_ms += 4;
39 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms)); 40 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms));
41 EXPECT_EQ(0, prober.CurrentClusterId());
40 prober.PacketSent(now_ms, 1000); 42 prober.PacketSent(now_ms, 1000);
41 } 43 }
42 for (int i = 0; i < 5; ++i) { 44 for (int i = 0; i < 5; ++i) {
43 EXPECT_EQ(4, prober.TimeUntilNextProbe(now_ms)); 45 EXPECT_EQ(4, prober.TimeUntilNextProbe(now_ms));
44 now_ms += 4; 46 now_ms += 4;
45 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms)); 47 EXPECT_EQ(0, prober.TimeUntilNextProbe(now_ms));
48 EXPECT_EQ(1, prober.CurrentClusterId());
46 prober.PacketSent(now_ms, 1000); 49 prober.PacketSent(now_ms, 1000);
47 } 50 }
48 51
49 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms)); 52 EXPECT_EQ(-1, prober.TimeUntilNextProbe(now_ms));
50 EXPECT_FALSE(prober.IsProbing()); 53 EXPECT_FALSE(prober.IsProbing());
51 } 54 }
52 55
53 TEST(BitrateProberTest, DoesntProbeWithoutRecentPackets) { 56 TEST(BitrateProberTest, DoesntProbeWithoutRecentPackets) {
54 BitrateProber prober; 57 BitrateProber prober;
55 EXPECT_FALSE(prober.IsProbing()); 58 EXPECT_FALSE(prober.IsProbing());
(...skipping 26 matching lines...) Expand all
82 TEST(BitrateProberTest, DoesntInitializeProbingForSmallPackets) { 85 TEST(BitrateProberTest, DoesntInitializeProbingForSmallPackets) {
83 BitrateProber prober; 86 BitrateProber prober;
84 prober.SetEnabled(true); 87 prober.SetEnabled(true);
85 EXPECT_FALSE(prober.IsProbing()); 88 EXPECT_FALSE(prober.IsProbing());
86 89
87 prober.OnIncomingPacket(300000, 100, 0); 90 prober.OnIncomingPacket(300000, 100, 0);
88 EXPECT_FALSE(prober.IsProbing()); 91 EXPECT_FALSE(prober.IsProbing());
89 } 92 }
90 93
91 } // namespace webrtc 94 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/pacing/bitrate_prober.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698