Don't prompt to save malicious downloads on exit

chrome/browser/download/download_service.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_
 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_
 
 #include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "base/memory/ref_counted.h"
@@ skipping 27 matching lines @@
 #if !defined(OS_ANDROID)
   ExtensionDownloadsEventRouter* GetExtensionEventRouter() {
     return extension_event_router_.get();
   }
 #endif
 
   // Has a download manager been created?
   bool HasCreatedDownloadManager();
 
   // Number of downloads associated with this instance of the service.
   int DownloadCount() const;

[asanka, 2013/10/11 16:02:35]

Are DownloadCount() and DownloadCountAllProfiles() used anywhere after this
change?

[felt, 2013/10/11 17:26:44]

They seemed like potentially useful utility functions, but I suppose if someone
actually needs one they can add it back in. Removed.

 
+  // Number of non-dangerous downloads associated with this instance of the
+  // service. This excludes downloads that are marked as dangerous.
+  int NonDangerousDownloadCount() const;
+
   // Number of downloads associated with all profiles.
   static int DownloadCountAllProfiles();
 
+  // Number of non-dangerous downloads associated with all profiles.
+  // This excludes downloads that are marked as dangerous.
+  static int NonDangerousDownloadCountAllProfiles();
+
   // Sets the DownloadManagerDelegate associated with this object and
   // its DownloadManager.  Takes ownership of |delegate|, and destroys
   // the previous delegate.  For testing.
   void SetDownloadManagerDelegateForTesting(
       ChromeDownloadManagerDelegate* delegate);
 
   // Will be called to release references on other services as part
   // of Profile shutdown.
   virtual void Shutdown() OVERRIDE;
 
@@ skipping 28 matching lines @@
   // should be a separate EDER for on-record and off-record managers.
   // There does not appear to be a separate ExtensionSystem for on-record and
   // off-record profiles, so ExtensionSystem cannot own the EDER.
   scoped_ptr<ExtensionDownloadsEventRouter> extension_event_router_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(DownloadService);
 };
 
 #endif  // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_