]> git.saurik.com Git - wxWidgets.git/blobdiff - src/mac/app.cpp
CarbonEvents Added
[wxWidgets.git] / src / mac / app.cpp
index 51980c75643f82f21767340dffb3d5923a1f1a0e..02232921edf15f392828c3bf40d30eda11dfa25d 100644 (file)
@@ -36,6 +36,8 @@
 #include "wx/tooltip.h"
 #include "wx/textctrl.h"
 #include "wx/menu.h"
+#include "wx/docview.h"
+
 #if wxUSE_WX_RESOURCES
 #  include "wx/resource.h"
 #endif
@@ -73,14 +75,16 @@ extern wxList wxPendingDelete;
 extern wxList *wxWinMacWindowList;
 extern wxList *wxWinMacControlList;
 
+static bool s_inYield = FALSE;
+
 wxApp *wxTheApp = NULL;
 
 #if !USE_SHARED_LIBRARY
 IMPLEMENT_DYNAMIC_CLASS(wxApp, wxEvtHandler)
 BEGIN_EVENT_TABLE(wxApp, wxEvtHandler)
     EVT_IDLE(wxApp::OnIdle)
-        EVT_END_SESSION(wxApp::OnEndSession)
-        EVT_QUERY_END_SESSION(wxApp::OnQueryEndSession)
+    EVT_END_SESSION(wxApp::OnEndSession)
+    EVT_QUERY_END_SESSION(wxApp::OnQueryEndSession)
 END_EVENT_TABLE()
 #endif
 
@@ -105,51 +109,37 @@ long      wxApp::s_macPreferencesMenuItemId = 0 ;
 long      wxApp::s_macExitMenuItemId = wxID_EXIT ;
 wxString  wxApp::s_macHelpMenuTitleName = "&Help" ;
 
+//----------------------------------------------------------------------
+// Core Apple Event Support
+//----------------------------------------------------------------------
+
 pascal OSErr AEHandleODoc( const AppleEvent *event , AppleEvent *reply , long refcon ) ;
 pascal OSErr AEHandleOApp( const AppleEvent *event , AppleEvent *reply , long refcon ) ;
 pascal OSErr AEHandlePDoc( const AppleEvent *event , AppleEvent *reply , long refcon ) ;
 pascal OSErr AEHandleQuit( const AppleEvent *event , AppleEvent *reply , long refcon ) ;
 
-
 pascal OSErr AEHandleODoc( const AppleEvent *event , AppleEvent *reply , long WXUNUSED(refcon) )
 {
-    // GD: UNUSED wxApp* app = (wxApp*) refcon ;
     return wxTheApp->MacHandleAEODoc( (AppleEvent*) event , reply) ;
 }
 
 pascal OSErr AEHandleOApp( const AppleEvent *event , AppleEvent *reply , long WXUNUSED(refcon) )
 {
-    // GD: UNUSED wxApp* app = (wxApp*) refcon ;
     return wxTheApp->MacHandleAEOApp( (AppleEvent*) event , reply ) ;
 }
 
 pascal OSErr AEHandlePDoc( const AppleEvent *event , AppleEvent *reply , long WXUNUSED(refcon) )
 {
-    // GD: UNUSED wxApp* app = (wxApp*) refcon ;
     return wxTheApp->MacHandleAEPDoc( (AppleEvent*) event , reply ) ;
 }
 
 pascal OSErr AEHandleQuit( const AppleEvent *event , AppleEvent *reply , long WXUNUSED(refcon) )
 {
-    // GD: UNUSED wxApp* app = (wxApp*) refcon ;
     return wxTheApp->MacHandleAEQuit( (AppleEvent*) event , reply) ;
 }
 
-// new virtual public method in wxApp
-void wxApp::MacOpenFile(const wxString & WXUNUSED(fileName) )
-{
-}
-
-void wxApp::MacPrintFile(const wxString & WXUNUSED(fileName) )
-{
-}
+// AEODoc Calls MacOpenFile on each of the files passed
 
