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

Unified Diff: chrome/browser/resources/md_downloads/item.js

Issue 2465723004: Make download warning icon and text more consistent (Closed)
Patch Set: rebase Created 4 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: chrome/browser/resources/md_downloads/item.js
diff --git a/chrome/browser/resources/md_downloads/item.js b/chrome/browser/resources/md_downloads/item.js
index d87845d1a0ab9a4dd404c5ac67ffba502db0f248..9bb8c5c5d0251a058df56a5909a979762bbe761c 100644
--- a/chrome/browser/resources/md_downloads/item.js
+++ b/chrome/browser/resources/md_downloads/item.js
@@ -121,19 +121,9 @@ cr.define('downloads', function() {
/** @private */
computeDangerIcon_: function() {
- if (!this.isDangerous_)
- return '';
-
- switch (this.data.danger_type) {
- case downloads.DangerType.DANGEROUS_CONTENT:
- case downloads.DangerType.DANGEROUS_HOST:
- case downloads.DangerType.DANGEROUS_URL:
- case downloads.DangerType.POTENTIALLY_UNWANTED:
- case downloads.DangerType.UNCOMMON_CONTENT:
- return 'downloads:remove-circle';
- default:
- return 'cr:warning';
- }
+ if (this.isDangerous_)
+ return 'cr:warning';
+ return '';
Dan Beam 2016/11/01 18:49:50 return this.isDangerous_ ? 'cr:warning' : '';
Jialiu Lin 2016/11/01 21:19:01 Done.
},
/** @private */

Powered by Google App Engine
This is Rietveld 408576698