Propagate the SafeBrowsing download protection verdict to the DownloadItem.

content/browser/download/download_state_info.h

 // Copyright (c) 2011 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 CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_STATE_INFO_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_STATE_INFO_H_
 #pragma once
 
 #include "base/file_path.h"
 #include "content/common/content_export.h"
 #include "content/public/common/page_transition_types.h"
 
 // Contains information relating to the process of determining what to do with
 // the download.
 struct DownloadStateInfo {
   DownloadStateInfo();
   DownloadStateInfo(bool has_user_gesture,
                     bool prompt_user_for_save_location);
   DownloadStateInfo(const FilePath& target,
                     const FilePath& forced_name,
                     bool has_user_gesture,
                     content::PageTransition transition_type,
                     bool prompt_user_for_save_location,
                     int uniquifier,
                     bool dangerous_file,
-                    bool dangerous_url);
+                    bool dangerous_url,
+                    bool dangerous_content,
+                    bool needs_quarantine_file);
 
   // Indicates if the download is dangerous.
   CONTENT_EXPORT bool IsDangerous() const;
 
   // The original name for a dangerous download, specified by the request.
   FilePath target_name;
 
   // The path where we save the download.  Typically generated.
   FilePath suggested_path;
 
@@ skipping 12 matching lines @@
   // False if the UI should be suppressed and the download performed to the
   // default location.
   bool prompt_user_for_save_location;
 
   // True if this download file is potentially dangerous (ex: exe, dll, ...).
   bool is_dangerous_file;
 
   // If safebrowsing believes this URL leads to malware.
   bool is_dangerous_url;
 
+  // If safebrowsing believes this content to be malicious.
+  bool is_dangerous_content;

[Randy Smith (Not in Mondays), 2011/11/16 17:52:41]

Any reason not to shift over to using an enum (maybe the standard enum?) for
these three?  The proliferation of bools is a bit disturbing.

[noelutz, 2011/11/16 23:15:07]

Done.

+
+  // True if the file is potentially dangerous.  If true, the download content
+  // will be written to a temporary file that isn't easily accessbile to the
+  // user.
+  bool needs_quarantine_file;

[Randy Smith (Not in Mondays), 2011/11/16 17:52:41]

I'd like to add this to the above enum, if we could (all the way into
DownloadItem is fine)--something like "MAYBE_DANGEROUS_CONTENT"?

[noelutz, 2011/11/16 23:15:07]

Done.

+
   // True if this download's file name was specified initially.
   FilePath force_file_name;
 };
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_STATE_INFO_H_