DialogClientView: Handle nested close requests gracefully.

ui/views/window/dialog_client_view.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_VIEWS_WINDOW_DIALOG_CLIENT_VIEW_H_
 #define UI_VIEWS_WINDOW_DIALOG_CLIENT_VIEW_H_
 
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "ui/base/ui_base_types.h"
@@ skipping 86 matching lines @@
   // a View should not appear, it will be null.
   std::array<View*, kNumButtons> GetButtonRowViews();
 
   // Installs and configures the LayoutManager for |button_row_container_|.
   void SetupLayout();
 
   // Creates or deletes any buttons that are required. Updates data members.
   // After calling this, no button row Views will be in the view hierarchy.
   void SetupViews();
 
+  // Updates |delegate_allowed_close_| using the provided |method|. If |close|
+  // is true, invokes Widget::Close() the first time the delegate allows it.
+  void CloseWithDelegateMethod(bool (DialogDelegate::*method)(), bool close);
+
   // How much to inset the button row.
   gfx::Insets button_row_insets_;
 
   // The minimum size of this dialog, regardless of the size of its content
   // view.
   gfx::Size minimum_size_;
 
   // The dialog buttons.
   LabelButton* ok_button_ = nullptr;
   LabelButton* cancel_button_ = nullptr;
 
   // The extra view shown in the row of buttons; may be NULL.
   View* extra_view_ = nullptr;
 
   // Container view for the button row.
   ButtonRowContainer* button_row_container_ = nullptr;
 
   // True if we've notified the delegate the window is closing and the delegate
   // allowed 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 delegate_allowed_close_ = false;
 
+  // True while asking the delegate whether a close should occur. The delegate
+  // may perform actions that trigger a nested close query. DialogClientView
+  // ignores these nested requests, and only acts on the first on the stack.
+  bool asking_delegate_whether_to_close_ = false;
+
   // Used to prevent unnecessary or potentially harmful changes during
   // SetupLayout(). Everything will be manually updated afterwards.
   bool adding_or_removing_views_ = false;
 
   DISALLOW_COPY_AND_ASSIGN(DialogClientView);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_WINDOW_DIALOG_CLIENT_VIEW_H_