AutomatedUITestBase.CloseTab fails on multiple platforms, not...

chrome/test/automated_ui_tests/automated_ui_test_test.cc

 // Copyright (c) 2009 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 "chrome/app/chrome_dll_resource.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/automated_ui_tests/automated_ui_test_base.h"
 #include "chrome/test/automation/browser_proxy.h"
 #include "chrome/test/automation/tab_proxy.h"
 #include "chrome/test/ui/ui_test.h"
 #include "googleurl/src/gurl.h"
 #include "net/base/net_util.h"
 
 #if defined(OS_MACOSX)
 // The window pops up, but doesn't close.
 #define MAYBE_IncognitoWindow DISABLED_IncognitoWindow
 #define MAYBE_OpenCloseBrowserWindowWithAccelerator \
     DISABLED_OpenCloseBrowserWindowWithAccelerator
 #else
 // http://code.google.com/p/chromium/issues/detail?id=14731
 #define MAYBE_IncognitoWindow DISABLED_IncognitoWindow
 // http://crbug.com/19394
 #define MAYBE_OpenCloseBrowserWindowWithAccelerator \
     DISABLED_OpenCloseBrowserWindowWithAccelerator
 #endif
 
-// http://code.google.com/p/chromium/issues/detail?id=14774
-#if defined(OS_WIN) && !defined(NDEBUG)
-#define MAYBE_CloseTab DISABLED_CloseTab
-#else
-#define MAYBE_CloseTab CloseTab
-#endif
-
 // Automation pieces are not implemented for these on platforms other than
 // Windows.
 #if defined(OS_WIN)
 #define MAYBE_ShowBookmarkBar ShowBookmarkBar
 #else
 #define MAYBE_ShowBookmarkBar DISABLED_ShowBookmarkBar
 #endif
 
 // FindBarTesting not implemented on mac.
 #if defined(OS_MACOSX)
@@ skipping 60 matching lines @@
   GURL test_url = net::FilePathToFileURL(path_prefix.AppendASCII("bot1.html"));
   GetActiveTab()->NavigateToURL(test_url);
   CloseActiveTab();
   active_browser()->GetTabCount(&tab_count);
   ASSERT_EQ(1, tab_count);
   RestoreTab();
   active_browser()->GetTabCount(&tab_count);
   ASSERT_EQ(2, tab_count);
 }
 
-TEST_F(AutomatedUITestBase, MAYBE_CloseTab) {
+// http://code.google.com/p/chromium/issues/detail?id=14774
+TEST_F(AutomatedUITestBase, FLAKY_CloseTab) {
   int num_browser_windows;
   int tab_count;
   NewTab();
   ASSERT_TRUE(automation()->GetBrowserWindowCount(&num_browser_windows));
   ASSERT_EQ(1, num_browser_windows);
   active_browser()->GetTabCount(&tab_count);
   ASSERT_EQ(2, tab_count);
 
   ASSERT_TRUE(OpenAndActivateNewBrowserWindow(NULL));
   NewTab();
@@ skipping 269 matching lines @@
   ASSERT_TRUE(GetActiveTab()->GetCurrentURL(&url));
   ASSERT_EQ(GURL(chrome::kChromeUIDownloadsURL), url);
 }
 
 TEST_F(AutomatedUITestBase, ShowHistory) {
   ASSERT_TRUE(ShowHistory());
   GURL url;
   ASSERT_TRUE(GetActiveTab()->GetCurrentURL(&url));
   ASSERT_EQ(GURL(chrome::kChromeUIHistoryURL), url);
 }