Added ability to set ChromeDownloadManagerDelegate for testing.

chrome/test/base/testing_profile.cc

Index: chrome/test/base/testing_profile.cc
diff --git a/chrome/test/base/testing_profile.cc b/chrome/test/base/testing_profile.cc
index dcb57ef6032245445e9e8404b4c2668a6f55dd51..a0fefffe4f0219e92ed603dad894736d12b3a893 100644
--- a/chrome/test/base/testing_profile.cc
+++ b/chrome/test/base/testing_profile.cc
@@ -786,6 +786,10 @@ prerender::PrerenderManager* TestingProfile::GetPrerenderManager() {
   return prerender_manager_.get();
 }
 
+void TestingProfile::SetDownloadManagerDelegate(
+    ChromeDownloadManagerDelegate* delegate) {

[Paweł Hajdan Jr., 2011/09/09 22:11:54]

Why empty? This is very surprising.

[Randy Smith (Not in Mondays), 2011/09/09 23:45:14]

Maybe there's something I'm missing?  The implementation of
TestingProfile::GetDownloadManager() just returns null, and I didn't see any
machinery for dealing with DownloadManagerDelegate's within it.  How do you
think I should implement this routine, given the rest of the profile
implementation?  DCHECK?

[benjhayden, 2011/09/12 16:32:25]

Maybe just a comment?

[Randy Smith (Not in Mondays), 2011/09/12 17:52:58]

Waiting on Pawel and will be guided by him.

[Paweł Hajdan Jr., 2011/09/12 20:06:44]

Not enough. I want to save people time with a debugger (they have to get to that
comment somehow).

Either DCHECK or NOTIMPLEMENTED, whatever you prefer.

[Randy Smith (Not in Mondays), 2011/09/13 03:13:18]

Done.

[jam, 2011/09/13 17:18:03]

given that GetDownloadManager above doesn't dcheck or has a NOTIMPLEMENTED, it
doesn't seem appropriate to add one here.

+}
+
 PrefService* TestingProfile::GetOffTheRecordPrefs() {
   return NULL;
 }