diff options
author | noms@chromium.org <noms@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-28 17:47:25 +0000 |
---|---|---|
committer | noms@chromium.org <noms@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-28 17:47:25 +0000 |
commit | c933541fdb0c2535c6ca144943788a7826a9c60b (patch) | |
tree | 85046dc5d41d1844848ae6c821a41a8e7f223859 /chrome/browser/ui/browser_command_controller_unittest.cc | |
parent | 3b268bc49a01e5f26ceb5e407bc794881aee3b3d (diff) | |
download | chromium_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.cc | 13 |
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)); } |