-void wxApp::MacNewFile()
-{
-}
-
-// new implementation, which parses the event and calls
-// MacOpenFile on each of the files it's passed
 short wxApp::MacHandleAEODoc(const WXEVENTREF event, WXEVENTREF WXUNUSED(reply))
 {
     AEDescList docList;
@@ -182,6 +172,8 @@ short wxApp::MacHandleAEODoc(const WXEVENTREF event, WXEVENTREF WXUNUSED(reply))
     return noErr;
 }
 
+// AEPDoc Calls MacPrintFile on each of the files passed
+
 short wxApp::MacHandleAEPDoc(const WXEVENTREF event , WXEVENTREF WXUNUSED(reply))
 {
     AEDescList docList;
@@ -214,12 +206,16 @@ short wxApp::MacHandleAEPDoc(const WXEVENTREF event , WXEVENTREF WXUNUSED(reply)
     return noErr;
 }
 
+// AEOApp calls MacNewFile
+
 short wxApp::MacHandleAEOApp(const WXEVENTREF WXUNUSED(event) , WXEVENTREF WXUNUSED(reply))
 {
     MacNewFile() ;
     return noErr ;
 }
 
+// AEQuit attempts to quite the application 
+
 short wxApp::MacHandleAEQuit(const WXEVENTREF WXUNUSED(event) , WXEVENTREF WXUNUSED(reply))
 {
     wxWindow* win = GetTopWindow() ;
@@ -234,6 +230,49 @@ short wxApp::MacHandleAEQuit(const WXEVENTREF WXUNUSED(event) , WXEVENTREF WXUNU
     return noErr ;
 }
 
+//----------------------------------------------------------------------
+// Support Routines linking the Mac...File Calls to the Document Manager
+//----------------------------------------------------------------------
+
+void wxApp::MacOpenFile(const wxString & fileName )
+{
+    wxDocManager* dm = wxDocManager::GetDocumentManager() ;
+    if ( dm )
+        dm->CreateDocument(fileName , wxDOC_SILENT ) ;
+}
+
+void wxApp::MacPrintFile(const wxString & fileName )
+{
+    wxDocManager* dm = wxDocManager::GetDocumentManager() ;
+    if ( dm )
+    {
+        wxDocument *doc = dm->CreateDocument(fileName , wxDOC_SILENT ) ;
+        if ( doc )
+        {
+            wxView* view = doc->GetFirstView() ;
+            if( view )
+            {
+                wxPrintout *printout = view->OnCreatePrintout();
+                if (printout)
+                {
+                    wxPrinter printer;
+                    printer.Print(view->GetFrame(), printout, TRUE);
+                    delete printout;
+                }
+            }
+            if (doc->Close())
+            {
+                doc->DeleteAllViews();
+                dm->RemoveDocument(doc) ;
+            }
+        }
+    }
+}
+
+void wxApp::MacNewFile()
+{
+}
+
 char StringMac[] =  "\x0d\x80\x81\x82\x83\x84\x85\x86\x87\x88\x89\x8a\x8b\x8c\x8d\x8e\x8f"
                     "\x90\x91\x92\x93\x94\x95\x96\x97\x98\x99\x9a\x9b\x9c\x9d\x9e\x9f"
                     "\xa0\xa1\xa2\xa3\xa4\xa5\xa6\xa7\xa8\xa9\xaa\xab\xac\xae\xaf"
@@ -446,6 +485,113 @@ void wxMacStringToPascal( const char * from , StringPtr to , bool pc2macEncoding
     }
 }
 
+//----------------------------------------------------------------------
+// Carbon Event Handler
+//----------------------------------------------------------------------
+
+#if TARGET_CARBON
+
+       static const EventTypeSpec eventList[] = 
+       {
+           { kEventClassCommand, kEventProcessCommand } ,
+           { kEventClassCommand, kEventCommandUpdateStatus } ,
+           { kEventClassApplication , kEventAppActivated } ,
+           { kEventClassApplication , kEventAppDeactivated } ,
+       } ;
+
+static pascal OSStatus CommandEventHandler( EventHandlerCallRef handler , EventRef event , void *data )
+{
+    OSStatus result = eventNotHandledErr ;
+
+       HICommand command ;
+       
+       GetEventParameter( event, kEventParamDirectObject, typeHICommand, NULL,
+               sizeof( HICommand ), NULL, &command );
+
+       MenuCommand id = command.commandID ;
+       if ( id == kHICommandPreferences )
+               id = wxApp::s_macPreferencesMenuItemId ;
+               
+    wxMenuBar* mbar = wxMenuBar::MacGetInstalledMenuBar() ;
+    wxMenu* menu = NULL ;
+    wxMenuItem* item = NULL ;
+
+    if ( mbar )
+        item = mbar->FindItem( id , &menu ) ;
+
+    if ( item == NULL || menu == NULL || mbar == NULL )
+       return result ;
+       
+       switch( GetEventKind( event ) )
+       {
+               case kEventProcessCommand :
+              {
+                       if (item->IsCheckable())
+                       {
+                           item->Check( !item->IsChecked() ) ;
+                       }
+                               
+                               menu->SendEvent( id , item->IsCheckable() ? item->IsChecked() : -1 ) ;
+                               result = noErr ;
+                       }
+                       break ;
+               case kEventCommandUpdateStatus:
+                       // eventually trigger an updateui round
+                               result = noErr ;
+                       break ;
+               default :
+                       break ;
+       }   
+    
+    return result ;
+}
+
+static pascal OSStatus ApplicationEventHandler( EventHandlerCallRef handler , EventRef event , void *data )
+{
+    OSStatus result = eventNotHandledErr ;
+       switch ( GetEventKind( event ) )
+       {
+               case kEventAppActivated :
+                       {
+                               wxTheApp->MacResume( true ) ;
+                               result = noErr ;
+                       }
+                       break ;
+               case kEventAppDeactivated :
+                       {
+                               wxTheApp->MacSuspend( true ) ;
+                               result = noErr ;
+                       }
+                       break ;
+               default :
+                       break ;
+       }
+    return result ;
+}
+
+pascal OSStatus wxAppEventHandler( EventHandlerCallRef handler , EventRef event , void *data )
+{
+    OSStatus result = eventNotHandledErr ;
+    switch( GetEventClass( event ) )
+    {
+       case kEventClassCommand :
+               result = CommandEventHandler( handler , event , data ) ;
+               break ;
+       case kEventClassApplication :
+               result = ApplicationEventHandler( handler , event , data ) ;
+               break ;
+       
+       default :
+               break ;
+    }
+
+    return result ;
+}
+
+DEFINE_ONE_SHOT_HANDLER_GETTER( wxAppEventHandler )
+
+#endif
+
 #if defined(WXMAKINGDLL) && !defined(__DARWIN__)
 // we know it's there ;-)
 WXIMPORT char std::__throws_bad_alloc ;
@@ -467,34 +613,6 @@ bool wxApp::Initialize()
     wxStAppResource::OpenSharedLibraryResource(NULL);
 #endif
 
-#if defined(UNIVERSAL_INTERFACES_VERSION) && (UNIVERSAL_INTERFACES_VERSION >= 0x0340)
-    AEInstallEventHandler( kCoreEventClass , kAEOpenDocuments ,
-                           NewAEEventHandlerUPP(AEHandleODoc) ,
-                           (long) wxTheApp , FALSE ) ;
-    AEInstallEventHandler( kCoreEventClass , kAEOpenApplication ,
-                           NewAEEventHandlerUPP(AEHandleOApp) ,
-                           (long) wxTheApp , FALSE ) ;
-    AEInstallEventHandler( kCoreEventClass , kAEPrintDocuments ,
-                           NewAEEventHandlerUPP(AEHandlePDoc) ,
-                           (long) wxTheApp , FALSE ) ;
-    AEInstallEventHandler( kCoreEventClass , kAEQuitApplication ,
-                           NewAEEventHandlerUPP(AEHandleQuit) ,
-                           (long) wxTheApp , FALSE ) ;
-#else
-    AEInstallEventHandler( kCoreEventClass , kAEOpenDocuments ,
-                           NewAEEventHandlerProc(AEHandleODoc) ,
-                           (long) wxTheApp , FALSE ) ;
-    AEInstallEventHandler( kCoreEventClass , kAEOpenApplication ,
-                           NewAEEventHandlerProc(AEHandleOApp) ,
-                           (long) wxTheApp , FALSE ) ;
-    AEInstallEventHandler( kCoreEventClass , kAEPrintDocuments ,
-                           NewAEEventHandlerProc(AEHandlePDoc) ,
-                           (long) wxTheApp , FALSE ) ;
-    AEInstallEventHandler( kCoreEventClass , kAEQuitApplication ,
-                           NewAEEventHandlerProc(AEHandleQuit) ,
-                           (long) wxTheApp , FALSE ) ;
-#endif
-
 #ifndef __DARWIN__
     // test the minimal configuration necessary
 
@@ -557,7 +675,11 @@ bool wxApp::Initialize()
 
 #ifndef __DARWIN__
     // now avoid exceptions thrown for new (bad_alloc)
-    std::__throws_bad_alloc = FALSE ;
+       // FIXME CS for some changes outside wxMac does not compile anymore
+#if 0
+    std::__throws_bad_alloc = 0 ;
+#endif
+
 #endif
 
     s_macCursorRgn = ::NewRgn() ;
@@ -609,12 +731,53 @@ bool wxApp::Initialize()
 
   wxMacCreateNotifierTable() ;
 
-
   UMAShowArrowCursor() ;
 
   return TRUE;
 }
 
