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

Unified Diff: content/browser/download/download_state_info.h

Issue 8468020: Propagate the SafeBrowsing download protection verdict to the DownloadItem. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Add missing file and fix some unit-tests Created 9 years, 1 month 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/browser/download/download_state_info.h
diff --git a/content/browser/download/download_state_info.h b/content/browser/download/download_state_info.h
index 0729fe2089ef0208cc4a4372e2b9f3245b6c8f76..840dd860b02c0cc1cc58d38fa82babf89778128f 100644
--- a/content/browser/download/download_state_info.h
+++ b/content/browser/download/download_state_info.h
@@ -23,7 +23,9 @@ struct DownloadStateInfo {
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;
@@ -56,6 +58,14 @@ struct DownloadStateInfo {
// If safebrowsing believes this URL leads to malware.
bool is_dangerous_url;
+ // If safebrowsing believes this content to be malicious.
+ bool is_dangerous_content;
+
+ // 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;
+
// True if this download's file name was specified initially.
FilePath force_file_name;
};

Powered by Google App Engine
This is Rietveld 408576698