Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6260)

Unified Diff: chrome/browser/ui/cocoa/browser_window_controller_private.h

Issue 655393002: mac: Fix a bug where a fullscreened window on Mavericks would be too short. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/browser_window_controller_private.h
diff --git a/chrome/browser/ui/cocoa/browser_window_controller_private.h b/chrome/browser/ui/cocoa/browser_window_controller_private.h
index d001f1dbb4481c48dfe7c32430dad235bf555002..f82a714bebd28b080a32d0cb924d0415f60b8e70 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller_private.h
+++ b/chrome/browser/ui/cocoa/browser_window_controller_private.h
@@ -157,6 +157,15 @@
// implementation for more details.
- (void)updateSubviewZOrderHack;
+// There is a bug in Mavericks for applications linked against OSX 10.8 and
+// earlier. It requires Screens Have Separate Spaces to be enabled, and for the
Robert Sesek 2014/10/17 19:13:16 "It" is a vague reference here -- the bug requires
erikchen 2014/10/17 21:02:32 Yes. I replaced "It" with "The bug".
+// window to be on a secondary screen. When AppKit Fullscreen is invoked on the
+// window, its final frame is 22pt too short. This method detects when the
+// relevant conditions have been met so that a hack can be applied to fix the
+// size of the window.
+// http://crbug.com/396980
+- (BOOL)shouldUseMavericksAppKitFullscreenHack;
+
@end // @interface BrowserWindowController(Private)
#endif // CHROME_BROWSER_UI_COCOA_BROWSER_WINDOW_CONTROLLER_PRIVATE_H_

Powered by Google App Engine
This is Rietveld 408576698