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

Unified Diff: chromeos/components/tether/initializer_unittest.cc

Issue 2861883002: [CrOS Tether] Transform NetworkConnectionHandler to an interface. (Closed)
Patch Set: Created 3 years, 8 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/components/tether/initializer_unittest.cc
diff --git a/chromeos/components/tether/initializer_unittest.cc b/chromeos/components/tether/initializer_unittest.cc
index e0c276febddd0c58fa1c7bc97481951089810c09..434de9710f42d5b8a3b2ada36ce4652b4eddf3cd 100644
--- a/chromeos/components/tether/initializer_unittest.cc
+++ b/chromeos/components/tether/initializer_unittest.cc
@@ -89,6 +89,31 @@ class TestNetworkConnectionHandler : public NetworkConnectionHandler {
public:
TestNetworkConnectionHandler() : NetworkConnectionHandler() {}
~TestNetworkConnectionHandler() override {}
+
+ // NetworkConnectionHandler:
+ void ConnectToNetwork(const std::string& service_path,
+ const base::Closure& success_callback,
+ const network_handler::ErrorCallback& error_callback,
+ bool check_error_state) override {}
stevenjb 2017/05/03 22:05:38 blank line between methods with an implementation.
Kyle Horimoto 2017/05/03 22:38:29 Done.
+ void DisconnectNetwork(
+ const std::string& service_path,
+ const base::Closure& success_callback,
+ const network_handler::ErrorCallback& error_callback) override {}
+ bool HasConnectingNetwork(const std::string& service_path) override {
+ return false;
+ }
+ bool HasPendingConnectRequest() override { return false; }
+ void Init(NetworkStateHandler* network_state_handler,
+ NetworkConfigurationHandler* network_configuration_handler,
+ ManagedNetworkConfigurationHandler*
+ managed_network_configuration_handler) override {}
+
+ // LoginState::Observer
+ void LoggedInStateChanged() override {}
+
+ // CertLoader::Observer
+ void OnCertificatesLoaded(const net::CertificateList& cert_list,
+ bool initial_load) override {}
};
} // namespace

Powered by Google App Engine
This is Rietveld 408576698