Add wifi AP info to system profile metrics

chromeos/network/network_change_notifier_chromeos.cc

 // Copyright (c) 2012 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 <string>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/network/network_change_notifier_chromeos.h"
+#include "chromeos/network/network_configuration_handler.h"
 #include "chromeos/network/network_event_log.h"
 #include "chromeos/network/network_state.h"
 #include "chromeos/network/network_state_handler.h"
 #include "net/base/network_change_notifier.h"
 #include "net/dns/dns_config_service_posix.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace chromeos {
 
 // DNS config services on Chrome OS are signalled by the network state handler
@@ skipping 53 matching lines @@
   NetworkHandler::Get()->network_state_handler()->RemoveObserver(
       this, FROM_HERE);
   DBusThreadManager::Get()->GetPowerManagerClient()->RemoveObserver(this);
 }
 
 net::NetworkChangeNotifier::ConnectionType
 NetworkChangeNotifierChromeos::GetCurrentConnectionType() const {
   return connection_type_;
 }
 
+bool NetworkChangeNotifierChromeos::GetCurrentWifiApInfo(
+    net::NetworkChangeNotifier::WifiApInfo &info) {
+  if (connection_type_ != CONNECTION_WIFI) {
+    return false;
+  }

[stevenjb, 2014/06/12 20:54:40]

nit: {} unnecessary.

[zqiu1, 2014/07/07 23:37:33]

Done.

+
+  info = wifi_ap_info_;
+  return true;
+}
+
 void NetworkChangeNotifierChromeos::SuspendDone(
     const base::TimeDelta& sleep_duration) {
   // Force invalidation of network resources on resume.
   NetworkChangeNotifier::NotifyObserversOfIPAddressChange();
 }
 
 
 void NetworkChangeNotifierChromeos::DefaultNetworkChanged(
     const chromeos::NetworkState* default_network) {
   bool connection_type_changed = false;
   bool ip_address_changed = false;
   bool dns_changed = false;
 
   UpdateState(default_network, &connection_type_changed,
               &ip_address_changed, &dns_changed);
 
   if (connection_type_changed)
     NetworkChangeNotifier::NotifyObserversOfConnectionTypeChange();
   if (ip_address_changed)
     NetworkChangeNotifier::NotifyObserversOfIPAddressChange();
   if (dns_changed)
     dns_config_service_->OnNetworkChange();
+
+  // Retrieve AP info for wifi connection.
+  if (connection_type_ == CONNECTION_WIFI) {
+    NetworkHandler::Get()->network_configuration_handler()->GetProperties(
+        default_network->path(),
+        base::Bind(&NetworkChangeNotifierChromeos::ParseWifiApInfo,
+                   AsWeakPtr()),
+        base::Bind(&NetworkChangeNotifierChromeos::GetPropertiesFailedHandler,
+                   AsWeakPtr()));
+  }
+}
+
+void NetworkChangeNotifierChromeos::ParseWifiApInfo(
+    const std::string &service_path,
+    const base::DictionaryValue& properties) {
+  net::NetworkChangeNotifier::WifiApInfo info;
+  properties.GetStringWithoutPathExpansion(
+      shill::kWifiBSsid, &(info.bssid));

[stevenjb, 2014/06/12 20:54:40]

nit: () after & unnecessary here and below

[zqiu1, 2014/07/07 23:37:33]

Done.

+  properties.GetStringWithoutPathExpansion(
+      shill::kSecurityProperty, &(info.security));
+  const base::DictionaryValue* vendor_dict = NULL;
+  if (properties.GetDictionaryWithoutPathExpansion(
+          shill::kWifiVendorInformationProperty,
+          &vendor_dict)) {
+    vendor_dict->GetStringWithoutPathExpansion(
+        shill::kVendorWPSModelNumberProperty,
+        &(info.model_number));
+    vendor_dict->GetStringWithoutPathExpansion(
+        shill::kVendorWPSModelNameProperty,
+        &(info.model_name));
+    vendor_dict->GetStringWithoutPathExpansion(
+        shill::kVendorWPSDeviceNameProperty,
+        &(info.device_name));
+    vendor_dict->GetStringWithoutPathExpansion(
+        shill::kVendorWPSManufacturerProperty,
+        &(info.manufacturer));
+    vendor_dict->GetStringWithoutPathExpansion(shill::kVendorOUIListProperty,
+                                               &(info.oui_list));
+  }
+  wifi_ap_info_ = info;
+}
+
+void NetworkChangeNotifierChromeos::GetPropertiesFailedHandler(
+    const std::string& error_name,
+    scoped_ptr<base::DictionaryValue> error_data) {
+  // Nothing to be done, most likely the network service is already removed.

[stevenjb, 2014/06/12 20:54:40]

Since this is a noop you can just pass network_handler::ErrorCallback() to
GetProperties instead.

[zqiu1, 2014/07/07 23:37:32]

Done.

 }
 
 void NetworkChangeNotifierChromeos::UpdateState(
     const chromeos::NetworkState* default_network,
     bool* connection_type_changed,
     bool* ip_address_changed,
     bool* dns_changed) {
   *connection_type_changed = false;
   *ip_address_changed = false;
   *dns_changed = false;
@@ skipping 111 matching lines @@
   // produce a single signal when switching between network connections.
   params.ip_address_offline_delay_ = base::TimeDelta::FromMilliseconds(4000);
   params.ip_address_online_delay_ = base::TimeDelta::FromMilliseconds(1000);
   params.connection_type_offline_delay_ =
       base::TimeDelta::FromMilliseconds(500);
   params.connection_type_online_delay_ = base::TimeDelta::FromMilliseconds(500);
   return params;
 }
 
 }  // namespace chromeos