CL: https://codereview.chromium.org/2805143002/

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8057405c22f4ca9727ab53b845ee87ed1d865a73

commit 8057405c22f4ca9727ab53b845ee87ed1d865a73
Author: yoshiki <yoshiki@chromium.org>
Date: Tue Apr 18 09:20:28 2017

Don't shrink the message center height while open

Previously, the height of the message center shrank if necessary. But this patch changed this behavior and made it not shrank while the message center opens.

This patch also changes animation. Previously, the notifications above the target were animated. With this patch, the notifications below the target are animated. This is because now we don't update the height of message center so animating below is natural.

The reason why we don't use AnimateNotificationsBelowTarget() is that it has a bug on removing notification.

BUG= 709337 
BUG=b/36517819
TEST=manual
TEST=unittests pass

Review-Url: https://codereview.chromium.org/2805143002
Cr-Commit-Position: refs/heads/master@{#465184}

[modify] https://crrev.com/8057405c22f4ca9727ab53b845ee87ed1d865a73/ui/message_center/views/message_center_view.cc
[modify] https://crrev.com/8057405c22f4ca9727ab53b845ee87ed1d865a73/ui/message_center/views/message_center_view_unittest.cc
[modify] https://crrev.com/8057405c22f4ca9727ab53b845ee87ed1d865a73/ui/message_center/views/message_list_view.cc
[modify] https://crrev.com/8057405c22f4ca9727ab53b845ee87ed1d865a73/ui/message_center/views/message_list_view.h
[modify] https://crrev.com/8057405c22f4ca9727ab53b845ee87ed1d865a73/ui/message_center/views/message_list_view_unittest.cc

Can I merge it to M58?

This bug requires manual review: We are only 6 days from stable.
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/000f488c2c280953accc9f9e872f8e0912969dc6

commit 000f488c2c280953accc9f9e872f8e0912969dc6
Author: yoshiki iguchi <yoshiki@chromium.org>
Date: Thu Apr 20 08:37:47 2017

Don't shrink the message center height while open

Previously, the height of the message center shrank if necessary. But this patch changed this behavior and made it not shrank while the message center opens.

This patch also changes animation. Previously, the notifications above the target were animated. With this patch, the notifications below the target are animated. This is because now we don't update the height of message center so animating below is natural.

The reason why we don't use AnimateNotificationsBelowTarget() is that it has a bug on removing notification.

BUG= 709337 
BUG=b/36517819
TEST=manual
TEST=unittests pass

Review-Url: https://codereview.chromium.org/2805143002
Cr-Commit-Position: refs/heads/master@{#465184}
(cherry picked from commit 8057405c22f4ca9727ab53b845ee87ed1d865a73)

Review-Url: https://codereview.chromium.org/2827253002 .
Cr-Commit-Position: refs/branch-heads/3029@{#755}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/000f488c2c280953accc9f9e872f8e0912969dc6/ui/message_center/views/message_center_view.cc
[modify] https://crrev.com/000f488c2c280953accc9f9e872f8e0912969dc6/ui/message_center/views/message_center_view_unittest.cc
[modify] https://crrev.com/000f488c2c280953accc9f9e872f8e0912969dc6/ui/message_center/views/message_list_view.cc
[modify] https://crrev.com/000f488c2c280953accc9f9e872f8e0912969dc6/ui/message_center/views/message_list_view.h
[modify] https://crrev.com/000f488c2c280953accc9f9e872f8e0912969dc6/ui/message_center/views/message_list_view_unittest.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4595f544ddf229fe129c50b890b672073206e33b

commit 4595f544ddf229fe129c50b890b672073206e33b
Author: yoshiki iguchi <yoshiki@chromium.org>
Date: Thu Apr 20 08:41:46 2017

Don't shrink the message center height while open

Previously, the height of the message center shrank if necessary. But this patch changed this behavior and made it not shrank while the message center opens.

This patch also changes animation. Previously, the notifications above the target were animated. With this patch, the notifications below the target are animated. This is because now we don't update the height of message center so animating below is natural.

The reason why we don't use AnimateNotificationsBelowTarget() is that it has a bug on removing notification.

BUG= 709337 
BUG=b/36517819
TEST=manual
TEST=unittests pass

Review-Url: https://codereview.chromium.org/2805143002
Cr-Commit-Position: refs/heads/master@{#465184}
(cherry picked from commit 8057405c22f4ca9727ab53b845ee87ed1d865a73)

Review-Url: https://codereview.chromium.org/2831063002 .
Cr-Commit-Position: refs/branch-heads/3071@{#79}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/4595f544ddf229fe129c50b890b672073206e33b/ui/message_center/views/message_center_view.cc
[modify] https://crrev.com/4595f544ddf229fe129c50b890b672073206e33b/ui/message_center/views/message_center_view_unittest.cc
[modify] https://crrev.com/4595f544ddf229fe129c50b890b672073206e33b/ui/message_center/views/message_list_view.cc
[modify] https://crrev.com/4595f544ddf229fe129c50b890b672073206e33b/ui/message_center/views/message_list_view.h
[modify] https://crrev.com/4595f544ddf229fe129c50b890b672073206e33b/ui/message_center/views/message_list_view_unittest.cc

9334.69.0, 58.0.3029.112 caroline