Update the visuals of about:flags "Relaunch Now" button.

Update the visuals of about:flags "Relaunch Now" button.

Make it visually aligned with other buttons from chrome error
pages.  Make it easier to see by giving it more contrast.  Update
the formatting to remove an explicit height being set to give a
nicer look on various orientations of devices.

BUG=486816
Committed: https://crrev.com/bca23e588da39bd335338f3fdebe470eacd77e2a
Cr-Commit-Position: refs/heads/master@{#329271}

[Ted C, 2015-05-11 18:28:21]

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

[Ted C, 2015-05-11 18:28:22]

PTAL

[Dan Beam, 2015-05-11 21:32:31]

i think looks confusingly similar to infobars on mobile, but fellow co-workers
polled (estade@) didn't think it's an issue, so lgtm

https://codereview.chromium.org/1136133003/diff/1/chrome/browser/resources/fl...
File chrome/browser/resources/flags.css (right):

https://codereview.chromium.org/1136133003/diff/1/chrome/browser/resources/fl...
chrome/browser/resources/flags.css:168: update body.padding-bottom */
nit: continuations should have a * at the beginning
nit: comments should end with .

/* If you change properties that update the height of this,
 * update body.padding-bottom. */
 ^                           ^

[Dan Beam, 2015-05-11 21:32:51]

On 2015/05/11 21:32:31, Dan Beam wrote:
> i think looks confusingly similar to infobars on mobile, but fellow co-workers
> polled (estade@) didn't think it's an issue, so lgtm
> 
>
https://codereview.chromium.org/1136133003/diff/1/chrome/browser/resources/fl...
> File chrome/browser/resources/flags.css (right):
> 
>
https://codereview.chromium.org/1136133003/diff/1/chrome/browser/resources/fl...
> chrome/browser/resources/flags.css:168: update body.padding-bottom */
> nit: continuations should have a * at the beginning
> nit: comments should end with .
> 
> /* If you change properties that update the height of this,
>  * update body.padding-bottom. */
>  ^                           ^

i think the new UI** looks...

[Ted C, 2015-05-11 22:19:06]

I agree they do look like infobars, but we've trained our users to look at the
bottom for infobar-y things so it seemed like an acceptable trade off.

But improving the previous L&F was a pretty low bar to set for myself, so I will
certainly not be against to further changes to make it better.

https://codereview.chromium.org/1136133003/diff/1/chrome/browser/resources/fl...
File chrome/browser/resources/flags.css (right):

https://codereview.chromium.org/1136133003/diff/1/chrome/browser/resources/fl...
chrome/browser/resources/flags.css:168: update body.padding-bottom */
On 2015/05/11 21:32:31, Dan Beam wrote:
> nit: continuations should have a * at the beginning
> nit: comments should end with .
> 
> /* If you change properties that update the height of this,
>  * update body.padding-bottom. */
>  ^                           ^

Done.  Slightly changed the wording to not use update twice in the same
sentence.

[Ted C, 2015-05-11 22:19:32]

The CQ bit was checked by tedchoc@chromium.org

[Ted C, 2015-05-11 22:19:32]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/1136133003/#ps20001
(title: "address review comments")

[commit-bot: I haz the power, 2015-05-11 22:21:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1136133003/20001

[commit-bot: I haz the power, 2015-05-11 23:42:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-11 23:44:18]

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