[CrOS Tether] Disconnect cleanly from active Tether networks when the user logs out or the Tether c…

chromeos/components/tether/initializer.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMEOS_COMPONENTS_TETHER_INITIALIZER_H_
 #define CHROMEOS_COMPONENTS_TETHER_INITIALIZER_H_
 
 #include <memory>
 
 #include "base/gtest_prod_util.h"
@@ skipping 29 matching lines @@
 class NetworkConnectionHandlerTetherDelegate;
 class DeviceIdTetherNetworkGuidMap;
 class HostScanCache;
 class HostScanner;
 class HostScanScheduler;
 class HostScanDevicePrioritizerImpl;
 class KeepAliveScheduler;
 class NetworkConfigurationRemover;
 class NotificationPresenter;
 class TetherConnector;
-class TetherDisconnector;
+class TetherDisconnectorImpl;
 class TetherHostFetcher;
 class TetherHostResponseRecorder;
 class TetherNetworkDisconnectionHandler;
 class WifiHotspotConnector;
 
 // Initializes the Tether Chrome OS component.
 class Initializer : public OAuth2TokenService::Observer {
  public:
   // Initializes the tether feature. If the feature has already been
   // initialized, this function is a no-op.
@@ skipping 65 matching lines @@
   std::unique_ptr<ActiveHostNetworkStateUpdater>
       active_host_network_state_updater_;
   std::unique_ptr<DeviceIdTetherNetworkGuidMap>
       device_id_tether_network_guid_map_;
   std::unique_ptr<HostScanCache> host_scan_cache_;
   std::unique_ptr<KeepAliveScheduler> keep_alive_scheduler_;
   std::unique_ptr<base::DefaultClock> clock_;
   std::unique_ptr<HostScanner> host_scanner_;
   std::unique_ptr<HostScanScheduler> host_scan_scheduler_;
   std::unique_ptr<TetherConnector> tether_connector_;
-  std::unique_ptr<TetherDisconnector> tether_disconnector_;

[Ryan Hansberry, 2017/07/11 22:23:56]

Was the change to this file necessary? (I don't feel that it was)

[Kyle Horimoto, 2017/07/12 01:42:59]

Nope - changed.

+  std::unique_ptr<TetherDisconnectorImpl> tether_disconnector_;
   std::unique_ptr<NetworkConfigurationRemover> network_configuration_remover_;
   std::unique_ptr<NetworkConnectionHandlerTetherDelegate>
       network_connection_handler_tether_delegate_;
   std::unique_ptr<TetherNetworkDisconnectionHandler>
       tether_network_disconnection_handler_;
 
   base::WeakPtrFactory<Initializer> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(Initializer);
 };
 
 }  // namespace tether
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_COMPONENTS_TETHER_INITIALIZER_H_