Sends updated event for click on notifications.

ui/message_center/views/message_popup_collection.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/message_center/views/message_popup_collection.h"
 
 #include <set>
 
 #include "base/bind.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 369 matching lines @@
   if (by_user && widgets_went_empty)
     RepositionWidgetsWithTarget(removed_bounds);
 }
 
 void MessagePopupCollection::OnNotificationUpdated(
     const std::string& notification_id) {
   ToastContainer::iterator toast_iter = toasts_.find(notification_id);
   if (toast_iter == toasts_.end())
     return;
 
-  NotificationList::Notifications notifications =
-      message_center_->GetNotifications();
+  NotificationList::PopupNotifications notifications =
+      message_center_->GetPopupNotifications();
   bool updated = false;
-  for (NotificationList::Notifications::iterator iter =
+  for (NotificationList::PopupNotifications::iterator iter =
            notifications.begin(); iter != notifications.end(); ++iter) {
     if ((*iter)->id() != notification_id)
       continue;
 
     MessageView* view = NotificationView::Create(
         *(*iter), message_center_, true);
     toast_iter->second->SetContents(view);
     toast_iter->second->ResetTimeout((*iter)->priority());
     toast_iter->second->RestartTimer();
     updated = true;
   }
 
-  if (updated) {
-    RepositionWidgets();
-    // Reposition could create extra space which allows additional widgets.
-    UpdateWidgets();
+  // Updated() can be called when a notification is excluded from the popup

[dewittj, 2013/04/18 17:40:51]

nit: OnNotificationUpdated() - it made me wonder if there was a function called
Updated()

[Jun Mukai, 2013/04/18 20:22:12]

Done.

+  // notification list but still remains in the full notification list.
+  // In that case the widget for the notification has to be closed here.
+  if (!updated) {
+    views::Widget* widget = toast_iter->second->GetWidget();
+    widget->RemoveObserver(this);
+    widgets_.erase(std::find(widgets_.begin(), widgets_.end(), widget));
+    widget->Close();
+    toasts_.erase(toast_iter);
   }
+
+  RepositionWidgets();
+  // Reposition could create extra space which allows additional widgets.
+  UpdateWidgets();
 }
 
 void MessagePopupCollection::SetWorkAreaForTest(const gfx::Rect& work_area) {
   work_area_ = work_area;
 }
 
+views::Widget* MessagePopupCollection::GetWidgetForId(const std::string& id) {
+  ToastContainer::const_iterator iter = toasts_.find(id);
+  return (iter == toasts_.end()) ? NULL : iter->second->GetWidget();
+}
+
 }  // namespace message_center