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

Unified Diff: chrome/browser/chromeos/tether/tether_service_unittest.cc

Issue 2975483002: [CrOS Tether] Disconnect cleanly from active Tether networks when the user logs out or the Tether c… (Closed)
Patch Set: hansberry@ comment. Created 3 years, 5 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
« no previous file with comments | « chrome/browser/chromeos/tether/tether_service.cc ('k') | chromeos/components/tether/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/tether/tether_service_unittest.cc
diff --git a/chrome/browser/chromeos/tether/tether_service_unittest.cc b/chrome/browser/chromeos/tether/tether_service_unittest.cc
index 76fdb5c5ea941823dc1deb6dbd13796ceab4ad45..be4f0faaf2b9e498236341b872299388d3f6508c 100644
--- a/chrome/browser/chromeos/tether/tether_service_unittest.cc
+++ b/chrome/browser/chromeos/tether/tether_service_unittest.cc
@@ -233,10 +233,12 @@ TEST_F(TetherServiceTest, TestShutdown) {
ShutdownTetherService();
- EXPECT_EQ(
- chromeos::NetworkStateHandler::TechnologyState::TECHNOLOGY_UNAVAILABLE,
- network_state_handler()->GetTechnologyState(
- chromeos::NetworkTypePattern::Tether()));
+ // The TechnologyState should not have changed due to Shutdown() being called.
+ // If it had changed, any settings UI that was previously open would have
+ // shown visual jank.
+ EXPECT_EQ(chromeos::NetworkStateHandler::TechnologyState::TECHNOLOGY_ENABLED,
+ network_state_handler()->GetTechnologyState(
+ chromeos::NetworkTypePattern::Tether()));
EXPECT_FALSE(test_initializer_delegate_->is_tether_running());
}
« no previous file with comments | « chrome/browser/chromeos/tether/tether_service.cc ('k') | chromeos/components/tether/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698