Tether: record each type of host connection result.

chromeos/components/tether/wifi_hotspot_connector.cc

 // Copyright 2017 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.
 
 #include "chromeos/components/tether/wifi_hotspot_connector.h"
 
 #include "base/bind.h"
 #include "base/guid.h"
 #include "base/memory/ptr_util.h"
+#include "chromeos/components/tether/host_connection_metrics_logger.h"
 #include "chromeos/network/network_connect.h"
 #include "chromeos/network/network_handler.h"
 #include "chromeos/network/network_state.h"
 #include "chromeos/network/network_state_handler.h"
 #include "chromeos/network/shill_property_util.h"
 #include "components/proximity_auth/logging/logging.h"
 #include "third_party/cros_system_api/dbus/shill/dbus-constants.h"
 
 namespace chromeos {
 
 namespace tether {
 
 WifiHotspotConnector::WifiHotspotConnector(
     NetworkStateHandler* network_state_handler,
-    NetworkConnect* network_connect)
+    NetworkConnect* network_connect,
+    HostConnectionMetricsLogger* host_connection_metrics_logger)
     : network_state_handler_(network_state_handler),
       network_connect_(network_connect),
+      host_connection_metrics_logger_(host_connection_metrics_logger),
       timer_(base::MakeUnique<base::OneShotTimer>()),
       weak_ptr_factory_(this) {
   network_state_handler_->AddObserver(this, FROM_HERE);
 }
 
 WifiHotspotConnector::~WifiHotspotConnector() {
   network_state_handler_->RemoveObserver(this, FROM_HERE);
 }
 
 void WifiHotspotConnector::ConnectToWifiHotspot(
@@ skipping 18 matching lines @@
       PA_LOG(INFO) << "Wi-Fi network (ID \"" << wifi_network_guid_ << "\") "
                    << "successfully disassociated from Tether network (ID "
                    << "\"" << tether_network_guid_ << "\").";
     } else {
       PA_LOG(INFO) << "Wi-Fi network (ID \"" << wifi_network_guid_ << "\") "
                    << "failed to disassociate from Tether network ID (\""
                    << tether_network_guid_ << "\").";
     }
 
     InvokeWifiConnectionCallback(std::string());
+
+    host_connection_metrics_logger_->RecordConnectionToHostResult(
+        HostConnectionMetricsLogger::ConnectionToHostResult::
+            CONNECTION_RESULT_FAILURE_CLIENT_CONNECTION_CANCELED_BY_NEW_ATTEMPT);
   }
 
   ssid_ = ssid;
   password_ = password;
   tether_network_guid_ = tether_network_guid;
   wifi_network_guid_ = base::GenerateGUID();
   callback_ = callback;
   timer_->Start(FROM_HERE,
                 base::TimeDelta::FromSeconds(kConnectionTimeoutSeconds),
                 base::Bind(&WifiHotspotConnector::OnConnectionTimeout,
@@ skipping 94 matching lines @@
                                              shill::kSecurityPsk);
     properties.SetStringWithoutPathExpansion(shill::kPassphraseProperty,
                                              password);
   }
 
   return properties;
 }
 
 void WifiHotspotConnector::OnConnectionTimeout() {
   InvokeWifiConnectionCallback(std::string());
+
+  host_connection_metrics_logger_->RecordConnectionToHostResult(
+      HostConnectionMetricsLogger::ConnectionToHostResult::
+          CONNECTION_RESULT_FAILURE_CLIENT_CONNECTION_TIMEOUT);
 }
 
 void WifiHotspotConnector::SetTimerForTest(std::unique_ptr<base::Timer> timer) {
   timer_ = std::move(timer);
 }
 
 }  // namespace tether
 
 }  // namespace chromeos