Propagate the SafeBrowsing download protection verdict to the DownloadItem.

chrome/browser/download/download_extension_api.cc

Index: chrome/browser/download/download_extension_api.cc
diff --git a/chrome/browser/download/download_extension_api.cc b/chrome/browser/download/download_extension_api.cc
index dda5e78c0727505d96b6616c7e96994813457365..fab816cd6a39919d82cc2bfa5c6cceb878a57f21 100644
--- a/chrome/browser/download/download_extension_api.cc
+++ b/chrome/browser/download/download_extension_api.cc
@@ -57,6 +57,7 @@ const char kDangerFile[] = "file";
 const char kDangerKey[] = "danger";
 const char kDangerSafe[] = "safe";
 const char kDangerUrl[] = "url";
+const char kDangerContent[] = "content";

[asanka, 2011/11/16 18:34:05]

Nit: These should be ordered alphabetically.

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

Done.

 const char kEndTimeKey[] = "endTime";
 const char kErrorKey[] = "error";
 const char kFileSizeKey[] = "fileSize";
@@ -82,6 +83,7 @@ const char* DangerString(DownloadItem::DangerType danger) {
     case DownloadItem::NOT_DANGEROUS: return kDangerSafe;
     case DownloadItem::DANGEROUS_FILE: return kDangerFile;
     case DownloadItem::DANGEROUS_URL: return kDangerUrl;
+    case DownloadItem::DANGEROUS_CONTENT: return kDangerContent;

[Randy Smith (Not in Mondays), 2011/11/16 18:04:04]

I think (Asanka?) that we also want to export the "quarantine" info up here,
since that might conceivably have UI effects.  More specifically, I think we
want to export the MAYBE_DANGEROUS_CONTENT enum (if we go that way) so that the
UI knows it might in the future have to show a dangerous download warning for
this item.  This isn't high priority as I don't have a use case, but given that
one of the goals of the extension system is to support everything an alternative
UI might want to have, it seems worth making this visible.

[asanka, 2011/11/16 18:34:05]

I can see how it could be useful.  However I'm not quite certain whether we want
to commit to making it a reliable signal.

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

Propagated the maybe.

     default:
       NOTREACHED();
       return "";