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

Unified Diff: net/android/network_change_notifier_android_unittest.cc

Issue 1967373003: Rename NetworkChangeNotifierAutoDetect.Observer.updateActiveNetworkList (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: expand renames 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 side-by-side diff with in-line comments
Download patch
Index: net/android/network_change_notifier_android_unittest.cc
diff --git a/net/android/network_change_notifier_android_unittest.cc b/net/android/network_change_notifier_android_unittest.cc
index c6ba7c53fed97b6c6de445a8652f30ff517d71e1..dee034fb290ace1d4976e83b159e2c9b19ff4f2c 100644
--- a/net/android/network_change_notifier_android_unittest.cc
+++ b/net/android/network_change_notifier_android_unittest.cc
@@ -252,9 +252,8 @@ class BaseNetworkChangeNotifierAndroidTest : public testing::Test {
base::MessageLoop::current()->RunUntilIdle();
}
- void FakeUpdateActiveNetworkList(
- NetworkChangeNotifier::NetworkList networks) {
- delegate_.FakeUpdateActiveNetworkList(networks);
+ void FakePurgeActiveNetworkList(NetworkChangeNotifier::NetworkList networks) {
+ delegate_.FakePurgeActiveNetworkList(networks);
// See comment above.
base::MessageLoop::current()->RunUntilIdle();
}
@@ -515,7 +514,7 @@ TEST_F(NetworkChangeNotifierAndroidTest, NetworkCallbacks) {
EXPECT_EQ(100, network_list[0]);
EXPECT_EQ(101, network_list[1]);
network_list.erase(network_list.begin() + 1); // Remove network 101
- FakeUpdateActiveNetworkList(network_list);
+ FakePurgeActiveNetworkList(network_list);
network_observer.ExpectChange(DISCONNECTED, 101);
NetworkChangeNotifier::GetConnectedNetworks(&network_list);
EXPECT_EQ(1u, network_list.size());

Powered by Google App Engine
This is Rietveld 408576698