Merge pull request #1454 from LibertyGlobal/offline_renderer_mem_leak_fix

Fix 'offline renderer' thread cleanup

Author: magomez

Source/WebCore/Modules/webaudio/OfflineAudioDestinationNode.cpp

@@ -86,10 +86,6 @@ void OfflineAudioDestinationNode::uninitialize()
         return;
 
     if (m_startedRendering) {
-        if (m_renderThread) {
-            m_renderThread->waitForCompletion();
-            m_renderThread = nullptr;
-        }
         if (auto* workletProxy = context().audioWorklet().proxy()) {
             BinarySemaphore semaphore;
             workletProxy->postTaskForModeToWorkletGlobalScope([&semaphore](ScriptExecutionContext&) mutable {
@@ -99,6 +95,11 @@ void OfflineAudioDestinationNode::uninitialize()
         }
     }
 
+    if (m_renderThread) {
+        m_renderThread->waitForCompletion();
+        m_renderThread = nullptr;
+    }
+
     AudioNode::uninitialize();
 }