+bool wxApp::OnInitGui()
+{
+       if( !wxAppBase::OnInitGui() )
+               return false ;
+               
+#if TARGET_CARBON              
+       InstallApplicationEventHandler(
+               GetwxAppEventHandlerUPP(), 
+               GetEventTypeCount(eventList), eventList, wxTheApp, &((EventHandlerRef)wxTheApp->m_macEventHandler));    
+#endif
+               
+#if defined(UNIVERSAL_INTERFACES_VERSION) && (UNIVERSAL_INTERFACES_VERSION >= 0x0340)
+    AEInstallEventHandler( kCoreEventClass , kAEOpenDocuments ,
+                           NewAEEventHandlerUPP(AEHandleODoc) ,
+                           0 , FALSE ) ;
+    AEInstallEventHandler( kCoreEventClass , kAEOpenApplication ,
+                           NewAEEventHandlerUPP(AEHandleOApp) ,
+                           0 , FALSE ) ;
+    AEInstallEventHandler( kCoreEventClass , kAEPrintDocuments ,
+                           NewAEEventHandlerUPP(AEHandlePDoc) ,
+                           0 , FALSE ) ;
+    AEInstallEventHandler( kCoreEventClass , kAEQuitApplication ,
+                           NewAEEventHandlerUPP(AEHandleQuit) ,
+                           0 , FALSE ) ;
+#else
+    AEInstallEventHandler( kCoreEventClass , kAEOpenDocuments ,
+                           NewAEEventHandlerProc(AEHandleODoc) ,
+                           0 , FALSE ) ;
+    AEInstallEventHandler( kCoreEventClass , kAEOpenApplication ,
+                           NewAEEventHandlerProc(AEHandleOApp) ,
+                           0 , FALSE ) ;
+    AEInstallEventHandler( kCoreEventClass , kAEPrintDocuments ,
+                           NewAEEventHandlerProc(AEHandlePDoc) ,
+                           0 , FALSE ) ;
+    AEInstallEventHandler( kCoreEventClass , kAEQuitApplication ,
+                           NewAEEventHandlerProc(AEHandleQuit) ,
+                           0 , FALSE ) ;
+#endif
+
+       return TRUE ;
+}
+
 void wxApp::CleanUp()
 {
     wxToolTip::RemoveToolTips() ;
@@ -1071,6 +1234,7 @@ bool wxMacConvertEventToRecord( EventRef event , EventRecord *rec)
     return converted ;
 }
 
