diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-08 23:44:16 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-08 23:44:16 +0000 |
| commit | 8f85effedc610e44d394a73a1b2008bfcbd73a5d (patch) | |
| tree | f6cfeea58d371a02f55fe26cbb399a54a0233a14 /chrome/browser/extensions/api/web_navigation | |
| parent | 10c85f5dbf56f1310205f4cfe9673db6ccda2080 (diff) | |
| download | chromium_src-8f85effedc610e44d394a73a1b2008bfcbd73a5d.zip chromium_src-8f85effedc610e44d394a73a1b2008bfcbd73a5d.tar.gz chromium_src-8f85effedc610e44d394a73a1b2008bfcbd73a5d.tar.bz2 | |
Make ~WebNavigationEventRouter remove itself from browsers
I don't think this is related to the crash I'm seeing, but I noticed
it when looking for dangling TabStripModelObservers.
BUG=286162
TEST=none
R=yoz@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23547006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221959 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/api/web_navigation')
| -rw-r--r-- | chrome/browser/extensions/api/web_navigation/web_navigation_api.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc b/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc index 7e2c8db..ed2e61c 100644 --- a/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc +++ b/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc @@ -91,12 +91,13 @@ WebNavigationEventRouter::WebNavigationEventRouter(Profile* profile) content::NotificationService::AllSources()); BrowserList::AddObserver(this); - for (chrome::BrowserIterator it; !it.done(); it.Next()) { + for (chrome::BrowserIterator it; !it.done(); it.Next()) OnBrowserAdded(*it); - } } WebNavigationEventRouter::~WebNavigationEventRouter() { + for (chrome::BrowserIterator it; !it.done(); it.Next()) + OnBrowserRemoved(*it); BrowserList::RemoveObserver(this); } |