Fixed some logic bugs in the Network Menu DOM UI (7343)

chrome/browser/chromeos/status/network_menu.cc

 // Copyright (c) 2010 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/status/network_menu.h"
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/command_line.h"
 #include "base/stringprintf.h"
@@ skipping 78 matching lines @@
         } else {
           info->status = kNetworkStatusDisconnected;
           info->message = l10n_util::GetStringUTF8(
               IDS_STATUSBAR_NETWORK_DEVICE_DISCONNECTED);
         }
       } else {
         info->status = kNetworkStatusDisconnected;
         info->message = l10n_util::GetStringUTF8(
             IDS_STATUSBAR_NETWORK_DEVICE_DISCONNECTED);
       }
+      if (wifi.encrypted()) {
+        if (wifi.IsCertificateLoaded() ||
+            wifi.encryption() == SECURITY_8021X) {
+          info->need_passphrase = false;
+        } else {
+          info->need_passphrase = true;
+        }
+      } else {
+        info->need_passphrase = false;
+      }
       info->ip_address = wifi.ip_address();
-      info->need_passphrase = wifi.encrypted();
       info->remembered = wifi.favorite();
     } else {
       res = false;  // Network not found, hide entry.
     }
   } else if (flags & FLAG_CELLULAR) {
     CellularNetwork cellular;
     bool found = cros->FindCellularNetworkByPath(
         menu_items_[index].wireless_path, &cellular);
     if (found) {
       info->network_type = kNetworkTypeCellular;
@@ skipping 33 matching lines @@
         IDS_STATUSBAR_NETWORK_DEVICE_DISCONNECTED);
     info->network_type = kNetworkTypeOther;
     info->need_passphrase = true;
     info->remembered = true;
   } else {
     // Not a network, e.g options, separator.
   }
   return res;
 }
 