+/*
 pascal OSStatus wxMacApplicationEventHandler( EventHandlerCallRef handler , EventRef event , void *data )
 {
     OSStatus result = eventNotHandledErr ;
@@ -1099,30 +1263,9 @@ pascal OSStatus wxMacApplicationEventHandler( EventHandlerCallRef handler , Even
     }
     return result ;
 }
-
+*/
 #endif
 
-bool wxApp::OnInit()
-{
-    if ( ! wxAppBase::OnInit() )
-        return FALSE ;
-
-#if 0 // TARGET_CARBON
-       static const EventTypeSpec eventList[] =
-       {
-           { kEventClassKeyboard, kEventRawKeyDown } ,
-           { kEventClassKeyboard, kEventRawKeyRepeat } ,
-           { kEventClassKeyboard, kEventRawKeyUp } ,
-           { kEventClassKeyboard, kEventRawKeyModifiersChanged } ,
-
-           { kEventClassTextInput , kEventTextInputUnicodeForKeyEvent } ,
-       } ;
-
-       InstallApplicationEventHandler(NewEventHandlerUPP(wxMacApplicationEventHandler)
-           , WXSIZEOF(eventList), eventList, this, NULL);
-#endif
-    return TRUE ;
-}
 // Static member initialization
 wxAppInitializerFunction wxAppBase::m_appInitFn = (wxAppInitializerFunction) NULL;
 
