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

Unified Diff: chrome/browser/download/download_service.h

Issue 26938003: Don't prompt to save malicious downloads on exit (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updated comments 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: chrome/browser/download/download_service.h
diff --git a/chrome/browser/download/download_service.h b/chrome/browser/download/download_service.h
index 3c3f6b6675729dda4079e97e518c84312f5964d8..47081eadd348b40220c82120407497d1b8fc00e5 100644
--- a/chrome/browser/download/download_service.h
+++ b/chrome/browser/download/download_service.h
@@ -44,11 +44,17 @@ class DownloadService : public BrowserContextKeyedService {
// Has a download manager been created?
bool HasCreatedDownloadManager();
- // Number of downloads associated with this instance of the service.
- int DownloadCount() const;
-
- // Number of downloads associated with all profiles.
- static int DownloadCountAllProfiles();
+ // Number of non-dangerous downloads associated with this instance of the
+ // service. This excludes downloads that are marked as dangerous (with one
+ // exception- it still counts DANGEROUS_FILE downloads, since the user
+ // probably wants them).
+ int NonDangerousDownloadCount() const;
+
+ // Number of non-dangerous downloads associated with all profiles.
+ // This excludes downloads that are marked as dangerous (with one exception-
+ // it still counts DANGEROUS_FILE downloads, since the user probably wants
+ // them).
+ static int NonDangerousDownloadCountAllProfiles();
// Sets the DownloadManagerDelegate associated with this object and
// its DownloadManager. Takes ownership of |delegate|, and destroys

Powered by Google App Engine
This is Rietveld 408576698