diff options
| author | dewittj@chromium.org <dewittj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-24 06:59:36 +0000 |
|---|---|---|
| committer | dewittj@chromium.org <dewittj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-24 06:59:36 +0000 |
| commit | ae37e7a768dec3692bef8214160bd9ee0e34a4e5 (patch) | |
| tree | f5d8d27f83b69bb4f72b3025d9766c32a84cbc85 /ui/message_center/notification_list.cc | |
| parent | 02e59b133c90bd62b1133673d867bfc7f7325aee (diff) | |
| download | chromium_src-ae37e7a768dec3692bef8214160bd9ee0e34a4e5.zip chromium_src-ae37e7a768dec3692bef8214160bd9ee0e34a4e5.tar.gz chromium_src-ae37e7a768dec3692bef8214160bd9ee0e34a4e5.tar.bz2 | |
Queue notification center deletes while visible to the user.
This extends the notification center queueing system to support deletes.
Formerly they would just pass through to the views, causing re-rendering
artifacts.
R=jianli@chromium.org
TBR=oshima@chromium.org,stevenjb@chromium.org,benwells@chromium.org
BUG=267066
Review URL: https://chromiumcodereview.appspot.com/24258004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224931 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/message_center/notification_list.cc')
| -rw-r--r-- | ui/message_center/notification_list.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ui/message_center/notification_list.cc b/ui/message_center/notification_list.cc index bed592c..73ea08b 100644 --- a/ui/message_center/notification_list.cc +++ b/ui/message_center/notification_list.cc @@ -113,6 +113,9 @@ void NotificationList::UpdateNotificationMessage( Notification* old = *iter; notifications_.erase(iter); delete old; + + // We really don't want duplicate IDs. + DCHECK(GetNotification(new_notification->id()) == notifications_.end()); notifications_.insert(new_notification.release()); } |