Not Remove Non-Closable Arc Popup When Close Button is Pressed

ui/arc/notification/arc_custom_notification_view.cc

Index: ui/arc/notification/arc_custom_notification_view.cc
diff --git a/ui/arc/notification/arc_custom_notification_view.cc b/ui/arc/notification/arc_custom_notification_view.cc
index cdb2c7da5a0f59fe3ac3440d5c51865e94e37ee3..617f920fe0d62cd97f0ea90696c4541f682d2b15 100644
--- a/ui/arc/notification/arc_custom_notification_view.cc
+++ b/ui/arc/notification/arc_custom_notification_view.cc
@@ -222,8 +222,10 @@ class ArcCustomNotificationView::CloseButton : public views::ImageButton {
 };
 
 ArcCustomNotificationView::ArcCustomNotificationView(
-    ArcCustomNotificationItem* item)
+    ArcCustomNotificationItem* item,
+    message_center::MessageView* parent)
     : item_(item),
+      parent_(parent),
       notification_key_(item->notification_key()),
       event_forwarder_(new EventForwarder(this)) {
   SetFocusBehavior(FocusBehavior::ALWAYS);
@@ -504,7 +506,7 @@ views::FocusTraversable* ArcCustomNotificationView::GetFocusTraversable() {
 void ArcCustomNotificationView::ButtonPressed(views::Button* sender,
                                               const ui::Event& event) {
   if (item_ && !item_->pinned() && sender == floating_close_button_) {
-    item_->CloseFromCloseButton();
+    parent_->OnCloseButtonPressed();

[dewittj, 2017/02/07 00:03:11]

any reason you didn't just use views::View::parent()?  If it's about casting,
I'd like to see a DCHECK(parent_ == parent()) so that any hierarchy mistakes are
apparent.

[yoshiki, 2017/02/22 08:55:23]

Thanks. I changed the code with using parent(). PTAL again?

   }
 }