@@ -1152,14 +1295,14 @@ bool wxApp::Initialized()
 
 int wxApp::MainLoop()
 {
-  m_keepGoing = TRUE;
+       m_keepGoing = TRUE;
 
-  while (m_keepGoing)
-  {
-        MacDoOneEvent() ;
-  }
+       while (m_keepGoing)
+       {
+           MacDoOneEvent() ;
+       }
 
-  return 0;
+       return 0;
 }
 
 // Returns TRUE if more time is needed.
@@ -1322,8 +1465,6 @@ void wxCYield()
 
 bool wxApp::Yield(bool onlyIfNeeded)
 {
-    static bool s_inYield = FALSE;
-
     if (s_inYield)
     {
         if ( !onlyIfNeeded )
@@ -1363,24 +1504,26 @@ bool wxApp::Yield(bool onlyIfNeeded)
 
 void wxApp::MacSuspend( bool convertClipboard )
 {
+#if !TARGET_CARBON
     // we have to deactive the top level windows manually
 
     wxNode* node = wxTopLevelWindows.First();
     while (node)
     {
         wxTopLevelWindow* win = (wxTopLevelWindow*) node->Data();
-        win->MacActivate( MacGetCurrentEvent() , false ) ;
+        win->MacActivate( ((EventRecord*) MacGetCurrentEvent())->when , false ) ;
 
         node = node->Next();
     }
 
+    ::HideFloatingWindows() ;
+#endif
     s_lastMouseDown = 0 ;
+
     if( convertClipboard )
     {
         MacConvertPrivateToPublicScrap() ;
     }
-
-    ::HideFloatingWindows() ;
 }
 
 extern wxList wxModalDialogs;
@@ -1393,8 +1536,8 @@ void wxApp::MacResume( bool convertClipboard )
         MacConvertPublicToPrivateScrap() ;
     }
 
+#if !TARGET_CARBON
     ::ShowFloatingWindows() ;
-
     // raise modal dialogs in case a non modal window was selected to activate the app
 
     wxNode* node = wxModalDialogs.First();
@@ -1405,6 +1548,7 @@ void wxApp::MacResume( bool convertClipboard )
 
         node = node->Next();
     }
+#endif
 }
 
 void wxApp::MacConvertPrivateToPublicScrap()
@@ -1518,6 +1662,7 @@ void wxApp::MacHandleOneEvent( WXEVENTREF evr )
             else
                 s_lastMouseDown = 1;
             break;
