Disable buttons in the Crostini Recovery

When the user has clicked the Restart button, there is a delay while
Crostini is shutting down before we close the dialog. We disable the
buttons in this state as feedback that something is happening and the
user shouldn't proceed yet.

Bug: 1060830
Change-Id: If0e654c6f63e182efc3e8d178cd2237db9a09807
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2099346Reviewed-by: Nic Hollingum <hollingum@google.com>
Reviewed-by: David Munro <davidmunro@google.com>
Commit-Queue: Nicholas Verne <nverne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749516}

chrome/browser/ui/views/crostini/crostini_recovery_view.cc

@@ -68,6 +68,12 @@ bool CrostiniRecoveryView::ShouldShowCloseButton() const {
   return false;
 }
 
+bool CrostiniRecoveryView::IsDialogButtonEnabled(
+    ui::DialogButton button) const {
+  // Buttons are disabled after Accept or Cancel have been clicked.
+  return closed_reason_ == views::Widget::ClosedReason::kUnspecified;
+}
+
 gfx::Size CrostiniRecoveryView::CalculatePreferredSize() const {
   const int dialog_width = ChromeLayoutProvider::Get()->GetDistanceMetric(
                                DISTANCE_STANDALONE_BUBBLE_PREFERRED_WIDTH) -
@@ -90,6 +96,7 @@ bool CrostiniRecoveryView::Accept() {
             }
           },
           weak_ptr_factory_.GetWeakPtr()));
+  DialogModelChanged();
   return false;
 }
 
@@ -116,6 +123,7 @@ bool CrostiniRecoveryView::Cancel() {
   }
   app_id_ = crostini::GetTerminalId();
   ScheduleAppLaunch();
+  DialogModelChanged();
   return false;
 }
 

chrome/browser/ui/views/crostini/crostini_recovery_view.h

@@ -29,6 +29,7 @@ class CrostiniRecoveryView : public views::BubbleDialogDelegateView {
   gfx::Size CalculatePreferredSize() const override;
   bool Accept() override;
   bool Cancel() override;
+  bool IsDialogButtonEnabled(ui::DialogButton button) const override;
 
   static CrostiniRecoveryView* GetActiveViewForTesting();