Merge pull request #478 from adobe/jeff/yosemite-traffic-lights

Enable Yosemite Native Traffic Lights on OSX 10.10
adobe · Oct 21, 2014 · a291bda · a291bda
2 parents e9cd327 + f7e83ad
commit a291bda
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 15 deletions.
diff --git a/appshell/cefclient_mac.mm b/appshell/cefclient_mac.mm
@@ -221,16 +221,35 @@ -(void)windowTitleDidChange:(NSString*)title {
 #endif
 }
 
+-(BOOL)isRunningOnYosemite {
+    NSDictionary* dict = [NSDictionary dictionaryWithContentsOfFile:@"/System/Library/CoreServices/SystemVersion.plist"];
+    NSString* version =  [dict objectForKey:@"ProductVersion"];
+    return [version hasPrefix:@"10.10"];
+}
+
 - (BOOL)isFullScreenSupported {
-    SInt32 version;
-    Gestalt(gestaltSystemVersion, &version);
-    return (version >= 0x1070);
+    // Return False on Yosemite so we
+    //  don't draw our own full screen button
+    //  and handle full screen mode
+    if (![self isRunningOnYosemite]) {
+        SInt32 version;
+        Gestalt(gestaltSystemVersion, &version);
+        return (version >= 0x1070);
+    }
+    return false;
 }
 
 -(BOOL)needsFullScreenActivateHack {
-    SInt32 version;
-    Gestalt(gestaltSystemVersion, &version);
-    return (version >= 0x1090);
+    if (![self isRunningOnYosemite]) {
+        SInt32 version;
+        Gestalt(gestaltSystemVersion, &version);
+        return (version >= 0x1090);
+    }
+    return false;
+}
+
+-(BOOL)useSystemTrafficLights {
+    return [self isRunningOnYosemite];
 }
 
 -(void)windowDidResize:(NSNotification *)notification
@@ -299,7 +318,7 @@ -(void)initUI:(NSWindow*)mainWindow {
     NSButton *windowButton = nil;
 
 #ifdef CUSTOM_TRAFFIC_LIGHTS
-    if (!trafficLightsView) {
+    if (![self useSystemTrafficLights] && !trafficLightsView) {
         windowButton = [mainWindow standardWindowButton:NSWindowCloseButton];
         [windowButton setHidden:YES];
         windowButton = [mainWindow standardWindowButton:NSWindowMiniaturizeButton];

diff --git a/appshell/client_handler_mac.mm b/appshell/client_handler_mac.mm
@@ -165,17 +165,35 @@ - (void)setIsReallyClosing {
   isReallyClosing = true;
 }
 
-- (BOOL)isFullScreenSupported {
-  SInt32 version;
-  Gestalt(gestaltSystemVersion, &version);
-  return (version >= 0x1070);
+-(BOOL)isRunningOnYosemite {
+    NSDictionary* dict = [NSDictionary dictionaryWithContentsOfFile:@"/System/Library/CoreServices/SystemVersion.plist"];
+    NSString* version =  [dict objectForKey:@"ProductVersion"];
+    return [version hasPrefix:@"10.10"];
 }
 
+- (BOOL)isFullScreenSupported {
+    // Return False on Yosemite so we
+    //  don't draw our own full screen button
+    //  and handle full screen mode
+    if (![self isRunningOnYosemite]) {
+        SInt32 version;
+        Gestalt(gestaltSystemVersion, &version);
+        return (version >= 0x1070);
+    }
+    return false;
+}
 
 -(BOOL)needsFullScreenActivateHack {
-    SInt32 version;
-    Gestalt(gestaltSystemVersion, &version);
-    return (version >= 0x1090);
+    if (![self isRunningOnYosemite]) {
+        SInt32 version;
+        Gestalt(gestaltSystemVersion, &version);
+        return (version >= 0x1090);
+    }
+    return false;
+}
+
+-(BOOL)useSystemTrafficLights {
+    return [self isRunningOnYosemite];
 }
 
 -(void)setFullScreenButtonView:(NSView *)view {
@@ -186,6 +204,8 @@ -(void)setTrafficLightsView:(NSView *)view {
   trafficLightsView = view;
 }
 
+
+
 -(void)windowTitleDidChange:(NSString*)title {
 #ifdef DARK_UI
   if (customTitlebar) {
@@ -237,7 +257,7 @@ - (void)initUI {
     NSButton* windowButton = nil;
 
 #ifdef CUSTOM_TRAFFIC_LIGHTS
-    if (!trafficLightsView) {
+    if (![self useSystemTrafficLights] && !trafficLightsView) {
         windowButton = [theWin standardWindowButton:NSWindowCloseButton];
         [windowButton setHidden:YES];
         windowButton = [theWin standardWindowButton:NSWindowMiniaturizeButton];