chromeos: Run beforeunload handlers when closing panels.

chrome/browser/chromeos/frame/panel_controller.cc

 // Copyright (c) 2010 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 "chrome/browser/chromeos/frame/panel_controller.h"
 
 #include <vector>
 
 #include "app/resource_bundle.h"
 #include "base/logging.h"
@@ skipping 288 matching lines @@
     title_window_ = NULL;
     title_ = NULL;
     title_content_->OnClose();
     title_content_ = NULL;
   }
 }
 
 void PanelController::OnCloseButtonPressed() {
   DCHECK(title_content_);
   if (title_window_) {
-    if (delegate_)
+    if (delegate_) {
+      if (!delegate_->CanClosePanel())

[oshima, 2011/01/13 21:18:12]

This may not be the scope of this CL, but wouldn't it be better to give visual
feedback that a user can't close the panel?

[Daniel Erat, 2011/01/13 21:26:34]

In the case of a beforeunload handler, a modal dialog is opened.

+        return;
       delegate_->ClosePanel();
+    }
     Close();
   }
 }
 
 PanelController::TitleContentView::TitleContentView(
     PanelController* panel_controller)
         : panel_controller_(panel_controller) {
   VLOG(1) << "panel: c " << this;
   InitializeResources();
   close_button_ = new views::ImageButton(this);
@@ skipping 77 matching lines @@
     views::Button* sender, const views::Event& event) {
   if (panel_controller_ && sender == close_button_)
     panel_controller_->OnCloseButtonPressed();
 }
 
 PanelController::TitleContentView::~TitleContentView() {
   VLOG(1) << "panel: delete " << this;
 }
 
 }  // namespace chromeos