Fix 'offline renderer' thread cleanup

Offline renderer thread in most of the cases doesn't clean up its resources.
Thread should be joined on every exit, not only when rendering process is in progress.
This problem was detected during webaudio testing which revealed memory leakage.

Author: marcin-mielczarczyk-red

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();
 }