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

Unified Diff: chrome/browser/repost_form_warning_controller.h

Issue 1657005: Merge 44481 - Ensure proper teardown of repost form warning objects.... (Closed) Base URL: svn://svn.chromium.org/chrome/branches/375/src/
Patch Set: Created 10 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
« no previous file with comments | « chrome/browser/gtk/repost_form_warning_gtk.cc ('k') | chrome/browser/repost_form_warning_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/repost_form_warning_controller.h
===================================================================
--- chrome/browser/repost_form_warning_controller.h (revision 44638)
+++ chrome/browser/repost_form_warning_controller.h (working copy)
@@ -13,11 +13,11 @@
// This class is used to continue or cancel a pending reload when the
// repost form warning is shown. It is owned by the platform-dependent
-// |RepostFormWarning{Gtk,Mac,View}| classes and deletes itself after closing
-// the dialog.
+// |RepostFormWarning{Gtk,Mac,View}| classes.
class RepostFormWarningController : public NotificationObserver {
public:
explicit RepostFormWarningController(TabContents* tab_contents);
+ virtual ~RepostFormWarningController();
// Show the warning dialog.
void Show(ConstrainedWindowDelegate* window_delegate);
@@ -29,8 +29,6 @@
void Continue();
private:
- virtual ~RepostFormWarningController();
-
// NotificationObserver implementation.
// Watch for a new load or a closed tab and dismiss the dialog if they occur.
void Observe(NotificationType type,
« no previous file with comments | « chrome/browser/gtk/repost_form_warning_gtk.cc ('k') | chrome/browser/repost_form_warning_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698