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

Unified Diff: chrome/browser/fullscreen_mac.mm

Issue 11697004: Switch to Cocoa APIs for fullscreen. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: BOOL Created 8 years 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
« no previous file with comments | « base/mac/mac_util.mm ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/fullscreen_mac.mm
diff --git a/chrome/browser/fullscreen_mac.mm b/chrome/browser/fullscreen_mac.mm
index 303f6f240779a48d8d8e7e004bebe0a158df79af..f4e73c06ab6a14877146fcb0715e103f341f9bb9 100644
--- a/chrome/browser/fullscreen_mac.mm
+++ b/chrome/browser/fullscreen_mac.mm
@@ -4,79 +4,80 @@
#import "chrome/browser/fullscreen.h"
-#import <Carbon/Carbon.h>
#import <Cocoa/Cocoa.h>
#import "base/logging.h"
+#import "third_party/GTM/Foundation/GTMNSObject+KeyValueObserving.h"
+
+// Replicate specific 10.7 SDK declarations for building with prior SDKs.
+#if !defined(MAC_OS_X_VERSION_10_7) || \
+ MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_7
+
+enum {
+ NSApplicationPresentationFullScreen = 1 << 10
+};
+
+#endif // MAC_OS_X_VERSION_10_7
+
+namespace {
+
+BOOL AreOptionsFullScreen(NSApplicationPresentationOptions options) {
+ // If both dock and menu bar are hidden, that is the equivalent of the Carbon
+ // SystemUIMode (or Info.plist's LSUIPresentationMode) kUIModeAllHidden.
+ if (((options & NSApplicationPresentationHideDock) ||
+ (options & NSApplicationPresentationAutoHideDock)) &&
+ ((options & NSApplicationPresentationHideMenuBar) ||
+ (options & NSApplicationPresentationAutoHideMenuBar))) {
+ return YES;
+ }
+
+ if (options & NSApplicationPresentationFullScreen)
+ return YES;
+
+ return NO;
+}
+
+} // namespace
@interface FullScreenMonitor : NSObject {
@private
BOOL fullScreen_;
- EventHandlerRef eventHandler_;
}
@property (nonatomic, getter=isFullScreen) BOOL fullScreen;
@end
-static OSStatus handleAppEvent(EventHandlerCallRef myHandler,
- EventRef event,
- void* userData) {
- DCHECK(userData);
-
- FullScreenMonitor* fullScreenMonitor =
- reinterpret_cast<FullScreenMonitor*>(userData);
-
- UInt32 mode = 0;
- OSStatus status = GetEventParameter(event,
- kEventParamSystemUIMode,
- typeUInt32,
- NULL,
- sizeof(UInt32),
- NULL,
- &mode);
- if (status != noErr)
- return status;
- BOOL isFullScreenMode = mode == kUIModeAllHidden;
- [fullScreenMonitor setFullScreen:isFullScreenMode];
- return noErr;
-}
-
@implementation FullScreenMonitor
@synthesize fullScreen = fullScreen_;
- (id)init {
if ((self = [super init])) {
- // Check if the user is in presentation mode initially.
- SystemUIMode currentMode;
- GetSystemUIMode(&currentMode, NULL);
- fullScreen_ = currentMode == kUIModeAllHidden;
-
- // Register a Carbon event to receive the notification about the login
- // session's UI mode change.
- EventTypeSpec events[] =
- {{ kEventClassApplication, kEventAppSystemUIModeChanged }};
- OSStatus status = InstallApplicationEventHandler(
- NewEventHandlerUPP(handleAppEvent),
- GetEventTypeCount(events),
- events,
- self,
- &eventHandler_);
- if (status) {
- [self release];
- self = nil;
- }
+ [NSApp gtm_addObserver:self
+ forKeyPath:@"currentSystemPresentationOptions"
+ selector:@selector(observeNotification:)
+ userInfo:nil
+ options:NSKeyValueObservingOptionNew |
+ NSKeyValueObservingOptionInitial];
}
return self;
}
- (void)dealloc {
- if (eventHandler_)
- RemoveEventHandler(eventHandler_);
+ [NSApp gtm_removeObserver:self
+ forKeyPath:@"currentSystemPresentationOptions"
+ selector:@selector(observeNotification:)];
[super dealloc];
}
+- (void)observeNotification:(GTMKeyValueChangeNotification*)notification {
+ NSDictionary* change = [notification change];
+ NSApplicationPresentationOptions options =
+ [[change objectForKey:NSKeyValueChangeNewKey] integerValue];
+ [self setFullScreen:AreOptionsFullScreen(options)];
+}
+
@end
static FullScreenMonitor* g_fullScreenMonitor = nil;
@@ -92,7 +93,7 @@ void StopFullScreenMonitor() {
}
bool IsFullScreenMode() {
- // Check if the main display has been captured (game in particular).
+ // Check if the main display has been captured (by games in particular).
if (CGDisplayIsCaptured(CGMainDisplayID()))
return true;
« no previous file with comments | « base/mac/mac_util.mm ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698