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

Unified Diff: chromeos/network/network_state.cc

Issue 11614035: Improve NetworkStateHandler API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Separate out OnDefaultNetworkChanged and move kMatchType* to NetworkStateHandler Created 8 years 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_state.cc
diff --git a/chromeos/network/network_state.cc b/chromeos/network/network_state.cc
index c87bdcb4763c9fdef56f09f60e16360a7e83b9d1..830e9938202368ab18ca655bc6dbde8bb26b4385 100644
--- a/chromeos/network/network_state.cc
+++ b/chromeos/network/network_state.cc
@@ -24,7 +24,7 @@ bool NetworkState::PropertyChanged(const std::string& key,
if (key == flimflam::kSignalStrengthProperty) {
return GetIntegerValue(key, value, &signal_strength_);
} else if (key == flimflam::kStateProperty) {
- return GetStringValue(key, value, &state_);
+ return GetStringValue(key, value, &connection_state_);
} else if (key == flimflam::kErrorProperty) {
return GetStringValue(key, value, &error_);
} else if (key == flimflam::kActivationStateProperty) {
@@ -42,25 +42,25 @@ bool NetworkState::PropertyChanged(const std::string& key,
}
bool NetworkState::IsConnectedState() const {
- return StateIsConnected(state_);
+ return StateIsConnected(connection_state_);
}
bool NetworkState::IsConnectingState() const {
- return StateIsConnecting(state_);
+ return StateIsConnecting(connection_state_);
}
// static
-bool NetworkState::StateIsConnected(const std::string& state) {
- return (state == flimflam::kStateReady ||
- state == flimflam::kStateOnline ||
- state == flimflam::kStatePortal);
+bool NetworkState::StateIsConnected(const std::string& connection_state) {
+ return (connection_state == flimflam::kStateReady ||
+ connection_state == flimflam::kStateOnline ||
+ connection_state == flimflam::kStatePortal);
}
// static
-bool NetworkState::StateIsConnecting(const std::string& state) {
- return (state == flimflam::kStateAssociation ||
- state == flimflam::kStateConfiguration ||
- state == flimflam::kStateCarrier);
+bool NetworkState::StateIsConnecting(const std::string& connection_state) {
+ return (connection_state == flimflam::kStateAssociation ||
+ connection_state == flimflam::kStateConfiguration ||
+ connection_state == flimflam::kStateCarrier);
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698