Commit 80cada9e authored by Marvin Scholz's avatar Marvin Scholz Committed by Felix Paul Kühne

osx_notifications: Use active state notifications for compatibility with OS X <10.10

This restores the code that used notifications to determine the app state.
Additionally the inital state is not active, as we get a notification if the
app switches to active after opening.
Signed-off-by: default avatarFelix Paul Kühne <fkuehne@videolan.org>
parent 16742c47
...@@ -86,6 +86,7 @@ ...@@ -86,6 +86,7 @@
NSString *notificationType; NSString *notificationType;
NSMutableDictionary *registrationDictionary; NSMutableDictionary *registrationDictionary;
id lastNotification; id lastNotification;
bool isInForeground;
intf_thread_t *interfaceThread; intf_thread_t *interfaceThread;
} }
...@@ -283,6 +284,21 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var, ...@@ -283,6 +284,21 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var,
if( !( self = [super init] ) ) if( !( self = [super init] ) )
return nil; return nil;
@autoreleasepool {
// Subscribe to notifications to determine if VLC is in foreground or not
[[NSNotificationCenter defaultCenter] addObserver:self
selector:@selector(applicationActiveChange:)
name:NSApplicationDidBecomeActiveNotification
object:nil];
[[NSNotificationCenter defaultCenter] addObserver:self
selector:@selector(applicationActiveChange:)
name:NSApplicationDidResignActiveNotification
object:nil];
}
// Start in background
isInForeground = NO;
applicationName = nil; applicationName = nil;
notificationType = nil; notificationType = nil;
registrationDictionary = nil; registrationDictionary = nil;
...@@ -301,6 +317,7 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var, ...@@ -301,6 +317,7 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var,
removeDeliveredNotification:(NSUserNotification *)lastNotification]; removeDeliveredNotification:(NSUserNotification *)lastNotification];
[lastNotification release]; [lastNotification release];
} }
[[NSNotificationCenter defaultCenter] removeObserver:self];
} }
#endif #endif
...@@ -340,7 +357,7 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var, ...@@ -340,7 +357,7 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var,
{ {
@autoreleasepool { @autoreleasepool {
// Do not notify if in foreground // Do not notify if in foreground
if ([NSApplication sharedApplication].active) if (isInForeground)
return; return;
// Init Cover // Init Cover
...@@ -436,6 +453,13 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var, ...@@ -436,6 +453,13 @@ static int ItemChange( vlc_object_t *p_this, const char *psz_var,
return applicationName; return applicationName;
} }
- (void)applicationActiveChange:(NSNotification *)n {
if (n.name == NSApplicationDidBecomeActiveNotification)
isInForeground = YES;
else if (n.name == NSApplicationDidResignActiveNotification)
isInForeground = NO;
}
#if __MAC_OS_X_VERSION_MAX_ALLOWED >= 1080 #if __MAC_OS_X_VERSION_MAX_ALLOWED >= 1080
- (void)userNotificationCenter:(NSUserNotificationCenter *)center - (void)userNotificationCenter:(NSUserNotificationCenter *)center
didActivateNotification:(NSUserNotification *)notification didActivateNotification:(NSUserNotification *)notification
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment