Commit ac728033 authored by Rainhard Findling's avatar Rainhard Findling Committed by Commit Bot

Safety check UI: icon for 'managed by extension'

Bug: 1015841
Change-Id: I885c7dc96bcd4410c288156344f017e130414071
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111315Reviewed-by: default avatarEsmael Elmoslimany <aee@chromium.org>
Commit-Queue: Rainhard Findling <rainhard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#752047}
parent b537f40d
...@@ -153,8 +153,8 @@ ...@@ -153,8 +153,8 @@
<template is="dom-if" <template is="dom-if"
if="[[shouldShowSafeBrowsingManagedIcon_(safeBrowsingStatus_)]]" if="[[shouldShowSafeBrowsingManagedIcon_(safeBrowsingStatus_)]]"
restamp> restamp>
<iron-icon id="safetyCheckSafeBrowsingManagedIcon" icon="cr20:domain" <iron-icon id="safetyCheckSafeBrowsingManagedIcon" aria-hidden="true"
aria-hidden="true"> icon="[[getSafeBrowsingManagedIcon_(safeBrowsingStatus_)]]">
</iron-icon> </iron-icon>
</template> </template>
</div> </div>
......
...@@ -500,12 +500,21 @@ Polymer({ ...@@ -500,12 +500,21 @@ Polymer({
* @return {boolean} * @return {boolean}
*/ */
shouldShowSafeBrowsingManagedIcon_: function() { shouldShowSafeBrowsingManagedIcon_: function() {
return this.getSafeBrowsingManagedIcon_() != null;
},
/**
* @private
* @return {?string}
*/
getSafeBrowsingManagedIcon_: function() {
switch (this.safeBrowsingStatus_) { switch (this.safeBrowsingStatus_) {
case settings.SafetyCheckSafeBrowsingStatus.DISABLED_BY_ADMIN: case settings.SafetyCheckSafeBrowsingStatus.DISABLED_BY_ADMIN:
return 'cr20:domain';
case settings.SafetyCheckSafeBrowsingStatus.DISABLED_BY_EXTENSION: case settings.SafetyCheckSafeBrowsingStatus.DISABLED_BY_EXTENSION:
return true; return 'cr:extension';
default: default:
return false; return null;
} }
}, },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment