Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1217)

Unified Diff: chrome/browser/ui/browser_command_controller_unittest.cc

Issue 294033002: Multi-profile flag cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Return --test-type flag back to profile_manager_unittest.cc Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/browser_command_controller_unittest.cc
diff --git a/chrome/browser/ui/browser_command_controller_unittest.cc b/chrome/browser/ui/browser_command_controller_unittest.cc
index 1b0eb27646d1693451cf86577ba595b4cfc44746..60ac52170faef588d77cea6cf7f31cb2e22a6d7e 100644
--- a/chrome/browser/ui/browser_command_controller_unittest.cc
+++ b/chrome/browser/ui/browser_command_controller_unittest.cc
@@ -172,12 +172,6 @@ TEST_F(BrowserCommandControllerTest, AppFullScreen) {
}
TEST_F(BrowserCommandControllerTest, OldAvatarMenuEnabledForOneOrMoreProfiles) {
-#if defined(OS_CHROMEOS)
- // TODO(nkostylev): Cleanup this code once multi-profiles are enabled by
- // default on CrOS. http://crbug.com/351655
- CommandLine::ForCurrentProcess()->AppendSwitch(switches::kMultiProfiles);
-#endif
-
if (!profiles::IsMultipleProfilesEnabled())
return;
@@ -213,12 +207,6 @@ TEST_F(BrowserCommandControllerTest, OldAvatarMenuEnabledForOneOrMoreProfiles) {
}
TEST_F(BrowserCommandControllerTest, NewAvatarMenuEnabledWhenOnlyOneProfile) {
-#if defined(OS_CHROMEOS)
- // TODO(nkostylev): Cleanup this code once multi-profiles are enabled by
- // default on CrOS. http://crbug.com/351655
- CommandLine::ForCurrentProcess()->AppendSwitch(switches::kMultiProfiles);
-#endif
-
if (!profiles::IsMultipleProfilesEnabled())
return;
@@ -247,12 +235,6 @@ TEST_F(BrowserCommandControllerTest, NewAvatarMenuEnabledWhenOnlyOneProfile) {
}
TEST_F(BrowserCommandControllerTest, NewAvatarMenuEnabledInGuestMode) {
-#if defined(OS_CHROMEOS)
- // TODO(nkostylev): Cleanup this code once multi-profiles are enabled by
- // default on CrOS. http://crbug.com/351655
- CommandLine::ForCurrentProcess()->AppendSwitch(switches::kMultiProfiles);
-#endif
-
if (!profiles::IsMultipleProfilesEnabled())
return;
@@ -290,12 +272,6 @@ TEST_F(BrowserCommandControllerTest, NewAvatarMenuEnabledInGuestMode) {
}
TEST_F(BrowserCommandControllerTest, AvatarMenuAlwaysDisabledInIncognitoMode) {
-#if defined(OS_CHROMEOS)
- // TODO(nkostylev): Cleanup this code once multi-profiles are enabled by
- // default on CrOS. http://crbug.com/351655
- CommandLine::ForCurrentProcess()->AppendSwitch(switches::kMultiProfiles);
-#endif
-
if (!profiles::IsMultipleProfilesEnabled())
return;

Powered by Google App Engine
This is Rietveld 408576698