Update screen is skipped even if user clicks on Ethernet.

chrome/browser/chromeos/status/network_menu.cc

 // Copyright (c) 2012 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 <algorithm>
 
 #include "ash/shell.h"
 #include "ash/shell_delegate.h"
@@ skipping 196 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(MainMenuModel);
 };
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkMenuModel, public methods:
 
 void NetworkMenuModel::ConnectToNetworkAt(int index) {
   const std::string& service_path = menu_items_[index].service_path;
   gfx::NativeWindow native_window = owner_->delegate()->GetNativeWindow();
   ash::network_connect::ConnectToNetwork(service_path, native_window);
-  owner_->delegate()->OnConnectToNetworkRequested(service_path);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkMenuModel, ui::MenuModel implementation:
 
 bool NetworkMenuModel::HasIcons() const {
   return true;
 }
 
 int NetworkMenuModel::GetItemCount() const {
@@ skipping 75 matching lines @@
     return;
 
   int flags = menu_items_[index].flags;
   if (flags & FLAG_OPTIONS) {
     owner_->delegate()->OpenButtonOptions();
   } else if (flags & FLAG_TOGGLE_WIFI) {
     ToggleTechnology(flimflam::kTypeWifi);
   } else if (flags & FLAG_TOGGLE_MOBILE) {
     ToggleTechnology(NetworkStateHandler::kMatchTypeMobile);
   } else if (flags & FLAG_ETHERNET) {
-    // Do nothing (used in login screen only)
+    owner_->delegate()->OnConnectToNetworkRequested(
+        menu_items_[index].service_path);
   } else if (flags & (FLAG_WIFI | FLAG_WIMAX | FLAG_CELLULAR)) {
     ConnectToNetworkAt(index);
+    owner_->delegate()->OnConnectToNetworkRequested(
+        menu_items_[index].service_path);
   } else if (flags & FLAG_ADD_WIFI) {
     ShowOther(flimflam::kTypeWifi);
   } else if (flags & FLAG_ADD_CELLULAR) {
     ShowOther(flimflam::kTypeCellular);
   }
 }
 
 void NetworkMenuModel::SetMenuModelDelegate(ui::MenuModelDelegate* delegate) {
 }
 
@@ skipping 343 matching lines @@
   return main_menu_model_.get();
 }
 
 void NetworkMenu::UpdateMenu() {
   refreshing_menu_ = true;
   main_menu_model_->InitMenuItems(delegate_->ShouldOpenButtonOptions());
   refreshing_menu_ = false;
 }
 
 }  // namespace chromeos