+#if !TARGET_CARBON
         case mouseUp:
             if ( s_lastMouseDown == 2 )
             {
@@ -1536,6 +1681,7 @@ void wxApp::MacHandleOneEvent( WXEVENTREF evr )
         case updateEvt:
             MacHandleUpdateEvent( ev ) ;
             break;
+#endif
         case keyDown:
         case autoKey:
             MacHandleKeyDownEvent( ev ) ;
@@ -1543,12 +1689,14 @@ void wxApp::MacHandleOneEvent( WXEVENTREF evr )
         case keyUp:
             MacHandleKeyUpEvent( ev ) ;
             break;
+#if !TARGET_CARBON
         case diskEvt:
             MacHandleDiskEvent( ev ) ;
             break;
         case osEvt:
             MacHandleOSEvent( ev ) ;
             break;
+#endif
         case kHighLevelEvent:
             MacHandleHighLevelEvent( ev ) ;
             break;
@@ -1560,8 +1708,12 @@ void wxApp::MacHandleOneEvent( WXEVENTREF evr )
 
 void wxApp::MacHandleHighLevelEvent( WXEVENTREF evr )
 {
+    // we must avoid reentrancy problems when processing high level events eg printing
+    bool former = s_inYield ;
+    s_inYield = TRUE ;
     EventRecord* ev = (EventRecord*) evr ;
     ::AEProcessAppleEvent( ev ) ;
+    s_inYield = former ;
 }
 
 bool s_macIsInModalLoop = false ;
@@ -1595,7 +1747,9 @@ void wxApp::MacHandleMouseDownEvent( WXEVENTREF evr )
             else
             {
                 UInt32 menuresult = MenuSelect(ev->where) ;
-                MacHandleMenuSelect( HiWord( menuresult ) , LoWord( menuresult ) );
+#if !TARGET_CARBON
+               MacHandleMenuSelect( HiWord( menuresult ) , LoWord( menuresult ) );
+#endif
                 s_lastMouseDown = 0;
             }
             break ;
@@ -1604,7 +1758,6 @@ void wxApp::MacHandleMouseDownEvent( WXEVENTREF evr )
             SystemClick( ev , window ) ;
             s_lastMouseDown = 0;
             break ;
-#endif
         case inDrag :
             if ( window != frontWindow && s_macIsInModalLoop && !(ev->modifiers & cmdKey ) )
             {
@@ -1722,12 +1875,14 @@ void wxApp::MacHandleMouseDownEvent( WXEVENTREF evr )
                         win->MacMouseDown( ev , windowPart ) ;
                 }
             break ;
-
+#endif
         default:
             break;
     }
 }
 
+#if !TARGET_CARBON
+
 void wxApp::MacHandleMouseUpEvent( WXEVENTREF evr )
 {
     EventRecord* ev = (EventRecord*) evr ;
@@ -1760,6 +1915,8 @@ void wxApp::MacHandleMouseUpEvent( WXEVENTREF evr )
     }
 }
 
+#endif 
+
 long wxMacTranslateKey(unsigned char key, unsigned char code) ;
 long wxMacTranslateKey(unsigned char key, unsigned char code)
 {
@@ -1874,6 +2031,7 @@ void wxApp::MacHandleKeyDownEvent( WXEVENTREF evr )
     EventRecord* ev = (EventRecord*) evr ;
     wxToolTip::RemoveToolTips() ;
 
+#if !TARGET_CARBON
     UInt32 menuresult = UMAMenuEvent(ev) ;
     if ( HiWord( menuresult ) )
     {
@@ -1881,6 +2039,7 @@ void wxApp::MacHandleKeyDownEvent( WXEVENTREF evr )
             MacHandleMenuSelect( HiWord( menuresult ) , LoWord( menuresult ) ) ;
     }
     else
+#endif
     {
          wxWindow* focus = wxWindow::FindFocus() ;
  
@@ -2087,6 +2246,7 @@ bool wxApp::MacSendKeyUpEvent( wxWindow* focus , long keymessage , long modifier
 
     return handled ;
 }
+#if !TARGET_CARBON
 void wxApp::MacHandleActivateEvent( WXEVENTREF evr )
 {
     EventRecord* ev = (EventRecord*) evr ;
@@ -2103,7 +2263,7 @@ void wxApp::MacHandleActivateEvent( WXEVENTREF evr )
         }
         wxTopLevelWindowMac* win = wxFindWinFromMacWindow( window ) ;
         if ( win )
-            win->MacActivate( ev , activate ) ;
+            win->MacActivate( ev->when , activate ) ;
     }
 }
 
@@ -2132,14 +2292,12 @@ void wxApp::MacHandleDiskEvent( WXEVENTREF evr )
     EventRecord* ev = (EventRecord*) evr ;
     if ( HiWord( ev->message ) != noErr )
   {
- #if !TARGET_CARBON
         OSErr err ;
         Point point ;
          SetPt( &point , 100 , 100 ) ;
 
           err = DIBadMount( point , ev->message ) ;
         wxASSERT( err == noErr ) ;
-#endif
     }
 }
 
