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

Unified Diff: trunk/src/chrome/browser/ui/views/tab_modal_confirm_dialog_views.cc

Issue 21372006: Revert 212329 "Reland "Close web contents modal dialogs on conte..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 5 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: trunk/src/chrome/browser/ui/views/tab_modal_confirm_dialog_views.cc
===================================================================
--- trunk/src/chrome/browser/ui/views/tab_modal_confirm_dialog_views.cc (revision 214772)
+++ trunk/src/chrome/browser/ui/views/tab_modal_confirm_dialog_views.cc (working copy)
@@ -39,8 +39,7 @@
TabModalConfirmDialogViews::TabModalConfirmDialogViews(
TabModalConfirmDialogDelegate* delegate,
content::WebContents* web_contents)
- : web_contents_(web_contents),
- delegate_(delegate),
+ : delegate_(delegate),
dialog_(NULL),
browser_context_(web_contents->GetBrowserContext()) {
views::MessageBoxView::InitParams init_params(delegate->GetMessage());
@@ -60,7 +59,7 @@
web_contents_modal_dialog_manager->delegate()->
GetWebContentsModalDialogHost());
web_contents_modal_dialog_manager->ShowDialog(dialog_->GetNativeView());
- delegate_->set_operations_delegate(this);
+ delegate_->set_close_delegate(this);
}
TabModalConfirmDialogViews::~TabModalConfirmDialogViews() {
@@ -78,13 +77,6 @@
dialog_->Close();
}
-void TabModalConfirmDialogViews::SetPreventCloseOnLoadStart(bool prevent) {
- WebContentsModalDialogManager* web_contents_modal_dialog_manager =
- WebContentsModalDialogManager::FromWebContents(web_contents_);
- web_contents_modal_dialog_manager->SetPreventCloseOnLoadStart(
- dialog_->GetNativeView(), prevent);
-}
-
//////////////////////////////////////////////////////////////////////////////
// TabModalConfirmDialogViews, views::LinkListener implementation:

Powered by Google App Engine
This is Rietveld 408576698