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

Side by Side Diff: chrome/browser/ui/unload_controller.h

Issue 11359201: Remove TabContents from TabStripModelObserver::TabReplacedAt. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/ui/toolbar/wrench_menu_model.cc ('k') | chrome/browser/ui/unload_controller.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_UNLOAD_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_UI_UNLOAD_CONTROLLER_H_
6 #define CHROME_BROWSER_UI_UNLOAD_CONTROLLER_H_ 6 #define CHROME_BROWSER_UI_UNLOAD_CONTROLLER_H_
7 7
8 #include <set> 8 #include <set>
9 9
10 #include "base/memory/weak_ptr.h" 10 #include "base/memory/weak_ptr.h"
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 const content::NotificationSource& source, 70 const content::NotificationSource& source,
71 const content::NotificationDetails& details) OVERRIDE; 71 const content::NotificationDetails& details) OVERRIDE;
72 72
73 // Overridden from TabStripModelObserver: 73 // Overridden from TabStripModelObserver:
74 virtual void TabInsertedAt(content::WebContents* contents, 74 virtual void TabInsertedAt(content::WebContents* contents,
75 int index, 75 int index,
76 bool foreground) OVERRIDE; 76 bool foreground) OVERRIDE;
77 virtual void TabDetachedAt(content::WebContents* contents, 77 virtual void TabDetachedAt(content::WebContents* contents,
78 int index) OVERRIDE; 78 int index) OVERRIDE;
79 virtual void TabReplacedAt(TabStripModel* tab_strip_model, 79 virtual void TabReplacedAt(TabStripModel* tab_strip_model,
80 TabContents* old_contents, 80 content::WebContents* old_contents,
81 TabContents* new_contents, 81 content::WebContents* new_contents,
82 int index) OVERRIDE; 82 int index) OVERRIDE;
83 virtual void TabStripEmpty() OVERRIDE; 83 virtual void TabStripEmpty() OVERRIDE;
84 84
85 void TabAttachedImpl(content::WebContents* contents); 85 void TabAttachedImpl(content::WebContents* contents);
86 void TabDetachedImpl(content::WebContents* contents); 86 void TabDetachedImpl(content::WebContents* contents);
87 87
88 // Processes the next tab that needs it's beforeunload/unload event fired. 88 // Processes the next tab that needs it's beforeunload/unload event fired.
89 void ProcessPendingTabs(); 89 void ProcessPendingTabs();
90 90
91 // Whether we've completed firing all the tabs' beforeunload/unload events. 91 // Whether we've completed firing all the tabs' beforeunload/unload events.
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 bool is_attempting_to_close_browser_; 130 bool is_attempting_to_close_browser_;
131 131
132 base::WeakPtrFactory<UnloadController> weak_factory_; 132 base::WeakPtrFactory<UnloadController> weak_factory_;
133 133
134 DISALLOW_COPY_AND_ASSIGN(UnloadController); 134 DISALLOW_COPY_AND_ASSIGN(UnloadController);
135 }; 135 };
136 136
137 } // namespace chrome 137 } // namespace chrome
138 138
139 #endif // CHROME_BROWSER_UI_UNLOAD_CONTROLLER_H_ 139 #endif // CHROME_BROWSER_UI_UNLOAD_CONTROLLER_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/toolbar/wrench_menu_model.cc ('k') | chrome/browser/ui/unload_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698