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

Unified Diff: ui/views/window/dialog_client_view.h

Issue 1686433002: Remove DialogDelegate::OnClosed() which is redundant with (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review Created 4 years, 10 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: ui/views/window/dialog_client_view.h
diff --git a/ui/views/window/dialog_client_view.h b/ui/views/window/dialog_client_view.h
index 1108904f5d519fd81f1e01b398fe39f990138a55..2fe4744f1aaefa4754ebb86594527e126b52c8d8 100644
--- a/ui/views/window/dialog_client_view.h
+++ b/ui/views/window/dialog_client_view.h
@@ -106,12 +106,6 @@ class VIEWS_EXPORT DialogClientView : public ClientView,
// The footnote view shown below the buttons; may be NULL.
View* footnote_view_;
- // True if we've notified the delegate the window is closing and the delegate
- // allosed the close. In some situations it's possible to get two closes (see
- // http://crbug.com/71940). This is used to avoid notifying the delegate
- // twice, which can have bad consequences.
- bool notified_delegate_;
sky 2016/02/11 22:56:56 Why is this no longer necessary?
-
DISALLOW_COPY_AND_ASSIGN(DialogClientView);
};
« no previous file with comments | « components/app_modal/views/javascript_app_modal_dialog_views.cc ('k') | ui/views/window/dialog_client_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698