Remove some obsolete FLAKY_ markers, and add some missing bug references

chrome/test/automation/automation_proxy_uitest.cc

 // Copyright (c) 2010 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 <string>
 
 #include "app/app_switches.h"
 #include "app/message_box_flags.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
@@ skipping 822 matching lines @@
 template <typename T> T** ReceivePointer(scoped_refptr<T>& p) {  // NOLINT
   return reinterpret_cast<T**>(&p);
 }
 
 AutomationProxy* ExternalTabUITest::CreateAutomationProxy(int exec_timeout) {
   mock_ = new ExternalTabUITestMockClient(exec_timeout);
   return mock_;
 }
 
 // Create with specifying a url
-// Flaky, http://crbug.com/32293.
+// Flaky, http://crbug.com/32293
 TEST_F(ExternalTabUITest, FLAKY_CreateExternalTab1) {
   scoped_refptr<TabProxy> tab;
   TimedMessageLoopRunner loop(MessageLoop::current());
   ASSERT_THAT(mock_, testing::NotNull());
 
   EXPECT_CALL(*mock_, OnNavigationStateChanged(_, _, _))
       .Times(testing::AnyNumber());
 
   EXPECT_CALL(*mock_, OnDidNavigate(testing::_, testing::_))
       .Times(1)
       .WillOnce(testing::InvokeWithoutArgs(mock_,
                 &ExternalTabUITestMockClient::DestroyHostWindow));
 
   EXPECT_CALL(*mock_, HandleClosed(1))
       .Times(1)
       .WillOnce(QUIT_LOOP(&loop));
 
   tab = mock_->CreateTabWithUrl(GURL(simple_data_url));
   loop.RunFor(action_max_timeout_ms());
 }
 
 // Create with empty url and then navigate
+// Flaky, http://crbug.com/32293
 TEST_F(ExternalTabUITest, FLAKY_CreateExternalTab2) {
   scoped_refptr<TabProxy> tab;
   TimedMessageLoopRunner loop(MessageLoop::current());
   ASSERT_THAT(mock_, testing::NotNull());
 
   EXPECT_CALL(*mock_, OnNavigationStateChanged(_, _, _))
       .Times(testing::AnyNumber());
 
   EXPECT_CALL(*mock_, OnDidNavigate(testing::_, testing::_))
     .Times(1)
@@ skipping 88 matching lines @@
           testing::InvokeWithoutArgs(CreateFunctor(mock_,
               &ExternalTabUITestMockClient::DestroyHostWindow)),
           QUIT_LOOP_SOON(&loop, 50)));
 
   EXPECT_CALL(*mock_, HandleClosed(1)).Times(1);
 
   tab = mock_->CreateTabWithUrl(GURL(content));
   loop.RunFor(action_max_timeout_ms());
 }
 
+// Flaky: http://crbug.com/62143
 TEST_F(ExternalTabUITest, FLAKY_PostMessageTarget)  {
   net::TestServer test_server(
       net::TestServer::TYPE_HTTP,
       FilePath(FILE_PATH_LITERAL("chrome/test/data/external_tab")));
   ASSERT_TRUE(test_server.Start());
 
   scoped_refptr<TabProxy> tab;
   TimedMessageLoopRunner loop(MessageLoop::current());
   ASSERT_THAT(mock_, testing::NotNull());
   EXPECT_CALL(*mock_, OnNavigationStateChanged(_, _, _))
@@ skipping 519 matching lines @@
 
   // Allow some time for the popup to show up and close.
   PlatformThread::Sleep(sleep_timeout_ms());
 
   std::wstring expected(L"string");
   std::wstring jscript = CreateJSString(L"\"" + expected + L"\"");
   std::wstring actual;
   ASSERT_TRUE(tab->ExecuteAndExtractString(L"", jscript, &actual));
   ASSERT_EQ(expected, actual);
 }