@@ -2151,11 +2309,8 @@ void wxApp::MacHandleOSEvent( WXEVENTREF evr )
         case suspendResumeMessage :
             {
                 bool isResuming = ev->message & resumeFlag ;
-#if !TARGET_CARBON
                 bool convertClipboard = ev->message & convertClipboardFlag ;
-#else
-                bool convertClipboard = false;
-#endif
+
                 bool doesActivate = UMAGetProcessModeDoesActivateOnFGSwitch() ;
                 if ( isResuming )
                 {
@@ -2176,31 +2331,18 @@ void wxApp::MacHandleOSEvent( WXEVENTREF evr )
                     {
                         wxTopLevelWindowMac* win = wxFindWinFromMacWindow( oldFrontWindow ) ;
                         if ( win )
-                            win->MacActivate( ev , false ) ;
+                            win->MacActivate( ev->when , false ) ;
                     }
                     if ( newFrontWindow )
                     {
                         wxTopLevelWindowMac* win = wxFindWinFromMacWindow( newFrontWindow ) ;
                         if ( win )
-                            win->MacActivate( ev , true ) ;
+                            win->MacActivate( ev->when , true ) ;
                     }
                 }
                 else
                 {
                     MacSuspend( convertClipboard ) ;
-
-                    // in case this suspending did close an active window, another one might
-                    // have surfaced -> lets deactivate that one
-
-/* TODO : find out what to do on systems < 10 , perhaps FrontNonFloatingWindow
-                    WindowRef newActiveWindow = ::ActiveNonFloatingWindow() ;
-                    if ( newActiveWindow )
-                    {
-                        wxWindow* win = wxFindWinFromMacWindow( newActiveWindow ) ;
-                        if ( win )
-                            win->MacActivate( ev , false ) ;
-                    }
-*/
                 }
             }
             break ;
@@ -2303,7 +2445,28 @@ void wxApp::MacHandleOSEvent( WXEVENTREF evr )
 
     }
 }
+#endif
 
+void wxApp::MacHandleMenuCommand( wxUint32 id )
+{
+        wxMenuBar* mbar = wxMenuBar::MacGetInstalledMenuBar() ;
+        wxMenu* menu = NULL ;
+        wxMenuItem* item = NULL ;
+        if ( mbar )
+        {
+            item = mbar->FindItem( id , &menu ) ;
+        }
+        wxCHECK_RET( item != NULL && menu != NULL && mbar != NULL, wxT("error in menu item callback") );
+            
+        if (item->IsCheckable())
+        {
+            item->Check( !item->IsChecked() ) ;
+        }
+               
+               menu->SendEvent( id , item->IsCheckable() ? item->IsChecked() : -1 ) ;
+}
+
+#if !TARGET_CARBON
 void wxApp::MacHandleMenuSelect( int macMenuId , int macMenuItemNum )
 {
     if (macMenuId == 0)
@@ -2325,29 +2488,8 @@ void wxApp::MacHandleMenuSelect( int macMenuId , int macMenuItemNum )
     {
         MenuCommand id ;
         GetMenuItemCommandID( GetMenuHandle(macMenuId) , macMenuItemNum , &id ) ;
-        wxMenuBar* mbar = wxMenuBar::MacGetInstalledMenuBar() ;
-        wxMenu* menu = NULL ;
-        wxMenuItem* item = NULL ;
-        if ( mbar )
-        {
-            item = mbar->FindItem( id , &menu ) ;
-        }
-        wxCHECK_RET( item != NULL && menu != NULL && mbar != NULL, wxT("error in menu item callback") );
-            
-        if (item->IsCheckable())
-        {
-            item->Check( !item->IsChecked() ) ;
-        }
-               
-               menu->SendEvent( id , item->IsCheckable() ? item->IsChecked() : -1 ) ;
-               /*
-        wxWindow* frontwindow = wxFindWinFromMacWindow( ::FrontWindow() )  ;
-        wxCommandEvent event(wxEVT_COMMAND_MENU_SELECTED, id );
-        event.m_timeStamp =  ((EventRecord*) MacGetCurrentEvent())->when ;
-        event.SetEventObject(menu);
-        event.SetInt(item->IsCheckable() ? item->IsChecked() : -1);
-        frontwindow->GetEventHandler()->ProcessEvent(event); 
-        */
+               MacHandleMenuCommand( id ) ;
     }
     HiliteMenu(0);
 }
+#endif
\ No newline at end of file