Refactor downloads into a ProfileKeyedService.

chrome/browser/extensions/extension_downloads_api.cc

Index: chrome/browser/extensions/extension_downloads_api.cc
diff --git a/chrome/browser/extensions/extension_downloads_api.cc b/chrome/browser/extensions/extension_downloads_api.cc
index 2182b000fdcf62ef0ff1d153824ef259c545a0f2..72f686b9f7da797a9058f04adf34cc6f5f3d466a 100644
--- a/chrome/browser/extensions/extension_downloads_api.cc
+++ b/chrome/browser/extensions/extension_downloads_api.cc
@@ -21,6 +21,8 @@
 #include "base/stringprintf.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/download/download_service.h"
+#include "chrome/browser/download/download_service_factory.h"
 #include "chrome/browser/download/download_util.h"
 #include "chrome/browser/extensions/extension_downloads_api_constants.h"
 #include "chrome/browser/extensions/extension_event_names.h"
@@ -414,7 +416,9 @@ base::DictionaryValue* DownloadItemToJSON(DownloadItem* item) {
 ExtensionDownloadsEventRouter::ExtensionDownloadsEventRouter(
     Profile* profile)
   : profile_(profile),
-    manager_(profile ? profile->GetDownloadManager() : NULL) {
+    manager_(profile ?
+             DownloadServiceFactory::GetForProfile(profile)
+                 ->GetDownloadManager() : NULL) {

[Miranda Callahan, 2011/10/06 20:07:32]

hmm, again, wrapping before the -> is not kosher, I believe; what if we indent
line 420 four spaces, and then fit the whole second argument of the ternary
operator on this line?

[Randy Smith (Not in Mondays), 2011/10/08 23:46:54]

Done.

   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(profile_);
   DCHECK(manager_);