Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(646)

Unified Diff: content/public/browser/download_manager.h

Issue 26938003: Don't prompt to save malicious downloads on exit (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed unit test compile error Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/browser/download_manager.h
diff --git a/content/public/browser/download_manager.h b/content/public/browser/download_manager.h
index 37c6d09da908f41dfdd78294aa55213577f244e8..40c33e9f40994aa9a3204605ec15a3572fd4a77b 100644
--- a/content/public/browser/download_manager.h
+++ b/content/public/browser/download_manager.h
@@ -158,6 +158,12 @@ class CONTENT_EXPORT DownloadManager : public base::SupportsUserData::Data {
// is too slow, use an AllDownloadItemNotifier to count in-progress items.
virtual int InProgressCount() const = 0;
asanka 2013/10/11 16:02:35 Is InProgressCount() used anywhere? And if so, can
felt 2013/10/11 17:26:44 It's used in several browser tests, and I think th
+ // The number of in progress (including paused) downloads.
+ // Performance note: this loops over all items. If profiling finds that this
+ // is too slow, use an AllDownloadItemNotifier to count in-progress items.
+ // This excludes downloads that are marked as any kind of dangerous.
+ virtual int NonDangerousInProgressCount() const = 0;
+
virtual BrowserContext* GetBrowserContext() const = 0;
// Checks whether downloaded files still exist. Updates state of downloads

Powered by Google App Engine
This is Rietveld 408576698