summaryrefslogtreecommitdiffstats
path: root/chrome/browser/ui/browser_command_controller_unittest.cc
diff options
context:
space:
mode:
authornoms@chromium.org <noms@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-07-28 17:47:25 +0000
committernoms@chromium.org <noms@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-07-28 17:47:25 +0000
commitc933541fdb0c2535c6ca144943788a7826a9c60b (patch)
tree85046dc5d41d1844848ae6c821a41a8e7f223859 /chrome/browser/ui/browser_command_controller_unittest.cc
parent3b268bc49a01e5f26ceb5e407bc794881aee3b3d (diff)
downloadchromium_src-c933541fdb0c2535c6ca144943788a7826a9c60b.zip
chromium_src-c933541fdb0c2535c6ca144943788a7826a9c60b.tar.gz
chromium_src-c933541fdb0c2535c6ca144943788a7826a9c60b.tar.bz2
Fix how we use the new profile management flags.
We are first going to release --new-avatar-menu, which is essentially what --new-profile-management used to be, minus the lock functionality. (--new-avatar-menu used to be an opt-in intermediate step, so functionality needs to be shuffled around. Basically this is a lot of s/new-profile-management/new-avatar-menu/g. BUG=391497 TEST=Start Chrome with --new-avatar-menu. You should see the new avatar menu, but not the lock button on any of the signed in profiles. Review URL: https://codereview.chromium.org/415643002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285942 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/browser_command_controller_unittest.cc')
-rw-r--r--chrome/browser/ui/browser_command_controller_unittest.cc13
1 files changed, 5 insertions, 8 deletions
diff --git a/chrome/browser/ui/browser_command_controller_unittest.cc b/chrome/browser/ui/browser_command_controller_unittest.cc
index 8f4b72b..e906fcf 100644
--- a/chrome/browser/ui/browser_command_controller_unittest.cc
+++ b/chrome/browser/ui/browser_command_controller_unittest.cc
@@ -175,7 +175,7 @@ TEST_F(BrowserCommandControllerTest, OldAvatarMenuEnabledForOneOrMoreProfiles) {
if (!profiles::IsMultipleProfilesEnabled())
return;
- EXPECT_FALSE(switches::IsNewProfileManagement());
+ EXPECT_FALSE(switches::IsNewAvatarMenu());
TestingProfileManager testing_profile_manager(
TestingBrowserProcess::GetGlobal());
@@ -211,8 +211,7 @@ TEST_F(BrowserCommandControllerTest, NewAvatarMenuEnabledWhenOnlyOneProfile) {
return;
// The command line is reset at the end of every test by the test suite.
- switches::EnableNewProfileManagementForTesting(
- CommandLine::ForCurrentProcess());
+ switches::EnableNewAvatarMenuForTesting(CommandLine::ForCurrentProcess());
TestingProfileManager testing_profile_manager(
TestingBrowserProcess::GetGlobal());
@@ -238,8 +237,7 @@ TEST_F(BrowserCommandControllerTest, NewAvatarMenuEnabledInGuestMode) {
return;
// The command line is reset at the end of every test by the test suite.
- switches::EnableNewProfileManagementForTesting(
- CommandLine::ForCurrentProcess());
+ switches::EnableNewAvatarMenuForTesting(CommandLine::ForCurrentProcess());
TestingProfileManager testing_profile_manager(
TestingBrowserProcess::GetGlobal());
@@ -287,13 +285,12 @@ TEST_F(BrowserCommandControllerTest, AvatarMenuAlwaysDisabledInIncognitoMode) {
const CommandUpdater* command_updater = command_controller.command_updater();
// The old style avatar menu should be disabled.
- EXPECT_FALSE(switches::IsNewProfileManagement());
+ EXPECT_FALSE(switches::IsNewAvatarMenu());
EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU));
// The new style avatar menu should also be disabled.
// The command line is reset at the end of every test by the test suite.
- switches::EnableNewProfileManagementForTesting(
- CommandLine::ForCurrentProcess());
+ switches::EnableNewAvatarMenuForTesting(CommandLine::ForCurrentProcess());
EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU));
}