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

chrome/browser/automation/testing_automation_provider.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 "chrome/browser/automation/testing_automation_provider.h"
 
 #include <map>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 159 matching lines @@
 #endif
 
 #if !defined(NO_TCMALLOC) && (defined(OS_LINUX) || defined(OS_CHROMEOS))
 #include "third_party/tcmalloc/chromium/src/gperftools/heap-profiler.h"
 #endif  // !defined(NO_TCMALLOC) && (defined(OS_LINUX) || defined(OS_CHROMEOS))
 
 using automation::Error;
 using automation::ErrorCode;
 using automation_util::SendErrorIfModalDialogActive;
 using content::BrowserChildProcessHostIterator;
+using content::BrowserContext;
 using content::BrowserThread;
 using content::ChildProcessHost;
 using content::DownloadItem;
 using content::DownloadManager;
 using content::InterstitialPage;
 using content::NativeWebKeyboardEvent;
 using content::NavigationController;
 using content::NavigationEntry;
 using content::PluginService;
 using content::OpenURLParams;
@@ skipping 951 matching lines @@
   new DomOperationMessageSender(this, reply_message, false);
   ExecuteJavascriptInRenderViewFrame(WideToUTF16Hack(frame_xpath),
                                      WideToUTF16Hack(script), reply_message,
                                      web_contents->GetRenderViewHost());
 }
 
 void TestingAutomationProvider::GetDownloadDirectory(
     int handle, FilePath* download_directory) {
   if (tab_tracker_->ContainsHandle(handle)) {
     NavigationController* tab = tab_tracker_->GetResource(handle);
-    DownloadManager* dlm = tab->GetBrowserContext()->GetDownloadManager();
+    DownloadManager* dlm =
+        BrowserContext::GetDownloadManager(tab->GetBrowserContext());
     *download_directory =
         DownloadPrefs::FromDownloadManager(dlm)->download_path();
   }
 }
 
 // Sample json input: { "command": "OpenNewBrowserWindowWithNewProfile" }
 // Sample output: {}
 void TestingAutomationProvider::OpenNewBrowserWindowWithNewProfile(
     base::DictionaryValue* args, IPC::Message* reply_message) {
   ProfileManager* profile_manager = g_browser_process->profile_manager();
@@ skipping 1459 matching lines @@
                                                  IPC::Message* reply_message) {
   AutomationJSONReply reply(this, reply_message);
   scoped_ptr<DictionaryValue> return_value(new DictionaryValue);
   ListValue* list_of_downloads = new ListValue;
 
   DownloadService* download_service(
       DownloadServiceFactory::GetForProfile(browser->profile()));
 
   if (download_service->HasCreatedDownloadManager()) {
     std::vector<DownloadItem*> downloads;
-    download_service->GetDownloadManager()->
+    BrowserContext::GetDownloadManager(browser->profile())->
         GetAllDownloads(FilePath(), &downloads);
 
     for (std::vector<DownloadItem*>::iterator it = downloads.begin();
          it != downloads.end();
          it++) {  // Fill info about each download item.
       list_of_downloads->Append(GetDictionaryFromDownloadItem(*it));
     }
   }
   return_value->Set("downloads", list_of_downloads);
   reply.SendSuccess(return_value.get());
@@ skipping 14 matching lines @@
   DownloadService* download_service =
       DownloadServiceFactory::GetForProfile(browser->profile());
   if (!download_service->HasCreatedDownloadManager()) {
     // No download manager, so no downloads to wait for.
     AutomationJSONReply(this, reply_message).SendSuccess(NULL);
     return;
   }
 
   // This observer will delete itself.
   new AllDownloadsCompleteObserver(
-      this, reply_message, download_service->GetDownloadManager(),
+      this, reply_message,
+      BrowserContext::GetDownloadManager(browser->profile()),
       pre_download_ids);
 }
 
 namespace {
 
 DownloadItem* GetDownloadItemFromId(int id, DownloadManager* download_manager) {
   std::vector<DownloadItem*> downloads;
   download_manager->GetAllDownloads(FilePath(), &downloads);
   DownloadItem* selected_item = NULL;
 
@@ skipping 25 matching lines @@
   if (!download_service->HasCreatedDownloadManager()) {
     AutomationJSONReply(this, reply_message).SendError("No download manager.");
     return;
   }
   if (!args->GetInteger("id", &id) || !args->GetString("action", &action)) {
     AutomationJSONReply(this, reply_message)
         .SendError("Must include int id and string action.");
     return;
   }
 
-  DownloadManager* download_manager = download_service->GetDownloadManager();
+  DownloadManager* download_manager =
+      BrowserContext::GetDownloadManager(browser->profile());
   DownloadItem* selected_item = GetDownloadItemFromId(id, download_manager);
   if (!selected_item) {
     AutomationJSONReply(this, reply_message)
         .SendError(StringPrintf("No download with an id of %d\n", id));
     return;
   }
 
   // We need to be IN_PROGRESS for these actions.
   if ((action == "toggle_pause" || action == "cancel") &&
       !selected_item->IsInProgress()) {
@@ skipping 600 matching lines @@
   if (!web_contents->SavePage(
           FilePath(filename),
           FilePath(parent_directory),
           content::SAVE_PAGE_TYPE_AS_ONLY_HTML)) {
     AutomationJSONReply(this, reply_message).SendError(
         "Could not initiate SavePage");
     return;
   }
   // The observer will delete itself when done.
   new SavePackageNotificationObserver(
-      DownloadServiceFactory::GetForProfile(
-          browser->profile())->GetDownloadManager(),
+      BrowserContext::GetDownloadManager(browser->profile()),
       this, reply_message);
 }
 
 // Refer to ImportSettings() in chrome/test/pyautolib/pyauto.py for sample
 // json input.
 // Sample json output: "{}"
 void TestingAutomationProvider::ImportSettings(Browser* browser,
                                                DictionaryValue* args,
                                                IPC::Message* reply_message) {
   // Map from the json string passed over to the import item masks.
@@ skipping 3311 matching lines @@
 }
 
 void TestingAutomationProvider::EnsureTabSelected(Browser* browser,
                                                   WebContents* tab) {
   if (browser->GetSelectedWebContents() != tab ||
       browser != BrowserList::GetLastActive()) {
     browser->ActivateTabAt(browser->GetIndexOfController(&tab->GetController()),
         true /* user_gesture */);
   }
 }