Unflaky tests.

chrome/browser/download/download_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 <sstream>
 #include <string>
 
 #include "build/build_config.h"
 #if defined(OS_WIN)
 #include <shlwapi.h>
@@ skipping 158 matching lines @@
 
     FAIL() << "Could not detect Internet ZoneIndentifier";
   }
 #endif  // defined(OS_WIN)
 
   FilePath download_prefix_;
 };
 
 // Download a file with non-viewable content, verify that the
 // download tab opened and the file exists.
 // This test is flaky. See bug 24889.

[Nicolas Sylvain, 2009/10/29 23:17:48]

remove the comment?

-TEST_F(DownloadTest, FLAKY_DownloadMimeType) {
+TEST_F(DownloadTest, DownloadMimeType) {
   FilePath file(FILE_PATH_LITERAL("download-test1.lib"));
 
   EXPECT_EQ(1, GetTabCount());
 
   NavigateToURL(URLRequestMockHTTPJob::GetMockUrl(file));
   // No new tabs created, downloads appear in the current tab's download shelf.
   WaitUntilTabCount(1);
 
   // Wait until the file is downloaded.
   PlatformThread::Sleep(action_timeout_ms());
@@ skipping 117 matching lines @@
   GURL url(URLRequestSlowDownloadJob::kKnownSizeUrl);
   FilePath filename;
   net::FileURLToFilePath(url, &filename);
   filename = filename.BaseName();
   RunSizeTest(url, L"71% - " + filename.ToWStringHack(),
               L"100% - " + filename.ToWStringHack());
 }
 
 // Test that when downloading an item in Incognito mode, we don't crash when
 // closing the last Incognito window (http://crbug.com/13983).
-// This test is flaky. See bug 24684.
-TEST_F(DownloadTest, FLAKY_IncognitoDownload) {
+TEST_F(DownloadTest, IncognitoDownload) {
   // Open a regular window and sanity check default values for window / tab
   // count and shelf visibility.
   scoped_refptr<BrowserProxy> browser(automation()->GetBrowserWindow(0));
   int window_count = 0;
   ASSERT_TRUE(automation()->GetBrowserWindowCount(&window_count));
   ASSERT_EQ(1, window_count);
   EXPECT_EQ(1, GetTabCount());
   bool is_shelf_visible;
   browser->IsShelfVisible(&is_shelf_visible);
   EXPECT_FALSE(is_shelf_visible);
@@ skipping 20 matching lines @@
   ASSERT_EQ(1, window_count);
 
   // Verify that the regular window does not have a download shelf.
   browser->IsShelfVisible(&is_shelf_visible);
   EXPECT_FALSE(is_shelf_visible);
 
   CleanUpDownload(file);
 }
 
 }  // namespace