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

Unified Diff: chrome/browser/instant/instant_loader.cc

Issue 3536017: Fixes uneeded delete in InstantLoader. This is no longer needed as (Closed)
Patch Set: Needed to merge Created 10 years, 2 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/instant/instant_loader.cc
diff --git a/chrome/browser/instant/instant_loader.cc b/chrome/browser/instant/instant_loader.cc
index 0ab07310c455ec19495a814777dd6963dfec463c..eb33c3c38cf7cd8c4f25fea4d83fb9ad209f7397 100644
--- a/chrome/browser/instant/instant_loader.cc
+++ b/chrome/browser/instant/instant_loader.cc
@@ -130,8 +130,6 @@ class InstantLoader::FrameLoadObserver : public NotificationObserver {
execute_js_id_(0) {
registrar_.Add(this, NotificationType::LOAD_COMPLETED_MAIN_FRAME,
Source<TabContents>(tab_contents_));
- registrar_.Add(this, NotificationType::TAB_CONTENTS_DESTROYED,
- Source<TabContents>(tab_contents_));
}
// Sets the text to send to the page.
@@ -173,10 +171,6 @@ class InstantLoader::FrameLoadObserver : public NotificationObserver {
return;
}
- case NotificationType::TAB_CONTENTS_DESTROYED:
- delete this;
- return;
-
default:
NOTREACHED();
break;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698