1 files changed, 98 insertions, 12 deletions

diff --git a/linden/indra/newview/llappviewermacosx.cpp b/linden/indra/newview/llappviewermacosx.cpp
index e3fd425..0e5062d 100644
--- a/linden/indra/newview/llappviewermacosx.cpp
+++ b/linden/indra/newview/llappviewermacosx.cpp
@@ -4,7 +4,7 @@
  *
  * $LicenseInfo:firstyear=2007&license=viewergpl$
  * 
- * Copyright (c) 2007-2008, Linden Research, Inc.
+ * Copyright (c) 2007-2009, Linden Research, Inc.
  * 
  * Second Life Viewer Source Code
  * The source code in this file ("Source Code") is provided by Linden Lab
@@ -49,12 +49,24 @@
 #include <Carbon/Carbon.h>
 #include "lldir.h"
 #include <signal.h>
+
 namespace 
 {
         // The command line args stored.
         // They are not used immediately by the app.
         int gArgC;
         char** gArgV;
+        
+        bool sCrashReporterIsRunning = false;
+        
+        OSErr AEQuitHandler(const AppleEvent *messagein, AppleEvent *reply, long refIn)
+        {
+                OSErr result = noErr;
+                
+                LLAppViewer::instance()->userQuit();
+                
+                return(result);
+        }
 }
 
 int main( int argc, char **argv ) 
@@ -256,8 +268,37 @@ void LLAppViewerMacOSX::handleSyncCrashTrace()
         // do nothing
 }
 
+static OSStatus CarbonEventHandler(EventHandlerCallRef inHandlerCallRef, 
+                                                                   EventRef inEvent, 
+                                                                   void* inUserData)
+{
+    ProcessSerialNumber psn;
+        
+    GetEventParameter(inEvent, 
+                                          kEventParamProcessID, 
+                                          typeProcessSerialNumber, 
+                                          NULL, 
+                                          sizeof(psn), 
+                                          NULL, 
+                                          &psn);
+        
+    if( GetEventKind(inEvent) == kEventAppTerminated ) 
+        {
+                Boolean matching_psn = FALSE;   
+                OSErr os_result = SameProcess(&psn, (ProcessSerialNumber*)inUserData, &matching_psn);
+                if(os_result >= 0 && matching_psn)
+                {
+                        sCrashReporterIsRunning = false;
+                }
+    }
+    return noErr;
+}
+
 void LLAppViewerMacOSX::handleCrashReporting(bool reportFreeze)
 {
+        // This used to use fork&exec, but is switched to LSOpenApplication to 
+        // Make sure the crash reporter launches in front of the SL window.
+        
         std::string command_str;
         //command_str = "open Second Life.app/Contents/Resources/mac-crash-logger.app";
         command_str = "mac-crash-logger.app/Contents/MacOS/mac-crash-logger";
@@ -279,15 +320,69 @@ void LLAppViewerMacOSX::handleCrashReporting(bool reportFreeze)
                 {
                         // Make sure freeze reporting launches the crash logger synchronously, lest 
                         // Log files get changed by SL while the logger is running.
+                
+                        // *NOTE:Mani A better way - make a copy of the data that the crash reporter will send
+                        // and let SL go about its business. This way makes the mac work like windows and linux
+                        // and is the smallest patch for the issue. 
+                        sCrashReporterIsRunning = true;
+                        ProcessSerialNumber o_psn;
+
+                        static EventHandlerRef sCarbonEventsRef = NULL;
+                        static const EventTypeSpec kEvents[] = 
+                        {
+                                { kEventClassApplication, kEventAppTerminated }
+                        };
+                        
+                        // Install the handler to detect crash logger termination
+                        InstallEventHandler(GetApplicationEventTarget(), 
+                                                                (EventHandlerUPP) CarbonEventHandler,
+                                                                GetEventTypeCount(kEvents),
+                                                                kEvents,
+                                                                &o_psn,
+                                                                &sCarbonEventsRef
+                                                                );
+                        
+                        // Remove, temporarily the quit handler - which has *crash* behavior before 
+                        // the mainloop gets running!
+                        AERemoveEventHandler(kCoreEventClass, 
+                                                                 kAEQuitApplication, 
+                                                                 NewAEEventHandlerUPP(AEQuitHandler),
+                                                                 false);
+
+                        // Launch the crash reporter.
+                        os_result = LSOpenApplication(&appParams, &o_psn);
+                        
+                        if(os_result >= 0)
+                        {       
+                                EventRecord evt;
+                                while(sCrashReporterIsRunning)
+                                {
+                                        while(WaitNextEvent(osMask, &evt, 0, NULL))
+                                        {
+                                                // null op!?!
+                                        }
+                                }
+                        }       
+
+                        // Re-install the apps quit handler.
+                        AEInstallEventHandler(kCoreEventClass, 
+                                                                  kAEQuitApplication, 
+                                                                  NewAEEventHandlerUPP(AEQuitHandler),
+                                                                  0, 
+                                                                  false);
+                        
+                        // Remove the crash reporter quit handler.
+                        RemoveEventHandler(sCarbonEventsRef);
                 }
                 else
                 {
                         appParams.flags |= kLSLaunchAsync;
                         clear_signals();
+
+                        ProcessSerialNumber o_psn;
+                        os_result = LSOpenApplication(&appParams, &o_psn);
                 }
                 
-                ProcessSerialNumber o_psn;
-                os_result = LSOpenApplication(&appParams, &o_psn);
         }
 
         if(!reportFreeze)
@@ -352,15 +447,6 @@ OSErr AEGURLHandler(const AppleEvent *messagein, AppleEvent *reply, long refIn)
         return(result);
 }
 
-OSErr AEQuitHandler(const AppleEvent *messagein, AppleEvent *reply, long refIn)
-{
-        OSErr result = noErr;
-        
-        LLAppViewer::instance()->userQuit();
-        
-        return(result);
-}
-
 OSStatus simpleDialogHandler(EventHandlerCallRef handler, EventRef event, void *userdata)
 {
         OSStatus result = eventNotHandledErr;