Makes a gap at the top boundary of message center.

Makes a gap at the top boundary of message center.


BUG=174097
TEST=manually checked on goobuntu

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182307

[stevenjb, 2013-02-13 18:54:45]

https://codereview.chromium.org/12248012/diff/1/ash/system/web_notification/w...
File ash/system/web_notification/web_notification_tray.cc (right):

https://codereview.chromium.org/12248012/diff/1/ash/system/web_notification/w...
ash/system/web_notification/web_notification_tray.cc:148: max_height -=
kTraySpacing;
We should just subtract kTraySpacing when appropriate and clamp to 0. As
written, max_height==8 -> 1, but 7 -> 7.

[Jun Mukai, 2013-02-13 19:42:27]

https://codereview.chromium.org/12248012/diff/1/ash/system/web_notification/w...
File ash/system/web_notification/web_notification_tray.cc (right):

https://codereview.chromium.org/12248012/diff/1/ash/system/web_notification/w...
ash/system/web_notification/web_notification_tray.cc:148: max_height -=
kTraySpacing;
On 2013/02/13 18:54:45, stevenjb (chromium) wrote:
> We should just subtract kTraySpacing when appropriate and clamp to 0. As
> written, max_height==8 -> 1, but 7 -> 7.

used std::max(0, ...)

[stevenjb, 2013-02-13 19:44:25]

lgtm

[commit-bot: I haz the power, 2013-02-13 19:46:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mukai@chromium.org/12248012/4001

[commit-bot: I haz the power, 2013-02-13 21:11:09]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...