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

Unified Diff: chromeos/geolocation/simple_geolocation_unittest.cc

Issue 2314853004: Remove calls to deprecated MessageLoop methods in chromeos. (Closed)
Patch Set: Created 4 years, 3 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: chromeos/geolocation/simple_geolocation_unittest.cc
diff --git a/chromeos/geolocation/simple_geolocation_unittest.cc b/chromeos/geolocation/simple_geolocation_unittest.cc
index 93ff93eba967824c648167a3e875000b1d07b228..d18502ec4195f43f9ecde93677f0404f0354c449 100644
--- a/chromeos/geolocation/simple_geolocation_unittest.cc
+++ b/chromeos/geolocation/simple_geolocation_unittest.cc
@@ -341,7 +341,7 @@ class SimpleGeolocationWiFiTest : public ::testing::TestWithParam<bool> {
ASSERT_TRUE(manager_test_);
geolocation_handler_.reset(new GeolocationHandler());
geolocation_handler_->Init();
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
}
void TearDown() override {
@@ -367,7 +367,7 @@ class SimpleGeolocationWiFiTest : public ::testing::TestWithParam<bool> {
properties.SetStringWithoutPathExpansion(shill::kGeoSignalStrengthProperty,
strength);
manager_test_->AddGeoNetwork(shill::kTypeWifi, properties);
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
}
protected:
@@ -408,10 +408,10 @@ TEST_P(SimpleGeolocationWiFiTest, WiFiExists) {
// Add an acces point.
AddAccessPoint(1);
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
// Inititial call should return false and request access points.
EXPECT_FALSE(GetWifiAccessPoints());
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
// Second call should return true since we have an access point.
EXPECT_TRUE(GetWifiAccessPoints());
ASSERT_EQ(1u, wifi_access_points_.size());

Powered by Google App Engine
This is Rietveld 408576698