Skip to content

Commit

Permalink
Fix missing supervised user delete message on CrOS
Browse files Browse the repository at this point in the history
This was broken in http://crrev.com/1248613003, which replaced JS code to show/hide the message with CSS selectors based on the 'legacy-supervised' class.
However, that class was only used on desktop, not on CrOS.

This CL is a minimal fix, for merging to M48: Set this particular class also on CrOS.
Longer-term, it would be nice to clean up the user pod stuff to be less different on CrOS vs desktop...

BUG=546183

Review URL: https://codereview.chromium.org/1446193002

Cr-Commit-Position: refs/heads/master@{#360037}
  • Loading branch information
treib authored and Commit bot committed Nov 17, 2015
1 parent 936f5a5 commit 36d442b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions ui/login/account_picker/user_pod_row.js
Original file line number Diff line number Diff line change
Expand Up @@ -1028,6 +1028,7 @@ cr.define('login', function() {
this.setUserPodIconType('child');
} else if (this.user_.legacySupervisedUser && !this.user_.isDesktopUser) {
this.setUserPodIconType('legacySupervised');
this.classList.add('legacy-supervised');
} else if (this.multiProfilesPolicyApplied) {
// Mark user pod as not focusable which in addition to the grayed out
// filter makes it look in disabled state.
Expand Down

0 comments on commit 36d442b

Please sign in to comment.