Add a bunch of missing bug references to FLAKY tests

BUG=None
TEST=N/A

Review URL: http://codereview.chromium.org/4135007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64150 0039d316-1c4b-4281-b951-d872f2087c98

12 files changed, 14 insertions, 4 deletions

diff --git a/chrome/browser/appcache/appcache_ui_test.cc b/chrome/browser/appcache/appcache_ui_test.cc
index c1000bd..a4814cf 100644
--- a/chrome/browser/appcache/appcache_ui_test.cc
+++ b/chrome/browser/appcache/appcache_ui_test.cc
@@ -10,6 +10,7 @@ class AppCacheUITest : public UILayoutTest {
   virtual ~AppCacheUITest() {}
 };
 
+// Flaky: http://crbug.com/54717
 TEST_F(AppCacheUITest, FLAKY_AppCacheLayoutTests) {
   static const char* kLayoutTestFiles[] = {
       "404-manifest.html",
diff --git a/chrome/browser/browser_init_browsertest.cc b/chrome/browser/browser_init_browsertest.cc
index ec52cae..37e2c47 100644
--- a/chrome/browser/browser_init_browsertest.cc
+++ b/chrome/browser/browser_init_browsertest.cc
@@ -65,6 +65,7 @@ IN_PROC_BROWSER_TEST_F(BrowserInitTest, OpenURLsPopup) {
 // TODO(jschuh): FLAKY because the process doesn't have sufficient time
 // to start on most BuildBot runs and I don't want to add longer delays to
 // the test. I'll circle back and make this work properly when i get a chance.
+// See http://crbug.com/60026
 IN_PROC_BROWSER_TEST_F(BrowserInitTest, FLAKY_BlockBadURLs) {
   const char* testurlstr = "http://localhost/";
   const GURL testurl(testurlstr);
diff --git a/chrome/browser/extensions/extension_uitest.cc b/chrome/browser/extensions/extension_uitest.cc
index 2076949..f074c9b 100644
--- a/chrome/browser/extensions/extension_uitest.cc
+++ b/chrome/browser/extensions/extension_uitest.cc
@@ -121,6 +121,7 @@ class ExtensionTestSimpleApiCall : public ExtensionUITest {
   DISALLOW_COPY_AND_ASSIGN(ExtensionTestSimpleApiCall);
 };
 
+// Flaky: http://crbug.com/44599
 TEST_F(ExtensionTestSimpleApiCall, FLAKY_RunTest) {
   namespace keys = extension_automation_constants;
 
diff --git a/chrome/browser/session_history_uitest.cc b/chrome/browser/session_history_uitest.cc
index 278672a..0c1332f 100644
--- a/chrome/browser/session_history_uitest.cc
+++ b/chrome/browser/session_history_uitest.cc
@@ -139,8 +139,8 @@ TEST_F(SessionHistoryTest, BasicBackForward) {
 }
 
 // Test that back/forward works when navigating in subframes.
-// Fails on Windows. See crbug.com/TODO
 #if defined(OS_WIN)
+// Flaky on Windows. See crbug.com/48833
 #define MAYBE_FrameBackForward FLAKY_FrameBackForward
 #else
 #define MAYBE_FrameBackForward FrameBackForward
diff --git a/chrome/browser/sessions/session_restore_uitest.cc b/chrome/browser/sessions/session_restore_uitest.cc
index 9e56711..b91ae93 100644
--- a/chrome/browser/sessions/session_restore_uitest.cc
+++ b/chrome/browser/sessions/session_restore_uitest.cc
@@ -378,6 +378,7 @@ TEST_F(SessionRestoreUITest, DontRestoreWhileIncognito) {
 
 // Launches an app window, closes tabbed browser, launches and makes sure
 // we restore the tabbed browser url.
+// Flaky: http://crbug.com/29110
 TEST_F(SessionRestoreUITest,
        FLAKY_RestoreAfterClosingTabbedBrowserWithAppAndLaunching) {
   NavigateToURL(url1_);
diff --git a/chrome/browser/tab_restore_uitest.cc b/chrome/browser/tab_restore_uitest.cc
index 4e48d5a..5764dde 100644
--- a/chrome/browser/tab_restore_uitest.cc
+++ b/chrome/browser/tab_restore_uitest.cc
@@ -208,7 +208,7 @@ TEST_F(TabRestoreUITest, MiddleTab) {
 
 // Close a tab, switch windows, then restore the tab. The tab should be in its
 // original window and position, and active.
-// This test is flaky. See http://crbug.com/14132 and 11213.
+// This test is flaky. See http://crbug.com/54894
 TEST_F(TabRestoreUITest, FLAKY_RestoreToDifferentWindow) {
   scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0));
   ASSERT_TRUE(browser_proxy.get());
diff --git a/chrome/browser/unload_uitest.cc b/chrome/browser/unload_uitest.cc
index 3ddd9b0..6abc736 100644
--- a/chrome/browser/unload_uitest.cc
+++ b/chrome/browser/unload_uitest.cc
@@ -324,7 +324,7 @@ TEST_F(UnloadTest, BrowserCloseBeforeUnloadCancel) {
 }
 
 #if defined(OS_LINUX)
-// Fails sometimes on Linux valgrind. http://crbug.com/32615
+// Fails sometimes on Linux valgrind. http://crbug.com/45675
 #define MAYBE_BrowserCloseWithInnerFocusedFrame \
     FLAKY_BrowserCloseWithInnerFocusedFrame
 #else
@@ -414,7 +414,8 @@ TEST_F(UnloadTest, BrowserCloseTwoSecondBeforeUnloadAlert) {
 #define MAYBE_BrowserCloseTabWhenOtherTabHasListener \
     DISABLED_BrowserCloseTabWhenOtherTabHasListener
 #else
-// Flaky on Linux as well. http://crbug.com/45562
+// Flaky on Linux under valgrind. http://crbug.com/46781
+// TODO(stuartmorgan): Switch to just disabling for valgrind.
 #define MAYBE_BrowserCloseTabWhenOtherTabHasListener \
     FLAKY_BrowserCloseTabWhenOtherTabHasListener
 #endif
diff --git a/chrome/test/plugin/plugin_test.cpp b/chrome/test/plugin/plugin_test.cpp
index a0afffa..c9b7027 100644
--- a/chrome/test/plugin/plugin_test.cpp
+++ b/chrome/test/plugin/plugin_test.cpp
@@ -193,6 +193,7 @@ TEST_F(PluginTest, FlashSecurity) {
 #if defined(OS_WIN)
 // TODO(port) Port the following tests to platforms that have the required
 // plugins.
+// Flaky: http://crbug.com/55915
 TEST_F(PluginTest, FLAKY_Quicktime) {
   TestPlugin("quicktime.html", action_max_timeout_ms(), false);
 }
diff --git a/chrome/test/ui/layout_plugin_uitest.cc b/chrome/test/ui/layout_plugin_uitest.cc
index 8df91c2..e309937 100644
--- a/chrome/test/ui/layout_plugin_uitest.cc
+++ b/chrome/test/ui/layout_plugin_uitest.cc
@@ -45,6 +45,7 @@ TEST_F(LayoutPluginTester, UnloadNoCrash) {
 
 // Tests if a plugin executing a self deleting script using NPN_GetURL
 // works without crashing or hanging
+// Flaky: http://crbug.com/59327
 TEST_F(LayoutPluginTester, FLAKY_SelfDeletePluginGetUrl) {
   const FilePath test_case(FILE_PATH_LITERAL("self_delete_plugin_geturl.html"));
   GURL url = ui_test_utils::GetTestUrl(FilePath(kTestDir), test_case);
diff --git a/chrome/worker/worker_uitest.cc b/chrome/worker/worker_uitest.cc
index 9550dc7..4b34b7e 100644
--- a/chrome/worker/worker_uitest.cc
+++ b/chrome/worker/worker_uitest.cc
@@ -202,6 +202,7 @@ TEST_F(WorkerTest, MultipleSharedWorkers) {
 }
 
 #if defined(OS_LINUX)
+// http://crbug.com/30021
 #define IncognitoSharedWorkers FLAKY_IncognitoSharedWorkers
 #endif
 
diff --git a/net/socket/ssl_client_socket_unittest.cc b/net/socket/ssl_client_socket_unittest.cc
index bfcbe81..0e9070a 100644
--- a/net/socket/ssl_client_socket_unittest.cc
+++ b/net/socket/ssl_client_socket_unittest.cc
@@ -181,6 +181,7 @@ TEST_F(SSLClientSocketTest, ConnectMismatched) {
 
 // Attempt to connect to a page which requests a client certificate. It should
 // return an error code on connect.
+// Flaky: http://crbug.com/54445
 TEST_F(SSLClientSocketTest, FLAKY_ConnectClientAuthCertRequested) {
   net::TestServer test_server(net::TestServer::TYPE_HTTPS_CLIENT_AUTH,
                               FilePath());
diff --git a/sandbox/src/unload_dll_test.cc b/sandbox/src/unload_dll_test.cc
index f1e28bb..063f43f 100644
--- a/sandbox/src/unload_dll_test.cc
+++ b/sandbox/src/unload_dll_test.cc
@@ -65,6 +65,7 @@ TEST(UnloadDllTest, UnloadAviCapDllNoPatching) {
   EXPECT_EQ(SBOX_TEST_FAILED, runner.RunTest(L"UseOneDLL B avicap32.dll"));
 }
 
+// Flaky: http://crbug.com/38404
 TEST(UnloadDllTest, FLAKY_UnloadAviCapDllWithPatching) {
   TestRunner runner;
   runner.SetTimeout(2000);