Added download error descriptions to tooltips for Mac & Linux.

chrome/browser/download/download_browsertest.cc

 // Copyright (c) 2012 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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
@@ skipping 772 matching lines @@
           // If it's not where we asked it to be, it should be in the
           // My Documents folder.
           FilePath my_docs_folder;
           EXPECT_TRUE(PathService::Get(chrome::DIR_USER_DOCUMENTS,
                                        &my_docs_folder));
           EXPECT_EQ(0u,
                     my_downloaded_file.value().find(my_docs_folder.value()));
         }
       }
     }
+
+    MessageLoop::current()->PostDelayedTask(

[sky, 2012/04/04 16:55:34]

Don't post tasks of arbitrary length. It makes for flaky and long running tests.
Instead listen for whatever it is you need to then exit when you receive the
event.

[ahendrickson, 2012/04/04 17:49:32]

Oh, sorry, I intended to remove this.  It was only to see if I could see the
tooltips during a browser test (I couldn't).

Removed.

+        FROM_HERE,
+        base::Bind(&MessageLoop::Quit,
+                   base::Unretained(MessageLoop::current())),
+        5000);
+
+    ui_test_utils::RunMessageLoop();
   }
 
   // Attempts to download a set of files, based on information in the
   // |download_info| array.  |count| is the number of files.
   // If a Select File dialog appears, it will choose the default and return
   // immediately.
   void DownloadFilesCheckErrors(size_t count, DownloadInfo* download_info) {
     DownloadFilesCheckErrorsSetup();
 
     for (size_t i = 0; i < count; ++i) {
@@ skipping 1714 matching lines @@
   downloads.clear();
   GetDownloads(browser(), &downloads);
   EXPECT_EQ(0u, downloads.size());
 
   if (creation_observer->succeeded()) {
     // Wait until the download is done.  We don't care how it's finished.
     backup_observer.WaitForFinished();
   }
   DownloadManagerForBrowser(browser())->RemoveAllDownloads();
 }