Mediate "LLAppViewer" "userQuit" et al. via "mainloop" WorkQueue.

Empirically, this works better than engaging the respective LLAppViewer
methods directly.

1 files changed, 10 insertions, 3 deletions

diff --git a/indra/newview/llappviewerlistener.cpp b/indra/newview/llappviewerlistener.cpp
index d02b1b4a79..4690c91b61 100644
--- a/indra/newview/llappviewerlistener.cpp
+++ b/indra/newview/llappviewerlistener.cpp
@@ -35,6 +35,7 @@
 // external library headers
 // other Linden headers
 #include "llappviewer.h"
+#include "workqueue.h"
 
 LLAppViewerListener::LLAppViewerListener(const LLAppViewerGetter& getter):
     LLEventAPI("LLAppViewer",
@@ -56,17 +57,23 @@ LLAppViewerListener::LLAppViewerListener(const LLAppViewerGetter& getter):
 void LLAppViewerListener::userQuit(const LLSD& event)
 {
     LL_INFOS() << "Listener requested user quit" << LL_ENDL;
-    mAppViewerGetter()->userQuit();
+    // Trying to engage this from (e.g.) a Lua-hosting C++ coroutine runs
+    // afoul of an assert in the logging machinery that LLMutex must be locked
+    // only from the main coroutine.
+    LL::WorkQueue::getInstance("mainloop")->post(
+        [appviewer=mAppViewerGetter()]{ appviewer->userQuit(); });
 }
 
 void LLAppViewerListener::requestQuit(const LLSD& event)
 {
     LL_INFOS() << "Listener requested quit" << LL_ENDL;
-    mAppViewerGetter()->requestQuit();
+    LL::WorkQueue::getInstance("mainloop")->post(
+        [appviewer=mAppViewerGetter()]{ appviewer->requestQuit(); });
 }
 
 void LLAppViewerListener::forceQuit(const LLSD& event)
 {
     LL_INFOS() << "Listener requested force quit" << LL_ENDL;
-    mAppViewerGetter()->forceQuit();
+    LL::WorkQueue::getInstance("mainloop")->post(
+        [appviewer=mAppViewerGetter()]{ appviewer->forceQuit(); });
 }