Remove the configuration of Tether-associated Wi-Fi networks once connectivity is lost.

chromeos/network/network_state_handler.cc

Index: chromeos/network/network_state_handler.cc
diff --git a/chromeos/network/network_state_handler.cc b/chromeos/network/network_state_handler.cc
index aeba782ca751cfe87730c31e6c33375bf4defc5b..40b07001b844735780f313a506eb79dd42db55cf 100644
--- a/chromeos/network/network_state_handler.cc
+++ b/chromeos/network/network_state_handler.cc
@@ -624,16 +624,15 @@ void NetworkStateHandler::UpdateManagedList(ManagedState::ManagedType type,
   if (type != ManagedState::ManagedType::MANAGED_TYPE_NETWORK)
     return;
 
-  // Remove associations Tether NetworkStates had with now removed Wi-Fi
-  // NetworkStates.
+  // If a Wi-Fi NetworkState was removed, also remove its associated
+  // TetherNetworkState (if it has one).
   for (auto& iter : managed_map) {
-    if (!iter.second->Matches(NetworkTypePattern::WiFi()))
-      continue;
-
-    NetworkState* tether_network = GetModifiableNetworkStateFromGuid(
-        iter.second->AsNetworkState()->tether_guid());
-    if (tether_network)
-      tether_network->set_tether_guid(std::string());
+    if (iter.second->Matches(NetworkTypePattern::WiFi())) {
+      std::string tether_network_guid =
+          iter.second->AsNetworkState()->tether_guid();
+      if (!tether_network_guid.empty())
+        RemoveTetherNetworkState(tether_network_guid);

[Kyle Horimoto, 2017/04/17 20:34:53]

I don't think this should be a responsibility of NetworkStateHandler. Since the
tether component is adding the network, I think it should also remove the
network when it no longer needs it. NetworkStateHandler should just clear the
GUID as it did before this CL. Let's revert the changes to this file and move
the RemoveTetherNetworkState() call to the tether component.

[stevenjb, 2017/04/18 21:03:36]

+1. That allows the Tether code to make the decision whether to remove a Tether
network when it's associated WiFi network goes out of range, or keep it around
and update it's state (e.g. to 'out of range').

[Ryan Hansberry, 2017/04/20 20:57:58]

Done.

+    }
   }
 }