diff options
author | stevenjb@google.com <stevenjb@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-25 21:12:24 +0000 |
---|---|---|
committer | stevenjb@google.com <stevenjb@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-25 21:12:24 +0000 |
commit | 5e64ec4808fbc756722cdbc55fadb80f774ecf70 (patch) | |
tree | 5d96c908921b28e6a6383fe0c75ea195b97b145e | |
parent | adf96bbc161da1eeb6c1ba27ea0790f95ca4678a (diff) | |
download | chromium_src-5e64ec4808fbc756722cdbc55fadb80f774ecf70.zip chromium_src-5e64ec4808fbc756722cdbc55fadb80f774ecf70.tar.gz chromium_src-5e64ec4808fbc756722cdbc55fadb80f774ecf70.tar.bz2 |
Remove --enable-vpn (enabled always)
Include change to CrosMock::SetNetworkLibraryStatusAreaExpectations() so that tests pass.
Original CL+Review: http://codereview.chromium.org/7031040/
BUG=none
TEST=ChromeOS tests pass, 'Private networks' option should always show up in ChromeOS Networks menu.
Review URL: http://codereview.chromium.org/6992074
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@86703 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/browser/about_flags.cc | 12 | ||||
-rw-r--r-- | chrome/browser/chromeos/cros/cros_mock.cc | 3 | ||||
-rw-r--r-- | chrome/browser/chromeos/status/network_menu.cc | 24 | ||||
-rw-r--r-- | chrome/common/chrome_switches.cc | 3 | ||||
-rw-r--r-- | chrome/common/chrome_switches.h | 1 |
5 files changed, 14 insertions, 29 deletions
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc index a1291e3..15b61a2 100644 --- a/chrome/browser/about_flags.cc +++ b/chrome/browser/about_flags.cc @@ -296,18 +296,6 @@ const Experiment kExperiments[] = { SINGLE_VALUE_TYPE(switches::kPpapiFlashInProcess) }, { - "enable-vpn", - IDS_FLAGS_ENABLE_VPN_NAME, - IDS_FLAGS_ENABLE_VPN_DESCRIPTION, - kOsCrOS, -#if defined(OS_CHROMEOS) - // The switch exists only on Chrome OS. - SINGLE_VALUE_TYPE(switches::kEnableVPN) -#else - SINGLE_VALUE_TYPE("") -#endif - }, - { "multi-profiles", IDS_FLAGS_MULTI_PROFILES_NAME, IDS_FLAGS_MULTI_PROFILES_DESCRIPTION, diff --git a/chrome/browser/chromeos/cros/cros_mock.cc b/chrome/browser/chromeos/cros/cros_mock.cc index 34a54fe..246b8ec 100644 --- a/chrome/browser/chromeos/cros/cros_mock.cc +++ b/chrome/browser/chromeos/cros/cros_mock.cc @@ -276,6 +276,9 @@ void CrosMock::SetNetworkLibraryStatusAreaExpectations() { EXPECT_CALL(*mock_network_library_, virtual_networks()) .Times(AnyNumber()) .WillRepeatedly((ReturnRef(virtual_networks_))); + EXPECT_CALL(*mock_network_library_, connected_network()) + .Times(AnyNumber()) + .WillRepeatedly((Return((const Network*)(NULL)))); // Set specific expectations for interesting functions: diff --git a/chrome/browser/chromeos/status/network_menu.cc b/chrome/browser/chromeos/status/network_menu.cc index 0665312..2d06590 100644 --- a/chrome/browser/chromeos/status/network_menu.cc +++ b/chrome/browser/chromeos/status/network_menu.cc @@ -780,19 +780,17 @@ void MainMenuModel::InitMenuItems(bool is_browser_mode, SkBitmap(), std::string(), FLAG_DISABLED)); } - if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kEnableVPN)) { - // If there's a connected network, add submenu for Private Networks. - const Network* connected_network = cros->connected_network(); - if (connected_network) { - menu_items_.push_back(MenuItem()); // Separator - menu_items_.push_back(MenuItem( - ui::MenuModel::TYPE_SUBMENU, - l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_PRIVATE_NETWORKS), - VPNMenuModel::IconForDisplay(connected_network), - vpn_menu_model_.get(), FLAG_NONE)); - vpn_menu_model_->InitMenuItems( - is_browser_mode, should_open_button_options); - } + // If there's a connected network, add submenu for Private Networks. + const Network* connected_network = cros->connected_network(); + if (connected_network) { + menu_items_.push_back(MenuItem()); // Separator + menu_items_.push_back(MenuItem( + ui::MenuModel::TYPE_SUBMENU, + l10n_util::GetStringUTF16(IDS_STATUSBAR_NETWORK_PRIVATE_NETWORKS), + VPNMenuModel::IconForDisplay(connected_network), + vpn_menu_model_.get(), FLAG_NONE)); + vpn_menu_model_->InitMenuItems( + is_browser_mode, should_open_button_options); } // Enable / disable wireless. diff --git a/chrome/common/chrome_switches.cc b/chrome/common/chrome_switches.cc index 9db1623..f9a7190 100644 --- a/chrome/common/chrome_switches.cc +++ b/chrome/common/chrome_switches.cc @@ -1020,9 +1020,6 @@ const char kDOMLogin[] = "dom-login"; // Enables device policy support on ChromeOS. const char kEnableDevicePolicy[] = "enable-device-policy"; -// Enables VPN support on ChromeOS. -const char kEnableVPN[] = "enable-vpn"; - // Enable the redirection of viewable document requests to the Google // Document Viewer. const char kEnableGView[] = "enable-gview"; diff --git a/chrome/common/chrome_switches.h b/chrome/common/chrome_switches.h index d181ab8..03b67f2 100644 --- a/chrome/common/chrome_switches.h +++ b/chrome/common/chrome_switches.h @@ -282,7 +282,6 @@ extern const char kWinHttpProxyResolver[]; #if defined(OS_CHROMEOS) extern const char kDOMLogin[]; extern const char kEnableDevicePolicy[]; -extern const char kEnableVPN[]; extern const char kEnableGView[]; extern const char kEnableLoginImages[]; extern const char kLoginManager[]; |