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

Unified Diff: components/metrics/net/network_metrics_provider.h

Issue 636363004: Make remaining parts of components/metrics use metrics namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix cros compile Created 6 years, 2 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
« no previous file with comments | « components/metrics/metrics_state_manager.cc ('k') | components/metrics/net/network_metrics_provider.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/metrics/net/network_metrics_provider.h
diff --git a/components/metrics/net/network_metrics_provider.h b/components/metrics/net/network_metrics_provider.h
index 9ddb3f8855fa003d7a9e14889652521dc3f29ef6..412ef67e5ba1f399a2f8db86b24b8904e621586c 100644
--- a/components/metrics/net/network_metrics_provider.h
+++ b/components/metrics/net/network_metrics_provider.h
@@ -14,10 +14,12 @@
#include "net/base/net_util.h"
#include "net/base/network_change_notifier.h"
+namespace metrics {
+
// Registers as observer with net::NetworkChangeNotifier and keeps track of
// the network environment.
class NetworkMetricsProvider
- : public metrics::MetricsProvider,
+ : public MetricsProvider,
public net::NetworkChangeNotifier::ConnectionTypeObserver {
public:
// Creates a NetworkMetricsProvider, where |io_task_runner| is used to post
@@ -26,19 +28,17 @@ class NetworkMetricsProvider
~NetworkMetricsProvider() override;
private:
- // metrics::MetricsProvider:
+ // MetricsProvider:
void OnDidCreateMetricsLog() override;
- void ProvideSystemProfileMetrics(
- metrics::SystemProfileProto* system_profile) override;
+ void ProvideSystemProfileMetrics(SystemProfileProto* system_profile) override;
// ConnectionTypeObserver:
void OnConnectionTypeChanged(
net::NetworkChangeNotifier::ConnectionType type) override;
- metrics::SystemProfileProto::Network::ConnectionType
- GetConnectionType() const;
- metrics::SystemProfileProto::Network::WifiPHYLayerProtocol
- GetWifiPHYLayerProtocol() const;
+ SystemProfileProto::Network::ConnectionType GetConnectionType() const;
+ SystemProfileProto::Network::WifiPHYLayerProtocol GetWifiPHYLayerProtocol()
+ const;
// Posts a call to net::GetWifiPHYLayerProtocol on the blocking pool.
void ProbeWifiPHYLayerProtocol();
@@ -50,7 +50,7 @@ class NetworkMetricsProvider
// connected to.
void WriteWifiAccessPointProto(
const WifiAccessPointInfoProvider::WifiAccessPointInfo& info,
- metrics::SystemProfileProto::Network* network_proto);
+ SystemProfileProto::Network* network_proto);
// Task runner used for blocking file I/O.
base::TaskRunner* io_task_runner_;
@@ -74,4 +74,6 @@ class NetworkMetricsProvider
DISALLOW_COPY_AND_ASSIGN(NetworkMetricsProvider);
};
+} // namespace metrics
+
#endif // COMPONENTS_METRICS_NET_NETWORK_METRICS_PROVIDER_H_
« no previous file with comments | « components/metrics/metrics_state_manager.cc ('k') | components/metrics/net/network_metrics_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698