Disable flaky ShowModalDialogTest.BasicTest

chrome/browser/browser_uitest.cc

 // Copyright (c) 2006-2008 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/file_path.h"
 #include "base/gfx/native_widget_types.h"
 #include "base/string_util.h"
 #include "base/sys_info.h"
 #include "base/values.h"
 #include "chrome/app/chrome_dll_resource.h"
@@ skipping 194 matching lines @@
 #endif
 
 #if defined(OS_WIN)  // only works on Windows for now: http:://crbug.com/15891
 class ShowModalDialogTest : public UITest {
  public:
   ShowModalDialogTest() {
     launch_arguments_.AppendSwitch(switches::kDisablePopupBlocking);
   }
 };
 
-TEST_F(ShowModalDialogTest, BasicTest) {
+// So flaky, http://crbug.com/17806. Do *not* re-enable without a real fix.
+// Increasing timeouts is not a fix.
+TEST_F(ShowModalDialogTest, DISABLED_BasicTest) {
   // Test that a modal dialog is shown.
   FilePath test_file(test_data_directory_);
   test_file = test_file.AppendASCII("showmodaldialog.html");
   NavigateToURL(net::FilePathToFileURL(test_file));
 
   ASSERT_TRUE(automation()->WaitForWindowCountToBecome(
       2, action_max_timeout_ms()));
 
   scoped_refptr<BrowserProxy> browser = automation()->GetBrowserWindow(1);
   scoped_refptr<TabProxy> tab = browser->GetActiveTab();
   ASSERT_TRUE(tab.get());
 
   std::wstring title;
   ASSERT_TRUE(tab->GetTabTitle(&title));
   ASSERT_EQ(title, L"ModalDialogTitle");
 
   // Test that window.close() works.  Since we don't have a way of executing a
   // JS function on the page through TabProxy, reload it and use an unload
   // handler that closes the page.
   ASSERT_EQ(tab->Reload(), AUTOMATION_MSG_NAVIGATION_SUCCESS);
   ASSERT_TRUE(automation()->WaitForWindowCountToBecome(
       1, action_max_timeout_ms()));
 }
 #endif
 
 }  // namespace