Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation dialog.

Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation dialog by adding a tint of rgba(0, 0, 0, .12) over the existing background color when the buttons are focused.

BUG=604350, 595349
Committed: https://crrev.com/aa33561a9697af09418ce35fb5876aea79256307
Cr-Commit-Position: refs/heads/master@{#390465}

[Moe, 2016-04-21 18:32:52]

Description was changed from

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog.

BUG=604350, 595349
==========

to

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog.

BUG=604350, 595349

Polymer sets "font-weight: bold" on the host element when it has focus using the
following selector:

:host.keyboard-focus {...}

which is less specific than:

#confirmButton, #undoButton {...}
==========

[Moe, 2016-04-21 18:32:53]

mahmadi@chromium.org changed reviewers:
+ dbeam@chromium.org

[Moe, 2016-04-21 18:34:19]

Hi Dan, Could you please take a look at this?

[Dan Beam, 2016-04-21 18:44:10]

lgtm but `bold` looks bad :(

[Moe, 2016-04-22 15:11:33]

Dan, it should look a lot nicer now. This patch also fixes the spacing between
the buttons that broke in:

https://chromium.googlesource.com/chromium/src/+/e2d047d6a1128c490aa0f0a9c435...

[Moe, 2016-04-27 18:29:49]

Patchset #2 (id:20001) has been deleted

[Moe, 2016-04-27 18:32:26]

Hi Dan, I replaced Patch #2 with an updated one with "cleaner" css.

This patch also fixes the margin between the buttons on Mac and Linux platforms.
Thank you.

[Moe, 2016-04-28 19:11:13]

mahmadi@chromium.org changed reviewers:
+ michaelpg@chromium.org

[michaelpg, 2016-04-28 19:20:15]

lgtm but:

1. change the description (not bold anymore)

On 2016/04/27 18:32:26, Moe wrote:
> Hi Dan, I replaced Patch #2 with an updated one with "cleaner" css.
> 
> This patch also fixes the margin between the buttons on Mac and Linux
platforms.

2. it would be nicer if the button margin fix were landed separately (up to you,
i'm available to stamp immediately if you upload another cl)

> Thank you.

[Moe, 2016-04-28 19:32:34]

Description was changed from

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog.

BUG=604350, 595349

Polymer sets "font-weight: bold" on the host element when it has focus using the
following selector:

:host.keyboard-focus {...}

which is less specific than:

#confirmButton, #undoButton {...}
==========

to

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog by adding a tint of rgba(0, 0, 0, .12) over the existing background color
when the buttons are focused.

BUG=604350, 595349
==========

[Moe, 2016-04-28 20:31:33]

The CQ bit was checked by mahmadi@chromium.org

[Moe, 2016-04-28 20:31:34]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org,
michaelpg@chromium.org
Link to the patchset: https://codereview.chromium.org/1910953003/#ps60001
(title: "Addressed Michael's comments")

[commit-bot: I haz the power, 2016-04-28 20:32:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1910953003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1910953003/60001

[commit-bot: I haz the power, 2016-04-28 20:38:28]

Description was changed from

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog by adding a tint of rgba(0, 0, 0, .12) over the existing background color
when the buttons are focused.

BUG=604350, 595349
==========

to

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog by adding a tint of rgba(0, 0, 0, .12) over the existing background color
when the buttons are focused.

BUG=604350, 595349
==========

[commit-bot: I haz the power, 2016-04-28 20:38:29]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-04-30 17:20:45]

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

[commit-bot: I haz the power, 2016-04-30 17:57:15]

Description was changed from

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog by adding a tint of rgba(0, 0, 0, .12) over the existing background color
when the buttons are focused.

BUG=604350, 595349
==========

to

==========
Fixes the broken focus issue of 'OK, GOT IT' and 'UNDO' in the sync confirmation
dialog by adding a tint of rgba(0, 0, 0, .12) over the existing background color
when the buttons are focused.

BUG=604350, 595349

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