Show message center on lock screen

ui/message_center/notification_list.cc

Index: ui/message_center/notification_list.cc
diff --git a/ui/message_center/notification_list.cc b/ui/message_center/notification_list.cc
index 14abd9b5c420b4a5994288acd9524248ec2a3ff5..a07d78c05f0683c376b9d2f637a9a4b112191f10 100644
--- a/ui/message_center/notification_list.cc
+++ b/ui/message_center/notification_list.cc
@@ -63,19 +63,16 @@ NotificationList::~NotificationList() {
   STLDeleteContainerPointers(notifications_.begin(), notifications_.end());
 }
 
-void NotificationList::SetMessageCenterVisible(
-    bool visible,
-    std::set<std::string>* updated_ids) {
-  if (message_center_visible_ == visible)
-    return;
-
+void NotificationList::SetMessageCenterVisible(bool visible) {
   message_center_visible_ = visible;

[dewittj, 2016/05/19 17:34:03]

This doesn't seem like the right place to track this bit anymore.

[yoshiki, 2016/05/24 17:00:14]

I moved the flag from NotificationList to MessageCenter. I think it's more
natural.

+}
 
-  if (!visible)
-    return;
+void NotificationList::SetNotificationsShown(

[yoshiki, 2016/05/16 19:04:46]

I splitted this method because I don't want to make notifications read when the
screen is locked.

+    const NotificationBlockers& blockers,

[yoshiki, 2016/05/16 19:04:46]

Adding blockers fixes the bug that this makes blocked notifications read.

+    std::set<std::string>* updated_ids) {
+  Notifications notifications = GetVisibleNotifications(blockers);
 
-  for (Notifications::iterator iter = notifications_.begin();
-       iter != notifications_.end(); ++iter) {
+  for (auto iter = notifications.begin(); iter != notifications.end(); ++iter) {
     Notification* notification = *iter;
     bool was_popup = notification->shown_as_popup();
     bool was_read = notification->IsRead();