[Mac] Increase infobar height under Material Design.

[Mac] Increase infobar height under Material Design.

Under Material Design the infobar height has increased slightly.

R=avi@chromium.org
BUG=593664
Committed: https://crrev.com/7fada7682f1b88e2615ef3261ed86e0b0a5f2765
Cr-Commit-Position: refs/heads/master@{#387146}

[shrike, 2016-04-13 21:15:43]

PTAL

[Avi (use Gerrit), 2016-04-13 21:35:00]

lgtm

https://codereview.chromium.org/1885903003/diff/1/chrome/browser/ui/cocoa/inf...
File chrome/browser/ui/cocoa/infobars/infobar_controller.mm (right):

https://codereview.chromium.org/1885903003/diff/1/chrome/browser/ui/cocoa/inf...
chrome/browser/ui/cocoa/infobars/infobar_controller.mm:86: }
Joy...

It's a race now between the switch to MD and the switch to Views. Are you going
to have to go in and tweak all the nibs when we switch to MD, or will Views hit
first and let you avoid the hassle? :)

[shrike, 2016-04-13 22:40:10]

On 2016/04/13 21:35:00, Avi wrote:
> lgtm
> 
>
https://codereview.chromium.org/1885903003/diff/1/chrome/browser/ui/cocoa/inf...
> File chrome/browser/ui/cocoa/infobars/infobar_controller.mm (right):
> 
>
https://codereview.chromium.org/1885903003/diff/1/chrome/browser/ui/cocoa/inf...
> chrome/browser/ui/cocoa/infobars/infobar_controller.mm:86: }
> Joy...
> 
> It's a race now between the switch to MD and the switch to Views. Are you
going
> to have to go in and tweak all the nibs when we switch to MD, or will Views
hit
> first and let you avoid the hassle? :)

I actually think I'm nearly done with nib tweaks (which are pretty much changes
in places like awakeFromNib to change buttons sizes, etc.). Switching to Views's
MD stuff will not be a cakewalk because the MD design for Mac is different than
Windows/Linux for things like animations and certain interactions. Basically I
will have to review all of the Views MD stuff and modify some of it. But we're a
long way from Views in the Mac top chrome.

[shrike, 2016-04-13 22:40:13]

The CQ bit was checked by shrike@chromium.org

[commit-bot: I haz the power, 2016-04-13 22:40:38]

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

[commit-bot: I haz the power, 2016-04-13 23:35:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-13 23:36:13]

Description was changed from

==========
[Mac] Increase infobar height under Material Design.

Under Material Design the infobar height has increased slightly.

R=avi@chromium.org
BUG=593664
==========

to

==========
[Mac] Increase infobar height under Material Design.

Under Material Design the infobar height has increased slightly.

R=avi@chromium.org
BUG=593664

Committed: https://crrev.com/7fada7682f1b88e2615ef3261ed86e0b0a5f2765
Cr-Commit-Position: refs/heads/master@{#387146}
==========

[commit-bot: I haz the power, 2016-04-13 23:36:14]

Patchset 1 (id:??) landed as
https://crrev.com/7fada7682f1b88e2615ef3261ed86e0b0a5f2765
Cr-Commit-Position: refs/heads/master@{#387146}