Remove ConnectivityStateHelper

chrome/browser/chromeos/login/helper.cc

Index: chrome/browser/chromeos/login/helper.cc
diff --git a/chrome/browser/chromeos/login/helper.cc b/chrome/browser/chromeos/login/helper.cc
index ff60abef99e245ce65ff0faee50b44724b64302e..a0c1d5c153960d5645b6db0f251e8173b98355ac 100644
--- a/chrome/browser/chromeos/login/helper.cc
+++ b/chrome/browser/chromeos/login/helper.cc
@@ -6,7 +6,9 @@
 
 #include "ash/shell.h"
 #include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/chromeos/net/connectivity_state_helper.h"
+#include "chromeos/network/network_handler.h"
+#include "chromeos/network/network_state.h"
+#include "chromeos/network/network_state_handler.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
@@ -26,31 +28,6 @@ gfx::Rect CalculateScreenBounds(const gfx::Size& size) {
   return bounds;
 }
 
-string16 GetCurrentNetworkName() {
-  ConnectivityStateHelper* csh =
-      ConnectivityStateHelper::Get();
-
-  if (csh->IsConnectedType(flimflam::kTypeEthernet)) {
-    return l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET);
-  } else if (csh->IsConnectedType(flimflam::kTypeWifi)) {
-    return UTF8ToUTF16(csh->NetworkNameForType(flimflam::kTypeWifi));
-  } else if (csh->IsConnectedType(flimflam::kTypeCellular)) {
-    return UTF8ToUTF16(csh->NetworkNameForType(flimflam::kTypeCellular));
-  } else if (csh->IsConnectedType(flimflam::kTypeWimax)) {
-    return UTF8ToUTF16(csh->NetworkNameForType(flimflam::kTypeWimax));
-  } else if (csh->IsConnectingType(flimflam::kTypeEthernet)) {
-    return l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET);
-  } else if (csh->IsConnectingType(flimflam::kTypeWifi)) {
-    return UTF8ToUTF16(csh->NetworkNameForType(flimflam::kTypeWifi));
-  } else if (csh->IsConnectingType(flimflam::kTypeCellular)) {
-    return UTF8ToUTF16(csh->NetworkNameForType(flimflam::kTypeCellular));
-  } else if (csh->IsConnectingType(flimflam::kTypeWimax)) {
-    return UTF8ToUTF16(csh->NetworkNameForType(flimflam::kTypeWimax));
-  } else {
-    return string16();
-  }
-}
-
 int GetCurrentUserImageSize() {
   // The biggest size that the profile picture is displayed at is currently
   // 220px, used for the big preview on OOBE and Change Picture options page.
@@ -62,4 +39,52 @@ int GetCurrentUserImageSize() {
       ui::GetScaleFactorScale(ui::GetMaxScaleFactor());
 }
 
+namespace login {
+
+NetworkStateHelper::NetworkStateHelper() {}
+NetworkStateHelper::~NetworkStateHelper() {}
+
+string16 NetworkStateHelper::GetCurrentNetworkName() const {
+  NetworkStateHandler* nsh = NetworkHandler::Get()->network_state_handler();
+  const NetworkState* network = NULL;
+
+  if (nsh->ConnectedNetworkByType(flimflam::kTypeEthernet)) {
+    return l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET);
+  } else if ((network = nsh->ConnectedNetworkByType(flimflam::kTypeWifi))) {
+    return UTF8ToUTF16(network->name());
+  } else if ((network = nsh->ConnectedNetworkByType(flimflam::kTypeCellular))) {
+    return UTF8ToUTF16(network->name());
+  } else if ((network = nsh->ConnectedNetworkByType(flimflam::kTypeWimax))) {
+    return UTF8ToUTF16(network->name());

[stevenjb, 2013/08/07 01:43:27]

I believe the above 4 could be replaced by a single call using
kMatchTypeNonVirtual (Shill uses the same ordering and we're probably better off
using Shill's prioritization anyway)

[gauravsh, 2013/08/07 17:28:19]

Done.

+  } else if ((network = nsh->ConnectingNetworkByType(
+      flimflam::kTypeEthernet))) {
+    return l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET);
+  } else if ((network = nsh->ConnectingNetworkByType(flimflam::kTypeWifi))) {
+    return UTF8ToUTF16(network->name());
+  } else if ((network = nsh->ConnectingNetworkByType(
+      flimflam::kTypeCellular))) {
+    return UTF8ToUTF16(network->name());
+  } else if ((network = nsh->ConnectingNetworkByType(flimflam::kTypeWimax))) {
+    return UTF8ToUTF16(network->name());

[stevenjb, 2013/08/07 01:43:27]

Ditto for Connecting

[gauravsh, 2013/08/07 17:28:19]

Done.

+  } else {
+    return string16();
+  }
+}
+
+bool NetworkStateHelper::IsConnected() const {
+  chromeos::NetworkStateHandler* nsh =
+      chromeos::NetworkHandler::Get()->network_state_handler();
+  return nsh->ConnectedNetworkByType(
+      chromeos::NetworkStateHandler::kMatchTypeDefault) != NULL;
+}
+
+bool NetworkStateHelper::IsConnecting() const {
+  chromeos::NetworkStateHandler* nsh =
+      chromeos::NetworkHandler::Get()->network_state_handler();
+  return nsh->ConnectingNetworkByType(
+      chromeos::NetworkStateHandler::kMatchTypeDefault) != NULL;
+}
+
+}  // namespace login
+
 }  // namespace chromeos