CrOS "remove this user" button should use red button style just like desktop browser

CrOS "remove this user" button should use red button style just like desktop browser

Remove this user button for supervised user is changed to back bold font style not by design. This CL restores the red button style for both removing non-owner user and supervised user.

BUG=641137
TEST=device test. Screenshots are in crbug.com/641137.

Committed: https://crrev.com/b316f87f5c0d4bb686c66867dded237fcc2275e4
Cr-Commit-Position: refs/heads/master@{#415570}

[Qiang(Joe) Xu, 2016-08-29 21:22:47]

Description was changed from

==========
CrOS "remove this user" button should use red button style just like desktop
browser

BUG=641137
==========

to

==========
CrOS "remove this user" button should use red button style just like desktop
browser

Remove this user button for supervised user is changed to back bold font style
not by design. This CL restores the red button style for both removing non-owner
user and supervised user.

BUG=641137
==========

[Qiang(Joe) Xu, 2016-08-29 21:22:47]

warx@chromium.org changed reviewers:
+ alemate@chromium.org

[Qiang(Joe) Xu, 2016-08-29 21:23:37]

Description was changed from

==========
CrOS "remove this user" button should use red button style just like desktop
browser

Remove this user button for supervised user is changed to back bold font style
not by design. This CL restores the red button style for both removing non-owner
user and supervised user.

BUG=641137
==========

to

==========
CrOS "remove this user" button should use red button style just like desktop
browser

Remove this user button for supervised user is changed to back bold font style
not by design. This CL restores the red button style for both removing non-owner
user and supervised user.

BUG=641137
TEST=device test. Screenshots are in crbug.com/641137.
==========

[Qiang(Joe) Xu, 2016-08-29 21:24:16]

Hi Alexander, could you help review on this? Thanks!

[Alexander Alekseev, 2016-08-31 00:52:29]

https://codereview.chromium.org/2293683002/diff/1/ui/login/account_picker/use...
File ui/login/account_picker/user_pod_template.html (right):

https://codereview.chromium.org/2293683002/diff/1/ui/login/account_picker/use...
ui/login/account_picker/user_pod_template.html:1: <style is="custom-style">
Please, move this style to a .css file. Our presubmit hooks are unable to
analyze inlined css yet.

[Qiang(Joe) Xu, 2016-08-31 05:01:03]

The CQ bit was checked by warx@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-31 05:01:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Qiang(Joe) Xu, 2016-08-31 05:02:25]

Hi Alexander, thanks for the help. New patch is uploaded.

https://codereview.chromium.org/2293683002/diff/1/ui/login/account_picker/use...
File ui/login/account_picker/user_pod_template.html (right):

https://codereview.chromium.org/2293683002/diff/1/ui/login/account_picker/use...
ui/login/account_picker/user_pod_template.html:1: <style is="custom-style">
On 2016/08/31 00:52:29, Alexander Alekseev wrote:
> Please, move this style to a .css file. Our presubmit hooks are unable to
> analyze inlined css yet.

done per discussion

[Alexander Alekseev, 2016-08-31 05:06:39]

Thank you!

lgtm

[commit-bot: I haz the power, 2016-08-31 05:37:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-31 05:37:40]

Dry run: This issue passed the CQ dry run.

[Qiang(Joe) Xu, 2016-08-31 05:45:38]

The CQ bit was checked by warx@chromium.org

[commit-bot: I haz the power, 2016-08-31 05:45:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-31 05:49:35]

Description was changed from

==========
CrOS "remove this user" button should use red button style just like desktop
browser

Remove this user button for supervised user is changed to back bold font style
not by design. This CL restores the red button style for both removing non-owner
user and supervised user.

BUG=641137
TEST=device test. Screenshots are in crbug.com/641137.
==========

to

==========
CrOS "remove this user" button should use red button style just like desktop
browser

Remove this user button for supervised user is changed to back bold font style
not by design. This CL restores the red button style for both removing non-owner
user and supervised user.

BUG=641137
TEST=device test. Screenshots are in crbug.com/641137.
==========

[commit-bot: I haz the power, 2016-08-31 05:49:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-31 05:51:44]

Description was changed from

==========
CrOS "remove this user" button should use red button style just like desktop
browser

Remove this user button for supervised user is changed to back bold font style
not by design. This CL restores the red button style for both removing non-owner
user and supervised user.

BUG=641137
TEST=device test. Screenshots are in crbug.com/641137.
==========

to

==========
CrOS "remove this user" button should use red button style just like desktop
browser

Remove this user button for supervised user is changed to back bold font style
not by design. This CL restores the red button style for both removing non-owner
user and supervised user.

BUG=641137
TEST=device test. Screenshots are in crbug.com/641137.

Committed: https://crrev.com/b316f87f5c0d4bb686c66867dded237fcc2275e4
Cr-Commit-Position: refs/heads/master@{#415570}
==========

[commit-bot: I haz the power, 2016-08-31 05:51:45]

Patchset 2 (id:??) landed as
https://crrev.com/b316f87f5c0d4bb686c66867dded237fcc2275e4
Cr-Commit-Position: refs/heads/master@{#415570}