| commit | 08252de51024711fea764c0f5bae503f1cac7aa7 | [log] [tgz] |
|---|---|---|
| author | Ken Rockot <[email protected]> | Wed Sep 23 17:56:59 2020 |
| committer | Commit Bot <[email protected]> | Wed Sep 23 17:56:59 2020 |
| tree | 37b47a3f1e4da623172d9abee2a4297f652c794e | |
| parent | 96223eb1e28dd2c077a2314526ae3331ec486714 [diff] [blame] |
Introduce NetworkChangeNotifier CONNECTION_5G enum This does not implement any support for 5G detection, but merely introduces the new enum value and ensures that consumers of the ConnectionType enum are aware of its existence. Bug: 1127134 Change-Id: I7fcc214958ad8a8132ac5b062b8af3559622b7ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425164 Reviewed-by: Kinuko Yasuda <[email protected]> Reviewed-by: Scott Violet <[email protected]> Reviewed-by: Tom Sepez <[email protected]> Reviewed-by: Zhongyi Shi <[email protected]> Reviewed-by: Eugene But <[email protected]> Reviewed-by: Min Qin <[email protected]> Reviewed-by: Clark DuVall <[email protected]> Commit-Queue: Ken Rockot <[email protected]> Cr-Commit-Position: refs/heads/master@{#809853}
diff --git a/net/base/network_change_notifier_unittest.cc b/net/base/network_change_notifier_unittest.cc index 289fba0..b5c2d86 100644 --- a/net/base/network_change_notifier_unittest.cc +++ b/net/base/network_change_notifier_unittest.cc
@@ -54,6 +54,11 @@ EXPECT_GE(100.0, max_bandwidth); EXPECT_LE(100.0, max_bandwidth); break; + case NetworkChangeNotifier::CONNECTION_5G: + // TODO(crbug.com/1127134): Expect proper bounds once we have introduced + // subtypes for 5G connections. + EXPECT_EQ(std::numeric_limits<double>::infinity(), max_bandwidth); + break; case NetworkChangeNotifier::CONNECTION_NONE: EXPECT_EQ(0.0, max_bandwidth); break;