Last patch in removing MessageLoop* caching.

chrome/browser/download/download_request_manager_unittest.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 "chrome/browser/chrome_thread.h"
 #include "chrome/browser/download/download_request_manager.h"
 #include "chrome/browser/renderer_host/test/test_render_view_host.h"
 #include "chrome/browser/tab_contents/navigation_controller.h"
 #include "chrome/test/testing_profile.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 class DownloadRequestManagerTest
     : public RenderViewHostTestHarness,
       public DownloadRequestManager::Callback {
  public:
+  DownloadRequestManagerTest() : io_thread_(ChromeThread::IO, &message_loop_) {}
+
   virtual void SetUp() {
     RenderViewHostTestHarness::SetUp();
 
     allow_download_ = true;
     ask_allow_count_ = cancel_count_ = continue_count_ = 0;
 
-    download_request_manager_ = new DownloadRequestManager(NULL, NULL);
+    download_request_manager_ = new DownloadRequestManager();
     test_delegate_.reset(new DownloadRequestManagerTestDelegate(this));
     DownloadRequestManager::SetTestingDelegate(test_delegate_.get());
   }
 
   virtual void TearDown() {
     DownloadRequestManager::SetTestingDelegate(NULL);
 
     RenderViewHostTestHarness::TearDown();
   }
 
   virtual void ContinueDownload() {
     continue_count_++;
   }
   virtual void CancelDownload() {
     cancel_count_++;
   }
 
   void CanDownload() {
     download_request_manager_->CanDownloadImpl(
         controller().tab_contents(), this);
+    message_loop_.RunAllPending();
   }
 
   bool ShouldAllowDownload() {
     ask_allow_count_++;
     return allow_download_;
   }
 
  protected:
   class DownloadRequestManagerTestDelegate
       : public DownloadRequestManager::TestingDelegate {
@@ skipping 16 matching lines @@
   int continue_count_;
 
   // Number of times CancelDownload was invoked.
   int cancel_count_;
 
   // Whether the download should be allowed.
   bool allow_download_;
 
   // Number of times ShouldAllowDownload was invoked.
   int ask_allow_count_;
+
+  ChromeThread io_thread_;
 };
 
 TEST_F(DownloadRequestManagerTest, Allow) {
   // All tabs should initially start at ALLOW_ONE_DOWNLOAD.
   ASSERT_EQ(DownloadRequestManager::ALLOW_ONE_DOWNLOAD,
             download_request_manager_->GetDownloadStatus(
                 controller().tab_contents()));
 
   // Ask if the tab can do a download. This moves to PROMPT_BEFORE_DOWNLOAD.
   CanDownload();
@@ skipping 105 matching lines @@
   ask_allow_count_ = continue_count_ = cancel_count_ = 0;
   CanDownload();
   ASSERT_EQ(0, ask_allow_count_);
   ASSERT_EQ(0, continue_count_);
   ASSERT_EQ(1, cancel_count_);
   // And the state shouldn't have changed.
   ASSERT_EQ(DownloadRequestManager::DOWNLOADS_NOT_ALLOWED,
             download_request_manager_->GetDownloadStatus(
                 controller().tab_contents()));
 }