Reland: Remove DialogDelegate::OnClosed() which is redundant with

chrome/browser/ui/views/passwords/auto_signin_first_run_dialog_view.cc

Index: chrome/browser/ui/views/passwords/auto_signin_first_run_dialog_view.cc
diff --git a/chrome/browser/ui/views/passwords/auto_signin_first_run_dialog_view.cc b/chrome/browser/ui/views/passwords/auto_signin_first_run_dialog_view.cc
index 6af5aadf2c9296aba932ef796ce7dda87f64e566..a25667ceab1763331cec31fa460021226fd5c66b 100644
--- a/chrome/browser/ui/views/passwords/auto_signin_first_run_dialog_view.cc
+++ b/chrome/browser/ui/views/passwords/auto_signin_first_run_dialog_view.cc
@@ -105,19 +105,17 @@ views::View* AutoSigninFirstRunDialogView::GetInitiallyFocusedView() {
   return ok_button_;
 }
 
+void AutoSigninFirstRunDialogView::WindowClosing() {
+  if (controller_)
+    controller_->OnCloseDialog();
+}
+
 int AutoSigninFirstRunDialogView::GetDialogButtons() const {
   // None because ESC is equivalent to Cancel. It shouldn't turn off the auto
   // signin.
   return ui::DIALOG_BUTTON_NONE;
 }
 
-void AutoSigninFirstRunDialogView::OnClosed() {
-  if (controller_)
-    controller_->OnCloseDialog();
-  // This method is called twice. crbug.com/583330
-  controller_ = nullptr;
-}
-
 gfx::Size AutoSigninFirstRunDialogView::GetPreferredSize() const {
   return gfx::Size(kDesiredWidth, GetHeightForWidth(kDesiredWidth));
 }