Disable ProfileListDesktopBrowserTest on Chrome OS since it has different multi-profiles UI

chrome/browser/profiles/profile_list_desktop_browsertest.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/avatar_menu.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/profiles/profile_window.h"
@@ skipping 31 matching lines @@
         browser()));
     return avatar_menu_.get();
   }
 
  private:
   scoped_ptr<AvatarMenu> avatar_menu_;
 
   DISALLOW_COPY_AND_ASSIGN(ProfileListDesktopBrowserTest);
 };
 
-#if defined (OS_WIN)
+#if defined(OS_WIN)
 // SignOut is flaky. So far only observed on Windows. crbug.com/357329.
 #define MAYBE_SignOut DISABLED_SignOut
+#elif defined(OS_CHROMEOS)

[rpetterson, 2014/05/12 17:53:55]

I've always seen that the disables are combined and then the comment is expanded
to explain the situations, but this is equally clear.

+// This test doesn't make sense for Chrome OS since it has a different
+// multi-profiles menu in the system tray instead.
+#define MAYBE_SignOut DISABLED_SignOut
 #else
 #define MAYBE_SignOut SignOut
 #endif
 IN_PROC_BROWSER_TEST_F(ProfileListDesktopBrowserTest, MAYBE_SignOut) {
   if (!profiles::IsMultipleProfilesEnabled())
     return;
 
   ProfileManager* profile_manager = g_browser_process->profile_manager();
   Profile* current_profile = browser()->profile();
   ProfileInfoCache& cache = profile_manager->GetProfileInfoCache();
@@ skipping 13 matching lines @@
   profiles::LockProfile(current_profile);
   window_close_observer.Wait();  // rely on test time-out for failure indication
 
   EXPECT_TRUE(cache.ProfileIsSigninRequiredAtIndex(index));
   EXPECT_EQ(0U, browser_list->size());
 
   // Signing out brings up the User Manager which we should close before exit.
   chrome::HideUserManager();
 }
 
-IN_PROC_BROWSER_TEST_F(ProfileListDesktopBrowserTest, SwitchToProfile) {
+#if defined(OS_CHROMEOS)
+// This test doesn't make sense for Chrome OS since it has a different
+// multi-profiles menu in the system tray instead.
+#define MAYBE_SwitchToProfile DISABLED_SwitchToProfile
+#else
+#define MAYBE_SwitchToProfile SwitchToProfile
+#endif
+IN_PROC_BROWSER_TEST_F(ProfileListDesktopBrowserTest, MAYBE_SwitchToProfile) {
 #if defined(OS_WIN) && defined(USE_ASH)
   // Disable this test in Metro+Ash for now (http://crbug.com/262796).
   if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kAshBrowserTests))
     return;
 #endif
 
   if (!profiles::IsMultipleProfilesEnabled())
     return;
 
   ProfileManager* profile_manager = g_browser_process->profile_manager();
@@ skipping 33 matching lines @@
                         false, ProfileMetrics::SWITCH_PROFILE_ICON);
   EXPECT_EQ(2U, browser_list->size());
 
   // Switch to the first profile without opening a new window.
   menu->SwitchToProfile(cache.GetIndexOfProfileWithPath(path_profile1),
                         false, ProfileMetrics::SWITCH_PROFILE_ICON);
   EXPECT_EQ(2U, browser_list->size());
   EXPECT_EQ(path_profile1, browser_list->get(0)->profile()->GetPath());
   EXPECT_EQ(path_profile2, browser_list->get(1)->profile()->GetPath());
 }