-void NetworkMenu::ConnectToNetworkAt(int index,
+bool NetworkMenu::ConnectToNetworkAt(int index,
                                      const std::string& passphrase,
-                                     const std::string& identity) const {
+                                     const std::string& ssid,
+                                     int remember) const {
   int flags = menu_items_[index].flags;
   NetworkLibrary* cros = CrosLibrary::Get()->GetNetworkLibrary();
   if (flags & FLAG_WIFI) {
     WifiNetwork wifi;
     bool found = cros->FindWifiNetworkByPath(
         menu_items_[index].wireless_path, &wifi);
     if (found) {
       // Connect or reconnect.
+      if (remember >= 0)
+        wifi.set_favorite(remember ? true : false);
       if (wifi.encrypted()) {
         if (wifi.IsCertificateLoaded()) {
-          cros->ConnectToWifiNetwork(wifi, std::string(),
-                                     std::string(), wifi.cert_path());
+          cros->ConnectToWifiNetwork(wifi, std::string(), std::string(),
+                                     wifi.cert_path());
+        } else if (wifi.encryption() == SECURITY_8021X) {
+          // Show the wifi settings/dialog to load/select a certificate.
+          ShowWifi(wifi, true);
         } else {
-          // TODO(stevenjb): use passphrase/identity once provided.
-          // cros->ConnectToWifiNetwork(wifi, passphrase,
-          //                            identity, menu_items[index].path);
-          const bool kFocusLogin = true;
-          ShowNetworkConfigView(new NetworkConfigView(wifi, true), kFocusLogin);
+          cros->ConnectToWifiNetwork(wifi, passphrase, std::string(),
+                                     std::string());
         }
       } else {
-        cros->ConnectToWifiNetwork(wifi, std::string(),
-                                   std::string(), std::string());
+        cros->ConnectToWifiNetwork(wifi, std::string(), std::string(),
+                                   std::string());
       }
     }
   } else if (flags & FLAG_CELLULAR) {
     CellularNetwork cellular;
     bool found = cros->FindCellularNetworkByPath(
         menu_items_[index].wireless_path, &cellular);
     if (found) {
       // Connect or reconnect.
       cros->ConnectToCellularNetwork(cellular);
     }
   } else if (flags & FLAG_OTHER_NETWORK) {
-    const bool kFocusLogin = true;
-    ShowNetworkConfigView(new NetworkConfigView(), kFocusLogin);
+    bool favorite = remember == 0 ? false : true;  // default is true
+    cros->ConnectToWifiNetwork(ssid, passphrase, std::string(), std::string(),
+                               favorite);
   }
+  return true;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkMenu, menus::MenuModel implementation:
 
 int NetworkMenu::GetItemCount() const {
   return static_cast<int>(menu_items_.size());
 }
 
 menus::MenuModel::ItemType NetworkMenu::GetTypeAt(int index) const {
@@ skipping 38 matching lines @@
     OpenButtonOptions();
   } else if (flags & FLAG_TOGGLE_ETHERNET) {
     cros->EnableEthernetNetworkDevice(!cros->ethernet_enabled());
   } else if (flags & FLAG_TOGGLE_WIFI) {
     cros->EnableWifiNetworkDevice(!cros->wifi_enabled());
   } else if (flags & FLAG_TOGGLE_CELLULAR) {
     cros->EnableCellularNetworkDevice(!cros->cellular_enabled());
   } else if (flags & FLAG_TOGGLE_OFFLINE) {
     cros->EnableOfflineMode(!cros->offline_mode());
   } else if (flags & FLAG_OTHER_NETWORK) {
-    const bool kFocusLogin = true;
-    ShowNetworkConfigView(new NetworkConfigView(), kFocusLogin);
+    ShowOther();
   } else if (flags & FLAG_ETHERNET) {
     if (cros->ethernet_connected()) {
-      if (CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kEnableTabbedOptions)) {
-        ShowTabbedNetworkSettings(cros->ethernet_network());
-      } else {
-        ShowNetworkConfigView(
-            new NetworkConfigView(cros->ethernet_network()), false);
-      }
+      ShowEthernet(cros->ethernet_network());
     }
   } else if (flags & FLAG_WIFI) {
     WifiNetwork wifi;
     bool wifi_exists = cros->FindWifiNetworkByPath(
         menu_items_[index].wireless_path, &wifi);
     if (!wifi_exists) {
       // If we are attempting to connect to a network that no longer exists,
       // display a notification.
       // TODO(stevenjb): Show notification.
     } else if (wifi.name() == cros->wifi_name()) {
-      if (cros->wifi_connected()) {
-        // If we are already connected, show the config settings.
-        if (CommandLine::ForCurrentProcess()->HasSwitch(
-            switches::kEnableTabbedOptions)) {
-          ShowTabbedNetworkSettings(wifi);
-        } else {
-          ShowNetworkConfigView(new NetworkConfigView(wifi, false), false);
-        }
-      } else {
-        // TODO(stevenjb): Connection in progress. Show dialog?
-      }
+      // Show the config settings for the active network.
+      ShowWifi(wifi, false);
     } else {
-      // If wifi network is not encrypted, then directly connect.
-      // Otherwise, open the password dialog window.
-      if (!wifi.encrypted()) {
-        cros->ConnectToWifiNetwork(wifi, std::string(),
-                                   std::string(), std::string());
-      } else {
-        const bool kFocusLogin = true;
-        ShowNetworkConfigView(new NetworkConfigView(wifi, true), kFocusLogin);
-      }
+      ConnectToNetworkAt(index, std::string(), std::string(), -1);
     }
   } else if (flags & FLAG_CELLULAR) {
     CellularNetwork cellular;
     bool cellular_exists = cros->FindCellularNetworkByPath(
         menu_items_[index].wireless_path, &cellular);
-
     if (!cellular_exists) {
       // If we are attempting to connect to a network that no longer exists,
       // display a notification.
       // TODO(stevenjb): Show notification.
     } else if (cellular.name() == cros->cellular_name()) {
-      // If clicked on a network that we are already connected to or we are
-      // currently trying to connect to, then open config dialog.
-      if (cros->cellular_connected()) {
-        if (CommandLine::ForCurrentProcess()->HasSwitch(
-            switches::kEnableTabbedOptions)) {
-          ShowTabbedNetworkSettings(cellular);
-        } else {
-          ShowNetworkConfigView(new NetworkConfigView(cellular), false);
-        }
-      } else {
-        // TODO(stevenjb): Connection in progress. Show dialog?
-      }
+      // Show the config settings for the cellular network.
+      ShowCellular(cellular, false);
     } else {
-      cros->ConnectToCellularNetwork(cellular);
+      ConnectToNetworkAt(index, std::string(), std::string(), -1);
     }
   }
 }
 
 void NetworkMenu::SetFirstLevelMenuWidth(int width) {
   min_width_ = width;
   // This actually has no effect since menu is rebuilt before showing.
   network_menu_->SetMinimumWidth(width);
 }
 
