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

Unified Diff: chrome/browser/resources/downloads.js

Issue 8558029: Update download page to deal with new DANGEROUS_CONTENT state. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments 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
« no previous file with comments | « chrome/browser/resources/active_downloads.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/downloads.js
diff --git a/chrome/browser/resources/downloads.js b/chrome/browser/resources/downloads.js
index ec8148d73190174dd9edaba75b849c0de17d34f1..9e78342e84281c047acf836baa28e736d08902ef 100644
--- a/chrome/browser/resources/downloads.js
+++ b/chrome/browser/resources/downloads.js
@@ -305,6 +305,7 @@ Download.DangerType = {
NOT_DANGEROUS: "NOT_DANGEROUS",
DANGEROUS_FILE: "DANGEROUS_FILE",
DANGEROUS_URL: "DANGEROUS_URL",
+ DANGEROUS_CONTENT: "DANGEROUS_CONTENT"
}
/**
@@ -347,6 +348,7 @@ Download.prototype.update = function(download) {
this.dangerDesc_.textContent = localStrings.getStringF('danger_file_desc',
this.fileName_);
} else {
+ // This is used by both DANGEROUS_URL and DANGEROUS_CONTENT.
this.dangerDesc_.textContent = localStrings.getString('danger_url_desc');
}
this.danger_.style.display = 'block';
@@ -462,6 +464,7 @@ Download.prototype.getStatusText_ = function() {
case Download.States.PAUSED:
return localStrings.getString('status_paused');
case Download.States.DANGEROUS:
+ // danger_url_desc is also used by DANGEROUS_CONTENT.
var desc = this.dangerType_ == Download.DangerType.DANGEROUS_FILE ?
'danger_file_desc' : 'danger_url_desc';
return localStrings.getString(desc);
« no previous file with comments | « chrome/browser/resources/active_downloads.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698