Pass content-type resources to web intents. Goes through download, then invokes the p…

chrome/browser/download/chrome_download_manager_delegate.h

Index: chrome/browser/download/chrome_download_manager_delegate.h
diff --git a/chrome/browser/download/chrome_download_manager_delegate.h b/chrome/browser/download/chrome_download_manager_delegate.h
index 93f6721ca74d4a61aa455f7f1c7050d9df011926..e5ee17dffb732aa0f92b0d3a75591949954ce948 100644
--- a/chrome/browser/download/chrome_download_manager_delegate.h
+++ b/chrome/browser/download/chrome_download_manager_delegate.h
@@ -146,6 +146,13 @@ class ChromeDownloadManagerDelegate
   // Callback from history system.
   void OnItemAddedToPersistentStore(int32 download_id, int64 db_handle);
 
+  // Check policy of whether we should open this download with a web intents
+  // dispatch.
+  bool ShouldOpenWithWebIntents(const content::DownloadItem* item);
+
+  // Open the given item with a web intent dispatch.
+  void OpenWithWebIntent(const content::DownloadItem* item);

[Randy Smith (Not in Mondays), 2012/03/29 18:47:24]

Any reason why these can't be file-local functions rather than in the
chrome_download_manager_delegate.h interface?

[Greg Billock, 2012/03/29 20:53:01]

I like having the policy function be instance-based. If the opener needs the
profile, it's plausible that it be instance-based, too (although it could take
it as a parameter).

On 2012/03/29 18:47:24, rdsmith wrote:

+
   Profile* profile_;
   int next_download_id_;
   scoped_ptr<DownloadPrefs> download_prefs_;