Stop using nested message loop for alert() and other JS dialogs

chrome/browser/ui/browser.cc

 // Copyright 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 "chrome/browser/ui/browser.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <string>
@@ skipping 670 matching lines @@
   DCHECK(IsFastTabUnloadEnabled());
   return fast_unload_controller_->HasCompletedUnloadProcessing();
 }
 
 bool Browser::IsAttemptingToCloseBrowser() const {
   if (IsFastTabUnloadEnabled())
     return fast_unload_controller_->is_attempting_to_close_browser();
   return unload_controller_->is_attempting_to_close_browser();
 }
 
+bool Browser::ShouldRunUnloadListenerBeforeClosing(
+    content::WebContents* web_contents) {
+  if (IsFastTabUnloadEnabled())
+    return fast_unload_controller_->ShouldRunUnloadEventsHelper(web_contents);
+  return unload_controller_->ShouldRunUnloadEventsHelper(web_contents);
+}
+
+bool Browser::RunUnloadListenerBeforeClosing(
+    content::WebContents* web_contents) {
+  if (IsFastTabUnloadEnabled())
+    return fast_unload_controller_->RunUnloadEventsHelper(web_contents);
+  return unload_controller_->RunUnloadEventsHelper(web_contents);
+}
+
 void Browser::OnWindowClosing() {
   if (!ShouldCloseWindow())
     return;
 
   // Application should shutdown on last window close if the user is explicitly
   // trying to quit, or if there is nothing keeping the browser alive (such as
   // AppController on the Mac, or BackgroundContentsService for background
   // pages).
   bool should_quit_if_last_browser =
       browser_shutdown::IsTryingToQuit() ||
@@ skipping 2044 matching lines @@
   if (contents && !allow_js_access) {
     contents->web_contents()->GetController().LoadURL(
         target_url,
         content::Referrer(),
         ui::PAGE_TRANSITION_LINK,
         std::string());  // No extra headers.
   }
 
   return contents != NULL;
 }