Download filename determination refactor (3/3)

chrome/browser/browsing_data_remover.cc

Index: chrome/browser/browsing_data_remover.cc
diff --git a/chrome/browser/browsing_data_remover.cc b/chrome/browser/browsing_data_remover.cc
index 118520296663f6e7f4aa7d18894bb64ef6e47504..5fa1b986e266c93f5dd2687e8e14224babec44a5 100644
--- a/chrome/browser/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data_remover.cc
@@ -294,7 +294,7 @@ void BrowsingDataRemover::RemoveImpl(int remove_mask,
     DownloadManager* download_manager =
         BrowserContext::GetDownloadManager(profile_);
     download_manager->RemoveDownloadsBetween(delete_begin_, delete_end_);
-    download_manager->ClearLastDownloadPath();
+    download_manager->ClearTransientState();

[Randy Smith (Not in Mondays), 2012/07/05 15:42:32]

Given that this is in chrome, could it get the CDMD and call the function on
that directly, thus avoiding the need for the indirects through the DM?

[asanka, 2012/07/09 21:13:46]

Hmm. That makes sense. I didn't see a very clean way to get at the CDMD from
here. Perhaps add a method to DownloadService to return the
ChromeDownloadManagerDelegate for the profile?

[Randy Smith (Not in Mondays), 2012/07/10 18:33:09]

I think it's already present, isn't it? 
(DownloadService::GetDownloadManagerDelegate())

[asanka, 2012/07/11 20:03:32]

Hmm. I'd like to avoid up-casting if possible, but I think that should be okay.

   }
 
   if (remove_mask & REMOVE_COOKIES) {