Sends Updated when message center is closed.

Sends Updated when message center is closed.

Because open/close will change the notification status (read/shown),
it should send Updated to the observers.

BUG=232030
TEST=manually (see repro steps in the bug)
R=dewittj@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194698

[dewittj, 2013-04-16 23:59:12]

lgtm with nits

https://codereview.chromium.org/14299007/diff/1/ui/message_center/notificatio...
File ui/message_center/notification_list.cc (right):

https://codereview.chromium.org/14299007/diff/1/ui/message_center/notificatio...
ui/message_center/notification_list.cc:57: std::set<std::string> dummy_ids;
I don't love keeping a dummy data structure around if it's unnecessary.  Prefer
a NULL check in the loop below?

https://codereview.chromium.org/14299007/diff/1/ui/message_center/notificatio...
ui/message_center/notification_list.cc:65: if (!visible) {
no braces

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

https://codereview.chromium.org/14299007/diff/1/ui/message_center/notificatio...
File ui/message_center/notification_list.cc (right):

https://codereview.chromium.org/14299007/diff/1/ui/message_center/notificatio...
ui/message_center/notification_list.cc:57: std::set<std::string> dummy_ids;
On 2013/04/16 23:59:13, dewittj wrote:
> I don't love keeping a dummy data structure around if it's unnecessary. 
Prefer
> a NULL check in the loop below?

Done.

https://codereview.chromium.org/14299007/diff/1/ui/message_center/notificatio...
ui/message_center/notification_list.cc:65: if (!visible) {
On 2013/04/16 23:59:13, dewittj wrote:
> no braces

Done.

[Jun Mukai, 2013-04-17 21:52:22]

Committed patchset #2 manually as r194698 (presubmit successful).