diff options
| author | johnnyg@chromium.org <johnnyg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-09 21:54:07 +0000 |
|---|---|---|
| committer | johnnyg@chromium.org <johnnyg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-09 21:54:07 +0000 |
| commit | 25270953f01d7b154d3ba86f1763f762f8138883 (patch) | |
| tree | 1bb4ae27b0254b8690c41e52b290ffb2365619ff /chrome/browser/notifications | |
| parent | e5fb9e30cb63487f00362f02db5364fb7ad8c544 (diff) | |
| download | chromium_src-25270953f01d7b154d3ba86f1763f762f8138883.zip chromium_src-25270953f01d7b154d3ba86f1763f762f8138883.tar.gz chromium_src-25270953f01d7b154d3ba86f1763f762f8138883.tar.bz2 | |
Stray ++ causes crashes. Don't walk off the end of the array.
BUG=66100
TEST=Have a notification open *not* from extension X, uninstall extension X.
Review URL: http://codereview.chromium.org/5746001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68771 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/notifications')
| -rw-r--r-- | chrome/browser/notifications/notification_ui_manager.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/notifications/notification_ui_manager.cc b/chrome/browser/notifications/notification_ui_manager.cc index c641b05..6e2ff02 100644 --- a/chrome/browser/notifications/notification_ui_manager.cc +++ b/chrome/browser/notifications/notification_ui_manager.cc @@ -88,7 +88,7 @@ bool NotificationUIManager::CancelAllBySourceOrigin(const GURL& source) { // because there may be multiple notifications from the same source. bool removed = false; NotificationDeque::iterator iter; - for (iter = show_queue_.begin(); iter != show_queue_.end(); ++iter) { + for (iter = show_queue_.begin(); iter != show_queue_.end();) { if ((*iter)->notification().origin_url() == source) { iter = show_queue_.erase(iter); removed = true; |