@@ skipping 190 matching lines @@
     // Network settings.
     if (ShouldOpenButtonOptions()) {
       label =
           l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_OPEN_OPTIONS_DIALOG);
       menu_items_.push_back(MenuItem(menus::MenuModel::TYPE_COMMAND, label,
           SkBitmap(), std::string(), FLAG_OPTIONS));
     }
   }
 }
 
-void NetworkMenu::ShowTabbedNetworkSettings(const Network& network) {
+void NetworkMenu::ShowTabbedNetworkSettings(const Network& network) const {
   Browser* browser = BrowserList::GetLastActive();
   if (!browser)
     return;
   std::string page = StringPrintf("%s?servicePath=%s&networkType=%d",
       chrome::kInternetOptionsSubPage,
       EscapeUrlEncodedData(network.service_path()).c_str(),
       network.type());
   browser->ShowOptionsTab(page);
 }
 
-// TODO(stevenjb): deprecate this once all of the UI is embedded in the menu.
+// TODO(stevenjb): deprecate this once we've committed to the embedded
+// menu UI and fully deprecated NetworkConfigView.
 void NetworkMenu::ShowNetworkConfigView(NetworkConfigView* view,
                                         bool focus_login) const {
   view->set_browser_mode(IsBrowserMode());
   views::Window* window = browser::CreateViewsWindow(
       GetNativeWindow(), gfx::Rect(), view);
   window->SetIsAlwaysOnTop(true);
   window->Show();
   if (focus_login)
     view->SetLoginTextfieldFocus();
 }
 
+void NetworkMenu::ShowWifi(const WifiNetwork& wifi, bool focus_login) const{
+  if (CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableTabbedOptions)) {
+    ShowTabbedNetworkSettings(wifi);
+  } else {
+    ShowNetworkConfigView(new NetworkConfigView(wifi, true), focus_login);
+  }
+}
+
+void NetworkMenu::ShowCellular(const CellularNetwork& cellular,
+                               bool focus_login) const {
+  if (CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableTabbedOptions)) {
+    ShowTabbedNetworkSettings(cellular);
+  } else {
+    ShowNetworkConfigView(new NetworkConfigView(cellular), focus_login);
+  }
+}
+
+void NetworkMenu::ShowEthernet(const EthernetNetwork& ethernet) const {
+  if (CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableTabbedOptions)) {
+    ShowTabbedNetworkSettings(ethernet);
+  } else {
+    ShowNetworkConfigView(new NetworkConfigView(ethernet), false);
+  }
+}
+
+void NetworkMenu::ShowOther() const {
+  if (CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableTabbedOptions)) {
+    Browser* browser = BrowserList::GetLastActive();
+    if (browser) {
+      std::string page = StringPrintf("%s?networkType=%d",
+                                      chrome::kInternetOptionsSubPage,
+                                      chromeos::TYPE_WIFI);
+      browser->ShowOptionsTab(page);
+    }
+  } else {
+    const bool kFocusLogin = true;
+    ShowNetworkConfigView(new NetworkConfigView(), kFocusLogin);
+  }
+}
+
 }  // namespace chromeos