Marks notifications as shown slightly earlier for message center.

Current: all marked as read and shown when closed
New: all marked as shown when opened, read when closed

If there're notifications of 'shown=false' but the center actually shows,
the message center misunderstand the current status at OnNotificationClicked().

BUG=169388


Review URL: https://chromiumcodereview.appspot.com/11874015

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176827 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 12 insertions, 9 deletions

diff --git a/ui/message_center/notification_list.cc b/ui/message_center/notification_list.cc
index ac447fc..2975360 100644
--- a/ui/message_center/notification_list.cc
+++ b/ui/message_center/notification_list.cc
@@ -35,17 +35,20 @@ void NotificationList::SetMessageCenterVisible(bool visible) {
   if (message_center_visible_ == visible)
     return;
   message_center_visible_ = visible;
-  if (!visible) {
-    // When the list is hidden, clear the unread count, and mark all
-    // notifications as read and shown.
+  // 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)
     unread_count_ = 0;
-    for (NotificationMap::iterator mapiter = notifications_.begin();
-         mapiter != notifications_.end(); ++mapiter) {
-      for (Notifications::iterator iter = mapiter->second.begin();
-           iter != mapiter->second.end(); ++iter) {
-        iter->is_read = true;
+
+  for (NotificationMap::iterator mapiter = notifications_.begin();
+       mapiter != notifications_.end(); ++mapiter) {
+    for (Notifications::iterator iter = mapiter->second.begin();
+         iter != mapiter->second.end(); ++iter) {
+      if (visible)
         iter->shown_as_popup = true;
-      }
+      else
+        iter->is_read = true;
     }
   }
 }