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

Unified Diff: chromeos/network/network_change_notifier_chromeos.h

Issue 328793002: Add wifi AP info to system profile metrics (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 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/network/network_change_notifier_chromeos.h
diff --git a/chromeos/network/network_change_notifier_chromeos.h b/chromeos/network/network_change_notifier_chromeos.h
index d70386e168c509d8aa28987960e1e0394dece442..b59d7f11f3651af03820e321f7c0b927100dae70 100644
--- a/chromeos/network/network_change_notifier_chromeos.h
+++ b/chromeos/network/network_change_notifier_chromeos.h
@@ -10,6 +10,7 @@
#include "base/basictypes.h"
#include "base/gtest_prod_util.h"
#include "base/memory/scoped_ptr.h"
+#include "base/values.h"
#include "chromeos/chromeos_export.h"
#include "chromeos/dbus/power_manager_client.h"
#include "chromeos/network/network_state_handler_observer.h"
@@ -20,7 +21,8 @@ namespace chromeos {
class CHROMEOS_EXPORT NetworkChangeNotifierChromeos
: public net::NetworkChangeNotifier,
public chromeos::PowerManagerClient::Observer,
- public chromeos::NetworkStateHandlerObserver {
+ public chromeos::NetworkStateHandlerObserver,
+ public base::SupportsWeakPtr<NetworkChangeNotifierChromeos> {
public:
NetworkChangeNotifierChromeos();
virtual ~NetworkChangeNotifierChromeos();
@@ -34,6 +36,8 @@ class CHROMEOS_EXPORT NetworkChangeNotifierChromeos
// NetworkChangeNotifier overrides.
virtual net::NetworkChangeNotifier::ConnectionType
GetCurrentConnectionType() const OVERRIDE;
+ virtual bool GetCurrentWifiAccessPointInfo(WifiAccessPointInfo *info)
Ilya Sherman 2014/07/11 01:32:17 nit: Swap position of space and asterisk.
zqiu1 2014/07/14 17:21:57 Done.
+ OVERRIDE;
// PowerManagerClient::Observer overrides.
virtual void SuspendDone(const base::TimeDelta& sleep_duration) OVERRIDE;
@@ -59,6 +63,11 @@ class CHROMEOS_EXPORT NetworkChangeNotifierChromeos
bool* ip_address_changed,
bool* dns_changed);
+ // Callback from Shill.Service.GetProperties. Parses |properties| to obtain
+ // the wifi access point information.
+ void ParseWifiAccessPointInfo(const std::string& service_path,
+ const base::DictionaryValue& properties);
+
// Maps the shill network type and technology to its NetworkChangeNotifier
// equivalent.
static net::NetworkChangeNotifier::ConnectionType
@@ -71,6 +80,8 @@ class CHROMEOS_EXPORT NetworkChangeNotifierChromeos
NetworkChangeCalculatorParamsChromeos();
NetworkChangeNotifier::ConnectionType connection_type_;
+ // Wifi access point info.
+ NetworkChangeNotifier::WifiAccessPointInfo wifi_ap_info_;
// IP address for the current default network.
std::string ip_address_;
// DNS servers for the current default network.

Powered by Google App Engine
This is Rietveld 408576698