MD - use new style of text button in infobars.

MD - use new style of text button in infobars.

BUG=520266
Committed: https://crrev.com/d034150c37234dfa2eedf8fde15f9fedd8c3cd10
Cr-Commit-Position: refs/heads/master@{#380410}

[Evan Stade, 2016-03-09 03:48:49]

estade@chromium.org changed reviewers:
+ pkasting@chromium.org

[Peter Kasting, 2016-03-09 07:18:10]

https://codereview.chromium.org/1777813002/diff/1/chrome/browser/ui/views/inf...
File chrome/browser/ui/views/infobars/confirm_infobar.cc (right):

https://codereview.chromium.org/1777813002/diff/1/chrome/browser/ui/views/inf...
chrome/browser/ui/views/infobars/confirm_infobar.cc:74: // Use a label button
even in MD mode as MD buttons don't support icons.
Has this been fixed?  We do still want the button to show an elevation icon if
need be.

If so this comment should be removed.

[Evan Stade, 2016-03-10 02:41:42]

https://codereview.chromium.org/1777813002/diff/1/chrome/browser/ui/views/inf...
File chrome/browser/ui/views/infobars/confirm_infobar.cc (right):

https://codereview.chromium.org/1777813002/diff/1/chrome/browser/ui/views/inf...
chrome/browser/ui/views/infobars/confirm_infobar.cc:74: // Use a label button
even in MD mode as MD buttons don't support icons.
On 2016/03/09 07:18:10, Peter Kasting wrote:
> Has this been fixed?  We do still want the button to show an elevation icon if
> need be.
> 
> If so this comment should be removed.

Right, good call, comment removed and code cleaned up. I couldn't test for real
because this only works on windows but it looks good when I stick a stand-in
image in there.

[Evan Stade, 2016-03-10 02:41:47]

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

[commit-bot: I haz the power, 2016-03-10 02:42:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1777813002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1777813002/20001

[commit-bot: I haz the power, 2016-03-10 03:50:52]

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

[commit-bot: I haz the power, 2016-03-10 03:50:53]

Dry run: This issue passed the CQ dry run.

[Peter Kasting, 2016-03-10 04:41:25]

LGTM

[Evan Stade, 2016-03-10 15:22:56]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2016-03-10 15:23:14]

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

[commit-bot: I haz the power, 2016-03-10 15:28:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-10 15:29:14]

Description was changed from

==========
MD - use new style of text button in infobars.

BUG=520266
==========

to

==========
MD - use new style of text button in infobars.

BUG=520266

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

[commit-bot: I haz the power, 2016-03-10 15:29:15]

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