Move creation and ownership of DownloadManager from the embedder to content. This matches all the o…

content/browser/download/download_manager_impl_unittest.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 <set>
 #include <string>
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 86 matching lines @@
     bool calculate_hash,
     const net::BoundNetLog& bound_net_log) {
   NOTREACHED();
   return NULL;
 }
 
 DownloadId::Domain kValidIdDomain = "valid DownloadId::Domain";
 
 class TestDownloadManagerDelegate : public content::DownloadManagerDelegate {
  public:
-  TestDownloadManagerDelegate()
+  explicit TestDownloadManagerDelegate(content::DownloadManager* dm)
       : mark_content_dangerous_(false),
         prompt_user_for_save_location_(false),
         should_complete_download_(true),
-        download_manager_(NULL) {
+        download_manager_(dm) {
   }
 
   void set_download_directory(const FilePath& path) {
     download_directory_ = path;
   }
 
-  void set_download_manager(content::DownloadManager* dm) {
-    download_manager_ = dm;
-  }
-
   void set_prompt_user_for_save_location(bool value) {
     prompt_user_for_save_location_ = value;
   }
 
   virtual bool ShouldStartDownload(int32 download_id) OVERRIDE {
     DownloadItem* item = download_manager_->GetActiveDownloadItem(download_id);
     FilePath path = net::GenerateFileName(item->GetURL(),
                                           item->GetContentDisposition(),
                                           item->GetReferrerCharset(),
                                           item->GetSuggestedFilename(),
@@ skipping 123 matching lines @@
 
 } // namespace
 
 class DownloadManagerTest : public testing::Test {
  public:
   static const char* kTestData;
   static const size_t kTestDataLen;
 
   DownloadManagerTest()
       : browser_context(new content::TestBrowserContext()),
-        download_manager_delegate_(new TestDownloadManagerDelegate()),
-        download_manager_(new DownloadManagerImpl(
-            download_manager_delegate_.get(), NULL)),
+        download_manager_(new DownloadManagerImpl(NULL)),
         ui_thread_(BrowserThread::UI, &message_loop_),
         file_thread_(BrowserThread::FILE, &message_loop_) {
+    download_manager_delegate_.reset(
+        new TestDownloadManagerDelegate(download_manager_.get()));
+    download_manager_->SetDelegate(download_manager_delegate_.get());
     download_manager_->Init(browser_context.get());
-    download_manager_delegate_->set_download_manager(download_manager_);
   }
 
   ~DownloadManagerTest() {
     download_manager_->Shutdown();
     // browser_context must outlive download_manager_, so we explicitly delete
     // download_manager_ first.
     download_manager_ = NULL;
     download_manager_delegate_.reset();
     browser_context.reset(NULL);
     message_loop_.RunAllPending();
@@ skipping 992 matching lines @@
   EXPECT_FALSE(observer->hit_state(DownloadItem::INTERRUPTED));
   EXPECT_TRUE(observer->hit_state(DownloadItem::COMPLETE));
   EXPECT_FALSE(observer->hit_state(DownloadItem::REMOVING));
   EXPECT_TRUE(observer->was_updated());
   EXPECT_FALSE(observer->was_opened());
   EXPECT_TRUE(download->GetFileExternallyRemoved());
   EXPECT_EQ(DownloadItem::COMPLETE, download->GetState());
 
   EXPECT_FALSE(file_util::PathExists(new_path));
 }