Unify ActivateCellular into network_connect.cc

chrome/browser/chromeos/options/network_connect.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/options/network_connect.h"
 
 #include "ash/shell.h"
 #include "ash/shell_delegate.h"
+#include "ash/system/chromeos/network/network_observer.h"
+#include "ash/system/tray/system_tray_notifier.h"
+#include "base/utf_string_conversions.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/cros/network_library.h"
 #include "chrome/browser/chromeos/enrollment_dialog_view.h"
 #include "chrome/browser/chromeos/options/network_config_view.h"
 #include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/webui/chromeos/mobile_setup_dialog.h"
+#include "grit/generated_resources.h"
+#include "third_party/cros_system_api/dbus/service_constants.h"
+#include "ui/base/l10n/l10n_util.h"
 
 namespace chromeos {
 namespace network_connect {
 
 namespace {
 
-void ActivateCellular(chromeos::CellularNetwork* cellular) {
-  DCHECK(cellular);
-  chromeos::NetworkLibrary* cros =
-      chromeos::CrosLibrary::Get()->GetNetworkLibrary();
-  if (cros->CellularDeviceUsesDirectActivation()) {
-    cellular->StartActivation();
-  } else {
-    ash::Shell::GetInstance()->delegate()->OpenMobileSetup(
-        cellular->service_path());
-  }
-}
-
 void DoConnect(Network* network, gfx::NativeWindow parent_window) {
   NetworkLibrary* cros = CrosLibrary::Get()->GetNetworkLibrary();
   if (network->type() == TYPE_VPN) {
     VirtualNetwork* vpn = static_cast<VirtualNetwork*>(network);
     if (vpn->NeedMoreInfoToConnect()) {
       // Show the connection UI if info for a field is missing.
       NetworkConfigView::Show(vpn, parent_window);
     } else {
       cros->ConnectToVirtualNetwork(vpn);
       // Connection failures are responsible for updating the UI, including
@@ skipping 16 matching lines @@
       NetworkConfigView::Show(wimax, parent_window);
     } else {
       cros->ConnectToWimaxNetwork(wimax);
       // Connection failures are responsible for updating the UI, including
       // reopening dialogs.
     }
   } else if (network->type() == TYPE_CELLULAR) {
     CellularNetwork* cellular = static_cast<CellularNetwork*>(network);
     if (cellular->activation_state() != ACTIVATION_STATE_ACTIVATED ||
         cellular->out_of_credits()) {
-      ActivateCellular(cellular);
+      ActivateCellular(cellular->service_path());
     } else {
       cros->ConnectToCellularNetwork(cellular);
     }
   }
 }
 
 }  // namespace
 
+void ActivateCellular(const std::string& service_path) {
+  chromeos::NetworkLibrary* cros =
+      chromeos::CrosLibrary::Get()->GetNetworkLibrary();
+  if (!cros->CellularDeviceUsesDirectActivation()) {
+    // For non direct activation, show the mobile setup dialog which can be
+    // used to activate the network.
+    ShowMobileSetup(service_path);
+    return;
+  }
+  chromeos::Network* network = NULL;
+  network = cros-> FindNetworkByPath(service_path);

[tbarzic, 2013/05/24 21:24:34]

FindCellularNetworkByPath? That way you don't have to check network type or use
cast. Also, I'd merge this with the previous line.

+ remove extra space after ->

[stevenjb, 2013/05/24 22:17:45]

Yes, done (copy/pasted old code).

+  if (!network || network->type() != chromeos::TYPE_CELLULAR)
+    return;
+  chromeos::CellularNetwork* cellular =
+      static_cast<chromeos::CellularNetwork*>(network);
+  if (cellular->activation_state() != chromeos::ACTIVATION_STATE_ACTIVATED)
+    cellular->StartActivation();
+  return;
+}
+
+void ShowMobileSetup(const std::string& service_path) {
+  NetworkLibrary* cros = CrosLibrary::Get()->GetNetworkLibrary();
+  const CellularNetwork* cellular =
+      cros->FindCellularNetworkByPath(service_path);
+  if (cellular && !cellular->activated() &&
+      cellular->activate_over_non_cellular_network() &&
+      (!cros->connected_network() || !cros->connected_network()->online())) {
+    NetworkTechnology technology = cellular->network_technology();
+    ash::NetworkObserver::NetworkType network_type =
+        (technology == chromeos::NETWORK_TECHNOLOGY_LTE ||
+         technology == chromeos::NETWORK_TECHNOLOGY_LTE_ADVANCED)
+        ? ash::NetworkObserver::NETWORK_CELLULAR_LTE
+        : ash::NetworkObserver::NETWORK_CELLULAR;
+    ash::Shell::GetInstance()->system_tray_notifier()->NotifySetNetworkMessage(
+        NULL,
+        ash::NetworkObserver::ERROR_CONNECT_FAILED,
+        network_type,
+        l10n_util::GetStringUTF16(IDS_NETWORK_ACTIVATION_ERROR_TITLE),
+        l10n_util::GetStringFUTF16(IDS_NETWORK_ACTIVATION_NEEDS_CONNECTION,
+                                   UTF8ToUTF16((cellular->name()))),
+        std::vector<string16>());
+    return;
+  }
+  MobileSetupDialog::Show(service_path);
+}
+
 ConnectResult ConnectToNetwork(const std::string& service_path,
                                gfx::NativeWindow parent_window) {
   NetworkLibrary* cros = CrosLibrary::Get()->GetNetworkLibrary();
   Network* network = cros->FindNetworkByPath(service_path);
   if (!network)
     return NETWORK_NOT_FOUND;
 
   if (network->connecting_or_connected())
     return CONNECT_NOT_STARTED;
 
@@ skipping 13 matching lines @@
 
   if (network->type() == TYPE_WIMAX) {
     WimaxNetwork* wimax = static_cast<WimaxNetwork*>(network);
     wimax->AttemptConnection(base::Bind(&DoConnect, wimax, parent_window));
     return CONNECT_STARTED;
   }
 
   if (network->type() == TYPE_CELLULAR) {
     CellularNetwork* cellular = static_cast<CellularNetwork*>(network);
     if (cellular->NeedsActivation() || cellular->out_of_credits()) {
-      ActivateCellular(cellular);
+      ActivateCellular(service_path);
       return CONNECT_STARTED;
     }
     if (cellular->activation_state() == ACTIVATION_STATE_ACTIVATING)
       return CONNECT_NOT_STARTED;
     cros->ConnectToCellularNetwork(cellular);
     return CONNECT_STARTED;
   }
 
   if (network->type() == TYPE_VPN) {
     VirtualNetwork* vpn = static_cast<VirtualNetwork*>(network);
     vpn->SetEnrollmentDelegate(
         chromeos::CreateEnrollmentDelegate(
             parent_window,
             vpn->name(),
             ProfileManager::GetLastUsedProfile()));
     vpn->AttemptConnection(base::Bind(&DoConnect, vpn, parent_window));
     return CONNECT_STARTED;
   }
 
   NOTREACHED();
   return CONNECT_NOT_STARTED;
 }
 
 }  // namespace network_connect
 }  // namespace chromeos