Update progress bar notification UI on Windows per latest UI decision.

Update progress bar notification UI on Windows per latest UI decision.

BUG=170924
TEST=Manual test by creating progress notifications

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

[dewittj, 2013-08-05 22:34:39]

doesn't seem to include updates to the constants?

https://codereview.chromium.org/22277003/diff/1/ui/message_center/views/notif...
File ui/message_center/views/notification_view.cc (right):

https://codereview.chromium.org/22277003/diff/1/ui/message_center/views/notif...
ui/message_center/views/notification_view.cc:506:
message_center::kProgressBarTopPadding, kProgressBarBottomPadding));
Why move one to message_center and not the other?

[jianli, 2013-08-05 22:42:16]

The constant updates are included in my previous Mac patch.

https://codereview.chromium.org/22277003/diff/1/ui/message_center/views/notif...
File ui/message_center/views/notification_view.cc (right):

https://codereview.chromium.org/22277003/diff/1/ui/message_center/views/notif...
ui/message_center/views/notification_view.cc:506:
message_center::kProgressBarTopPadding, kProgressBarBottomPadding));
On 2013/08/05 22:34:39, dewittj wrote:
> Why move one to message_center and not the other?

We do not need extra bottom padding for progress bar since the bottom padding is
indeed provided by kTextBottomPadding at line 467.

[dewittj, 2013-08-05 22:43:38]

lgtm

[commit-bot: I haz the power, 2013-08-05 22:55:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jianli@chromium.org/22277003/1

[commit-bot: I haz the power, 2013-08-05 23:28:38]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-08-05 23:49:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jianli@chromium.org/22277003/18003

[commit-bot: I haz the power, 2013-08-06 06:31:32]

Change committed as 215827