Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5865)

Unified Diff: chrome/browser/alternate_nav_url_fetcher.cc

Issue 10079023: Move notifications used only in chrome/ out of content/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: six! Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/alternate_nav_url_fetcher.cc
diff --git a/chrome/browser/alternate_nav_url_fetcher.cc b/chrome/browser/alternate_nav_url_fetcher.cc
index e346278cb1344ac9e00ceff27fe90ca9fb4ff9ed..8e3d997c7afc2cb7fa74395d2146929e710114b5 100644
--- a/chrome/browser/alternate_nav_url_fetcher.cc
+++ b/chrome/browser/alternate_nav_url_fetcher.cc
@@ -161,7 +161,7 @@ void AlternateNavURLFetcher::Observe(
// WARNING: |this| may be deleted!
break;
- case content::NOTIFICATION_TAB_CLOSED:
+ case content::NOTIFICATION_WEB_CONTENTS_DESTROYED:
// We have been closed. In order to prevent the URLFetcher from trying to
// access the controller that will be invalid, we delete ourselves.
// This deletes the URLFetcher and insures its callback won't be called.
@@ -184,8 +184,10 @@ void AlternateNavURLFetcher::OnURLFetchComplete(
void AlternateNavURLFetcher::StartFetch(NavigationController* controller) {
controller_ = controller;
- registrar_.Add(this, content::NOTIFICATION_TAB_CLOSED,
- content::Source<NavigationController>(controller_));
+ registrar_.Add(
+ this,
+ content::NOTIFICATION_WEB_CONTENTS_DESTROYED,
+ content::Source<content::WebContents>(controller_->GetWebContents()));
DCHECK_EQ(NOT_STARTED, state_);
state_ = IN_PROGRESS;

Powered by Google App Engine
This is Rietveld 408576698