OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "chrome/browser/ui/cocoa/browser_window_cocoa.h" | 5 #include "chrome/browser/ui/cocoa/browser_window_cocoa.h" |
6 | 6 |
7 #include "base/bind.h" | 7 #include "base/bind.h" |
8 #include "base/command_line.h" | 8 #include "base/command_line.h" |
9 #include "base/logging.h" | 9 #include "base/logging.h" |
10 #include "base/mac/mac_util.h" | 10 #include "base/mac/mac_util.h" |
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
163 // Callers assume that this doesn't immediately delete the Browser object. | 163 // Callers assume that this doesn't immediately delete the Browser object. |
164 // The controller implementing the window delegate methods called from | 164 // The controller implementing the window delegate methods called from |
165 // |-performClose:| must take precautions to ensure that. | 165 // |-performClose:| must take precautions to ensure that. |
166 void BrowserWindowCocoa::Close() { | 166 void BrowserWindowCocoa::Close() { |
167 // If there is an overlay window, we contain a tab being dragged between | 167 // If there is an overlay window, we contain a tab being dragged between |
168 // windows. Don't hide the window as it makes the UI extra confused. We can | 168 // windows. Don't hide the window as it makes the UI extra confused. We can |
169 // still close the window, as that will happen when the drag completes. | 169 // still close the window, as that will happen when the drag completes. |
170 if ([controller_ overlayWindow]) { | 170 if ([controller_ overlayWindow]) { |
171 [controller_ deferPerformClose]; | 171 [controller_ deferPerformClose]; |
172 } else { | 172 } else { |
173 [window() performClose:controller_]; | 173 // Using |-performClose:| can prevent the window from actually closing if |
| 174 // a JavaScript beforeunload handler opens an alert during shutdown, as |
| 175 // documented at <http://crbug.com/118424>. Re-implement |
| 176 // -[NSWindow performClose:] as closely as possible to how Apple documents |
| 177 // it. |
| 178 id<NSWindowDelegate> delegate = [window() delegate]; |
| 179 SEL window_should_close = @selector(windowShouldClose:); |
| 180 if ([delegate respondsToSelector:window_should_close]) { |
| 181 if ([delegate windowShouldClose:window()]) |
| 182 [window() close]; |
| 183 } else if ([window() respondsToSelector:window_should_close]) { |
| 184 if ([window() performSelector:window_should_close withObject:window()]) |
| 185 [window() close]; |
| 186 } else { |
| 187 [window() close]; |
| 188 } |
174 } | 189 } |
175 } | 190 } |
176 | 191 |
177 void BrowserWindowCocoa::Activate() { | 192 void BrowserWindowCocoa::Activate() { |
178 [controller_ activate]; | 193 [controller_ activate]; |
179 } | 194 } |
180 | 195 |
181 void BrowserWindowCocoa::Deactivate() { | 196 void BrowserWindowCocoa::Deactivate() { |
182 // TODO(jcivelli): http://crbug.com/51364 Implement me. | 197 // TODO(jcivelli): http://crbug.com/51364 Implement me. |
183 NOTIMPLEMENTED(); | 198 NOTIMPLEMENTED(); |
(...skipping 471 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
655 AvatarMenuBubbleController* menu = | 670 AvatarMenuBubbleController* menu = |
656 [[AvatarMenuBubbleController alloc] initWithBrowser:browser_ | 671 [[AvatarMenuBubbleController alloc] initWithBrowser:browser_ |
657 anchoredAt:point]; | 672 anchoredAt:point]; |
658 [[menu bubble] setAlignment:info_bubble::kAlignEdgeToAnchorEdge]; | 673 [[menu bubble] setAlignment:info_bubble::kAlignEdgeToAnchorEdge]; |
659 [menu showWindow:nil]; | 674 [menu showWindow:nil]; |
660 } | 675 } |
661 | 676 |
662 void BrowserWindowCocoa::ShowAvatarBubbleFromAvatarButton() { | 677 void BrowserWindowCocoa::ShowAvatarBubbleFromAvatarButton() { |
663 [[controller_ avatarButtonController] showAvatarBubble]; | 678 [[controller_ avatarButtonController] showAvatarBubble]; |
664 } | 679 } |
OLD | NEW |