Remove DialogDelegate::OnClosed() which is redundant with

components/app_modal/views/javascript_app_modal_dialog_views.cc

 // 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.
 
 #include "components/app_modal/views/javascript_app_modal_dialog_views.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "components/app_modal/javascript_app_modal_dialog.h"
 #include "components/constrained_window/constrained_window_views.h"
 #include "grit/components_strings.h"
@@ skipping 93 matching lines @@
   parent_->OnCancel(message_box_view_->IsCheckBoxSelected());
   return true;
 }
 
 bool JavaScriptAppModalDialogViews::Accept() {
   parent_->OnAccept(message_box_view_->GetInputText(),
                     message_box_view_->IsCheckBoxSelected());
   return true;
 }
 
-void JavaScriptAppModalDialogViews::OnClosed() {
-  parent_->OnClose();
-}
-
 views::Widget* JavaScriptAppModalDialogViews::GetWidget() {

[msw, 2016/02/11 18:08:58]

nit: reorder these GetWidget impls too, please.

[Evan Stade, 2016/02/11 20:02:11]

Done.

   return message_box_view_->GetWidget();
 }
 
 const views::Widget* JavaScriptAppModalDialogViews::GetWidget() const {
   return message_box_view_->GetWidget();
 }
 
 base::string16 JavaScriptAppModalDialogViews::GetDialogButtonLabel(
     ui::DialogButton button) const {
   if (parent_->is_before_unload_dialog()) {
@@ skipping 22 matching lines @@
 views::View* JavaScriptAppModalDialogViews::GetContentsView() {
   return message_box_view_;
 }
 
 views::View* JavaScriptAppModalDialogViews::GetInitiallyFocusedView() {
   if (message_box_view_->text_box())
     return message_box_view_->text_box();
   return views::DialogDelegate::GetInitiallyFocusedView();
 }
 
+void JavaScriptAppModalDialogViews::WindowClosing() {
+  parent_->OnClose();
+}
+
 }  // namespace app_modal