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..e9c98aa6dd3cd2b015a2c09e7e764e256de23ad9 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,47 @@ 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* connected_network = nsh->ConnectedNetworkByType(
+      NetworkStateHandler::kMatchTypeNonVirtual);
+  const NetworkState* connecting_network = nsh->ConnectingNetworkByType(
+      NetworkStateHandler::kMatchTypeNonVirtual);

[stevenjb, 2013/08/07 19:10:19]

move this below the first if

+
+  if (connected_network) {
+    if (connected_network->type() == flimflam::kTypeEthernet)
+      return l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET);
+    else
+      return UTF8ToUTF16(connected_network->name());
+  } else if (connecting_network) {

[stevenjb, 2013/08/07 19:10:19]

no need for else here

+    if (connecting_network->type() == flimflam::kTypeEthernet)
+      return l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET);
+    else
+      return UTF8ToUTF16(connecting_network->name());
+  } else {

[stevenjb, 2013/08/07 19:10:19]

no need for 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