Don't shrink the message center height while open

ui/message_center/views/message_center_view.cc

Index: ui/message_center/views/message_center_view.cc
diff --git a/ui/message_center/views/message_center_view.cc b/ui/message_center/views/message_center_view.cc
index 375d42be82d5bd5532a66982c25208c3d9056774..ffe24d155d788e688067f40f1154ebed93925b2d 100644
--- a/ui/message_center/views/message_center_view.cc
+++ b/ui/message_center/views/message_center_view.cc
@@ -22,6 +22,7 @@
 #include "ui/message_center/message_center_style.h"
 #include "ui/message_center/message_center_tray.h"
 #include "ui/message_center/message_center_types.h"
+#include "ui/message_center/views/custom_notification_view.h"
 #include "ui/message_center/views/message_center_button_bar.h"
 #include "ui/message_center/views/message_list_view.h"
 #include "ui/message_center/views/message_view.h"
@@ -105,6 +106,7 @@ MessageCenterView::MessageCenterView(MessageCenter* message_center,
   scroller_->layer()->SetMasksToBounds(true);
 
   message_list_view_.reset(new MessageListView());
+  message_list_view_->set_scroller(scroller_);
   message_list_view_->set_owned_by_client();
   message_list_view_->AddObserver(this);
 
@@ -395,6 +397,15 @@ bool MessageCenterView::SetRepositionTarget() {
   if (message_list_view_->IsMouseHovered()) {
     for (const auto& hover_id_view : notification_views_) {
       MessageView* hover_view = hover_id_view.second;
+
+      // Skip if the view is a custom view. Custom views can be expanded by

[Eliot Courtney, 2017/04/11 00:58:26]

discussed offline: We want to keep the on-hover reposition target behaviour for
custom notifications as well, so we (probably?) still need to set the reposition
target here.

[yoshiki, 2017/04/11 07:46:26]

I just removed this, since the reason I added this code is fixed by a recent
patch in Android.

+      // click and we don't want to stick to the target on expansion by user
+      // action.
+      // TODO(yoshiki): Distinguish if the update event comes from user action
+      // or not, and skip only in the case of user action
+      if (hover_view->GetClassName() == CustomNotificationView::kViewClassName)
+        continue;
+
       if (hover_view->IsMouseHovered()) {
         message_list_view_->SetRepositionTarget(hover_view->bounds());
         return true;