DownloadManager intereface refactoring to allow cleaner DownloadItem unit tests.

content/browser/download/download_item.h

Index: content/browser/download/download_item.h
diff --git a/content/browser/download/download_item.h b/content/browser/download/download_item.h
index 3592d529f320dc5f903458dcc412b71c6e67385e..3b83def86c56552ead09c08d80f5b1972ebd3e75 100644
--- a/content/browser/download/download_item.h
+++ b/content/browser/download/download_item.h
@@ -21,11 +21,11 @@
 #include <string>
 
 #include "base/string16.h"
+#include "content/browser/download/download_id.h"
 #include "content/browser/download/download_state_info.h"
 #include "content/browser/download/interrupt_reasons.h"
 
 class DownloadFileManager;
-class DownloadId;
 class DownloadManager;
 class FilePath;
 class GURL;
@@ -37,6 +37,10 @@ class Time;
 class TimeDelta;
 }
 
+namespace content {
+class BrowserContext;
+}
+
 // One DownloadItem per download. This is the model class that stores all the
 // state for a download. Multiple views, such as a tab's download shelf and the
 // Destination tab's download view, may refer to a given DownloadItem.
@@ -246,7 +250,6 @@ class CONTENT_EXPORT DownloadItem {
   virtual base::Time GetEndTime() const = 0;
   virtual void SetDbHandle(int64 handle) = 0;
   virtual int64 GetDbHandle() const = 0;
-  virtual DownloadManager* GetDownloadManager() = 0;
   virtual bool IsPaused() const = 0;
   virtual bool GetOpenWhenComplete() const = 0;
   virtual void SetOpenWhenComplete(bool open) = 0;
@@ -271,6 +274,7 @@ class CONTENT_EXPORT DownloadItem {
   virtual InterruptReason GetLastReason() const = 0;
   virtual DownloadPersistentStoreInfo GetPersistentStoreInfo() const = 0;
   virtual DownloadStateInfo GetStateInfo() const = 0;
+  virtual content::BrowserContext* BrowserContext() const = 0;
   virtual TabContents* GetTabContents() const = 0;
 
   // Returns the final target file path for the download.