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

Unified Diff: webrtc/base/network_unittest.cc

Issue 1284113003: Move the concept of multiple route into Network (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/base/network_unittest.cc
diff --git a/webrtc/base/network_unittest.cc b/webrtc/base/network_unittest.cc
index fdf75caf37aed77c4538495175c791c341b72609..b089b973f55b5f384bce618a2a54169eb18d4116 100644
--- a/webrtc/base/network_unittest.cc
+++ b/webrtc/base/network_unittest.cc
@@ -220,7 +220,9 @@ TEST_F(NetworkTest, TestBasicMergeNetworkList) {
NetworkManager::NetworkList list;
list.push_back(new Network(ipv4_network1));
bool changed;
+ EXPECT_EQ(manager.network_permission_state(), NetworkManager::STATE_UNKNOWN);
juberti1 2015/08/14 23:25:55 I think this should go into TestUpdateNetworks (wh
NetworkManager::Stats stats = MergeNetworkList(manager, list, &changed);
+ EXPECT_EQ(manager.network_permission_state(), NetworkManager::STATE_ALLOWED);
EXPECT_TRUE(changed);
EXPECT_EQ(stats.ipv6_network_count, 0);
EXPECT_EQ(stats.ipv4_network_count, 1);

Powered by Google App Engine
This is Rietveld 408576698