Avi Drissman | 6459548 | 2022-09-14 20:52:29 | [diff] [blame] | 1 | // Copyright 2017 The Chromium Authors |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include "net/nqe/event_creator.h" |
| 6 | |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 7 | #include "base/time/time.h" |
| 8 | #include "net/log/net_log_with_source.h" |
| 9 | #include "net/log/test_net_log.h" |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 10 | #include "net/nqe/effective_connection_type.h" |
| 11 | #include "net/nqe/network_quality.h" |
| 12 | #include "testing/gtest/include/gtest/gtest.h" |
| 13 | |
Tsuyoshi Horo | 4f516be | 2022-06-14 11:53:13 | [diff] [blame] | 14 | namespace net::nqe::internal { |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 15 | |
| 16 | namespace { |
| 17 | |
| 18 | // Returns the number of entries in |net_log| that have type set to |
| 19 | // |NetLogEventType::NETWORK_QUALITY_CHANGED|. |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 20 | int GetNetworkQualityChangedEntriesCount(RecordingNetLogObserver* net_log) { |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 21 | return net_log->GetEntriesWithType(NetLogEventType::NETWORK_QUALITY_CHANGED) |
| 22 | .size(); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 23 | } |
| 24 | |
| 25 | // Verify that the net log events are recorded correctly. |
| 26 | TEST(NetworkQualityEstimatorEventCreatorTest, Notified) { |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 27 | RecordingNetLogObserver net_log_observer; |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 28 | |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 29 | EventCreator event_creator(NetLogWithSource::Make(NetLogSourceType::NONE)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 30 | |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 31 | NetworkQuality network_quality_100(base::Milliseconds(100), |
| 32 | base::Milliseconds(100), 100); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 33 | |
| 34 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 35 | EFFECTIVE_CONNECTION_TYPE_2G, network_quality_100); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 36 | EXPECT_EQ(1, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 37 | |
| 38 | // No new entry should be created since the network quality has not changed. |
| 39 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 40 | EFFECTIVE_CONNECTION_TYPE_2G, network_quality_100); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 41 | EXPECT_EQ(1, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 42 | |
| 43 | // A new entry should be created since effective connection type has changed. |
| 44 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 45 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_100); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 46 | EXPECT_EQ(2, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 47 | |
| 48 | // A new entry should not be created since HTTP RTT has not changed |
| 49 | // meaningfully. |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 50 | NetworkQuality network_quality_http_rtt_110(base::Milliseconds(110), |
| 51 | base::Milliseconds(100), 100); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 52 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 53 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_http_rtt_110); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 54 | EXPECT_EQ(2, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 55 | |
| 56 | // A new entry should be created since HTTP RTT has changed meaningfully. |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 57 | NetworkQuality network_quality_http_rtt_300(base::Milliseconds(300), |
| 58 | base::Milliseconds(100), 100); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 59 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 60 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_http_rtt_300); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 61 | EXPECT_EQ(3, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 62 | |
| 63 | // A new entry should be created since transport RTT has changed meaningfully. |
| 64 | NetworkQuality network_quality_transport_rtt_300( |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 65 | base::Milliseconds(300), base::Milliseconds(300), 100); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 66 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 67 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_transport_rtt_300); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 68 | EXPECT_EQ(4, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 69 | |
| 70 | // A new entry should be created since bandwidth has changed meaningfully. |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 71 | NetworkQuality network_quality_kbps_300(base::Milliseconds(300), |
| 72 | base::Milliseconds(300), 300); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 73 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 74 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_kbps_300); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 75 | EXPECT_EQ(5, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 76 | |
| 77 | // A new entry should not be created since network quality has not changed |
| 78 | // meaningfully. |
| 79 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 80 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_kbps_300); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 81 | EXPECT_EQ(5, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 82 | |
| 83 | // A new entry should be created since bandwidth has changed meaningfully. |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 84 | NetworkQuality network_quality_kbps_2000(base::Milliseconds(300), |
| 85 | base::Milliseconds(300), 2000); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 86 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 87 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_kbps_2000); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 88 | EXPECT_EQ(6, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 89 | |
| 90 | // A new entry should not be created since bandwidth has not changed by more |
| 91 | // than 20%. |
Peter Kasting | e5a38ed | 2021-10-02 03:06:35 | [diff] [blame] | 92 | NetworkQuality network_quality_kbps_2200(base::Milliseconds(300), |
| 93 | base::Milliseconds(300), 2200); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 94 | event_creator.MaybeAddNetworkQualityChangedEventToNetLog( |
| 95 | EFFECTIVE_CONNECTION_TYPE_3G, network_quality_kbps_2200); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 96 | EXPECT_EQ(6, GetNetworkQualityChangedEntriesCount(&net_log_observer)); |
tbansal | f38471f | 2017-03-04 01:07:04 | [diff] [blame] | 97 | } |
| 98 | |
| 99 | } // namespace |
| 100 | |
Tsuyoshi Horo | 4f516be | 2022-06-14 11:53:13 | [diff] [blame] | 101 | } // namespace net::nqe::internal |