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

Unified Diff: chrome/browser/ui/cocoa/browser_window_controller.mm

Issue 10947019: Constrained Window Cocoa: Disable fullscreen (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 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.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller.mm b/chrome/browser/ui/cocoa/browser_window_controller.mm
index 8da5a93c9549389abf03aa7a9a1b60f739cdfda3..362e0de2251111a42db69aea650a414df9d3db98 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller.mm
@@ -1009,14 +1009,6 @@ enum {
}
}
-- (BOOL)supportsFullscreen {
- // TODO(avi, thakis): GTMWindowSheetController has no api to move
- // tabsheets between windows. Until then, we have to prevent having to
- // move a tabsheet between windows, e.g. no fullscreen toggling
- NSArray* a = [[tabStripController_ sheetController] viewsWithAttachedSheets];
- return [a count] == 0;
-}
-
// Called to validate menu and toolbar items when this window is key. All the
// items we care about have been set with the |-commandDispatch:| or
// |-commandDispatchUsingKeyModifiers:| actions and a target of FirstResponder
@@ -1046,7 +1038,6 @@ enum {
enable &= !![[static_cast<NSMenuItem*>(item) keyEquivalent] length];
break;
case IDC_FULLSCREEN: {
- enable &= [self supportsFullscreen];
if ([static_cast<NSObject*>(item) isKindOfClass:[NSMenuItem class]]) {
NSString* menuTitle = l10n_util::GetNSString(
[self isFullscreen] && ![self inPresentationMode] ?
@@ -1060,7 +1051,6 @@ enum {
break;
}
case IDC_PRESENTATION_MODE: {
- enable &= [self supportsFullscreen];
if ([static_cast<NSObject*>(item) isKindOfClass:[NSMenuItem class]]) {
NSString* menuTitle = l10n_util::GetNSString(
[self inPresentationMode] ? IDS_EXIT_PRESENTATION_MAC :
@@ -1985,7 +1975,7 @@ willAnimateFromState:(bookmarks::VisualState)oldState
if (fullscreen == [self isFullscreen])
return;
- if (![self supportsFullscreen])
+ if (!chrome::IsCommandEnabled(browser_.get(), IDC_FULLSCREEN))
return;
if (base::mac::IsOSLionOrLater()) {

Powered by Google App Engine
This is Rietveld 408576698