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

Unified Diff: chrome/browser/extensions/api/networking_private/networking_private_apitest.cc

Issue 289383004: Merge FavoriteState into NetworkState (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 6 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: chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
diff --git a/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc b/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
index 2a0530f4e7e0782743a1eceadd0bb3a5c7d45d45..eda0f53d6b4b7af1fad0a1a120d8269cc3c64464 100644
--- a/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
+++ b/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
@@ -225,8 +225,6 @@ class ExtensionNetworkingPrivateApiTest
"stub_cellular1",
shill::kRoamingStateProperty,
base::StringValue(shill::kRoamingStateHome));
- DBusThreadManager::Get()->GetShillManagerClient()->GetTestInterface()->
- SortManagerServices();
content::RunAllPendingInMessageLoop();
}
@@ -263,7 +261,6 @@ class ExtensionNetworkingPrivateApiTest
device_test_->ClearDevices();
service_test_->ClearServices();
- profile_test->ClearProfiles();
// Sends a notification about the added profile.
profile_test->AddProfile(kUser1ProfilePath, userhash_);
@@ -354,8 +351,6 @@ class ExtensionNetworkingPrivateApiTest
AddService("stub_vpn1", "vpn1", shill::kTypeVPN, shill::kStateOnline);
- manager_test_->SortManagerServices();
-
content::RunAllPendingInMessageLoop();
}
#else // !defined(OS_CHROMEOS)
@@ -427,11 +422,14 @@ IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest,
#if defined(OS_CHROMEOS)
// TODO(stevenjb/mef): Fix these on non-Chrome OS, crbug.com/371442.
IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest, GetNetworks) {
- // Remove "stub_wifi2" from the visible list.
- manager_test_->RemoveManagerService("stub_wifi2", false);
+ // Hide stub_wifi2.
+ service_test_->SetServiceProperty("stub_wifi2",
+ shill::kVisibleProperty,
+ base::FundamentalValue(false));
// Add a couple of additional networks that are not configured (saved).
AddService("stub_wifi3", "wifi3", shill::kTypeWifi, shill::kStateIdle);
AddService("stub_wifi4", "wifi4", shill::kTypeWifi, shill::kStateIdle);
+ content::RunAllPendingInMessageLoop();
EXPECT_TRUE(RunNetworkingSubtest("getNetworks")) << message_;
}

Powered by Google App Engine
This is Rietveld 408576698