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

Unified Diff: ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js

Issue 2624003003: WebUI: Add cr-policy-pref-indicator tests (Closed)
Patch Set: Elim registerTests Created 3 years, 11 months 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: ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js
diff --git a/ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js b/ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js
index 5367637bc34f7f6f612dfeccde990ef57723920c..dd1df6281d87f145b5aa7d44745f833387cfaf0d 100644
--- a/ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js
+++ b/ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js
@@ -48,11 +48,12 @@ var CrPolicyIndicatorBehavior = {
* @private
*/
getPolicyIndicatorIcon: function(type) {
- var icon = '';
+ var icon;
switch (type) {
case CrPolicyIndicatorType.EXTENSION:
case CrPolicyIndicatorType.NONE:
- return icon;
+ icon = 'domain'; // Set a valid icon for accessability audit
Dan Beam 2017/01/18 01:06:34 accessibility *
stevenjb 2017/01/18 23:13:03 Done.
+ break;
case CrPolicyIndicatorType.PRIMARY_USER:
icon = 'group';
break;
@@ -71,7 +72,7 @@ var CrPolicyIndicatorBehavior = {
},
/**
- * @param {CrPolicyIndicatorType} type
+ * @param {!CrPolicyIndicatorType} type
* @param {string} name The name associated with the indicator. See
* chrome.settingsPrivate.PrefObject.controlledByName
* @param {boolean=} opt_matches For RECOMMENDED only, whether the indicator

Powered by Google App Engine
This is Rietveld 408576698