Fixed issue with DownloadTestObserver.

chrome/browser/ui/browser_close_browsertest.cc

Index: chrome/browser/ui/browser_close_browsertest.cc
diff --git a/chrome/browser/ui/browser_close_browsertest.cc b/chrome/browser/ui/browser_close_browsertest.cc
index c0aacc690782137786b3c3194f06dac902f96ba4..43688584d841047e7b859eaa6215e7cd972bf1d9 100644
--- a/chrome/browser/ui/browser_close_browsertest.cc
+++ b/chrome/browser/ui/browser_close_browsertest.cc
@@ -115,13 +115,12 @@ class BrowserCloseTest : public InProcessBrowserTest {
     DownloadManager* download_manager =
         browser->profile()->GetDownloadManager();
     scoped_ptr<DownloadTestObserver> observer(
-        new DownloadTestObserver(
-            download_manager, num_downloads,
-            DownloadItem::IN_PROGRESS,
-            true,  // Bail on select file.
-            DownloadTestObserver::ON_DANGEROUS_DOWNLOAD_FAIL));
+        new DownloadTestObserverInProgress(download_manager,
+                                           num_downloads,
+                                           true));  // Bail on select file.
 
     // Set of that number of downloads.
+    size_t count_downloads = num_downloads;
     while (num_downloads--)
       ui_test_utils::NavigateToURLWithDisposition(
           browser, url, NEW_BACKGROUND_TAB,
@@ -129,6 +128,8 @@ class BrowserCloseTest : public InProcessBrowserTest {
 
     // Wait for them.
     observer->WaitForFinished();
+    DCHECK_EQ(count_downloads,

[sky, 2012/03/09 20:20:28]

Why are you using DCHECK in a test?

[ahendrickson, 2012/03/09 20:49:14]

Oops, missed that one.  Fixed.

+              observer->NumDownloadsSeenInState(DownloadItem::IN_PROGRESS));
   }
 
   // All all downloads created in CreateStalledDownloads() to