Speculative Revert 195108 "Changes to closing contents with beforeunload/unl..."

trunk/src/chrome/browser/ui/cocoa/browser_window_controller.mm

Index: trunk/src/chrome/browser/ui/cocoa/browser_window_controller.mm
===================================================================
--- trunk/src/chrome/browser/ui/cocoa/browser_window_controller.mm	(revision 195133)
+++ trunk/src/chrome/browser/ui/cocoa/browser_window_controller.mm	(working copy)
@@ -586,16 +586,12 @@
   [self saveWindowPositionIfNeeded];
 
   if (!browser_->tab_strip_model()->empty()) {
+    // Tab strip isn't empty.  Hide the frame (so it appears to have closed
+    // immediately) and close all the tabs, allowing the renderers to shut
+    // down. When the tab strip is empty we'll be called back again.
     [[self window] orderOut:self];
     browser_->OnWindowClosing();
-    browser_->tab_strip_model()->CloseAllTabs();
     return NO;
-  } else if (!browser_->HasCompletedUnloadProcessing()) {
-    // The browser needs to finish running unload handlers.
-    // Hide the window (so it appears to have closed immediately), and
-    // the browser will call us back again when it is ready to close.
-    [[self window] orderOut:self];
-    return NO;
   }
 
   // the tab strip is empty, it's ok to close the window