diff options
| author | pinkerton@chromium.org <pinkerton@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-09 16:25:41 +0000 |
|---|---|---|
| committer | pinkerton@chromium.org <pinkerton@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-09 16:25:41 +0000 |
| commit | 4d0e9e6625cb72c6091087cba9049ffac5cbf0d3 (patch) | |
| tree | 62489948cd0715cd6eedb5ff00d39a9794da0b8f /chrome/browser/cocoa/browser_window_cocoa.mm | |
| parent | 810d6408e8553435b97151805a9ed48ffe8d610e (diff) | |
| download | chromium_src-4d0e9e6625cb72c6091087cba9049ffac5cbf0d3.zip chromium_src-4d0e9e6625cb72c6091087cba9049ffac5cbf0d3.tar.gz chromium_src-4d0e9e6625cb72c6091087cba9049ffac5cbf0d3.tar.bz2 | |
Make sure we call orderOut: before calling performClose:. This ensures the window is hidden immediately so we don't see any teardown that may ensure.
BUG=23959
TEST=closing windows with lots of tabs, quitting with lots of tabs. dragging while windows close.
Review URL: http://codereview.chromium.org/264022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28542 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/cocoa/browser_window_cocoa.mm')
| -rw-r--r-- | chrome/browser/cocoa/browser_window_cocoa.mm | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/chrome/browser/cocoa/browser_window_cocoa.mm b/chrome/browser/cocoa/browser_window_cocoa.mm index a9d256a..6ab8128 100644 --- a/chrome/browser/cocoa/browser_window_cocoa.mm +++ b/chrome/browser/cocoa/browser_window_cocoa.mm @@ -60,8 +60,13 @@ void BrowserWindowCocoa::Close() { // still close the window, as that will happen when the drag completes. if ([controller_ overlayWindow]) [controller_ deferPerformClose]; - else + else { + // Make sure we hide the window immediately. Even though performClose: + // calls orderOut: eventually, it leaves the window on-screen long enough + // that we start to see tabs shutting down. http://crbug.com/23959 + [window_ orderOut:controller_]; [window_ performClose:controller_]; + } } void BrowserWindowCocoa::Activate() { |