Sends Updated when message center is closed.

ui/message_center/notification_list.cc

 // Copyright (c) 2012 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/notification_list.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "base/time.h"
@@ skipping 30 matching lines @@
 NotificationList::NotificationList()
     : message_center_visible_(false),
       unread_count_(0),
       quiet_mode_(false) {
 }
 
 NotificationList::~NotificationList() {
   STLDeleteContainerPointers(notifications_.begin(), notifications_.end());
 }
 
-void NotificationList::SetMessageCenterVisible(bool visible) {
+void NotificationList::SetMessageCenterVisible(
+    bool visible,
+    std::set<std::string>* updated_ids) {
   if (message_center_visible_ == visible)
     return;
+
+  std::set<std::string> dummy_ids;

[dewittj, 2013/04/16 23:59:13]

I don't love keeping a dummy data structure around if it's unnecessary.  Prefer
a NULL check in the loop below?

[Jun Mukai, 2013/04/17 01:04:43]

Done.

+  if (!updated_ids)
+    updated_ids = &dummy_ids;
+
   message_center_visible_ = visible;
   // When the center appears, mark all notifications as shown, and
   // when the center is hidden, clear the unread count, and mark all
   // notifications as read.
-  if (!visible)
+  if (!visible) {

[dewittj, 2013/04/16 23:59:13]

no braces

[Jun Mukai, 2013/04/17 01:04:43]

Done.

     unread_count_ = 0;
+  }
 
   for (Notifications::iterator iter = notifications_.begin();
        iter != notifications_.end(); ++iter) {
-    if (visible)
+    if (visible) {
+      if (!(*iter)->shown_as_popup())
+        updated_ids->insert((*iter)->id());
       (*iter)->set_shown_as_popup(true);
-    else
+    } else {
+      if (!(*iter)->is_read())
+        updated_ids->insert((*iter)->id());
       (*iter)->set_is_read(true);
+    }
   }
 }
 
 void NotificationList::AddNotification(
     NotificationType type,
     const std::string& id,
     const string16& title,
     const string16& message,
     const string16& display_source,
     const std::string& extension_id,
@@ skipping 255 matching lines @@
     notification->set_shown_as_popup(quiet_mode_);
     if (!quiet_mode_ && notification->priority() > MIN_PRIORITY)
         ++unread_count_;
   }
   // Take ownership. The notification can only be removed from the list
   // in EraseNotification(), which will delete it.
   notifications_.insert(notification.release());
 }
 
 }  // namespace message_center