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

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: Removed unneeded code 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
« no previous file with comments | « content/browser/download/download_manager_impl.cc ('k') | content/public/test/download_test_observer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..6bb1f35156d26e5a4bca8b87de7e7f26fefbebfd 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;
+ // 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 malicious.
+ virtual int NonMaliciousInProgressCount() const = 0;
+
virtual BrowserContext* GetBrowserContext() const = 0;
// Checks whether downloaded files still exist. Updates state of downloads
« no previous file with comments | « content/browser/download/download_manager_impl.cc ('k') | content/public/test/download_test_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698