summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorstevenjb@google.com <stevenjb@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2011-05-25 21:12:24 +0000
committerstevenjb@google.com <stevenjb@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2011-05-25 21:12:24 +0000
commit5e64ec4808fbc756722cdbc55fadb80f774ecf70 (patch)
tree5d96c908921b28e6a6383fe0c75ea195b97b145e
parentadf96bbc161da1eeb6c1ba27ea0790f95ca4678a (diff)
downloadchromium_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.cc12
-rw-r--r--chrome/browser/chromeos/cros/cros_mock.cc3
-rw-r--r--chrome/browser/chromeos/status/network_menu.cc24
-rw-r--r--chrome/common/chrome_switches.cc3
-rw-r--r--chrome/common/chrome_switches.h1
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[];