LibGfx: Use recursive mutex where needed

This commit makes the skia backend context use a recursive mutex as the
normal one is now not recursive. Same for the pulse audio context.

Author: R-Goc

Libraries/LibCore/EventLoopImplementationUnix.cpp

@@ -267,7 +267,7 @@ struct ThreadData {
         s_thread_data.remove(s_thread_id);
     }
 
-    Sync::Mutex mutex;
+    Sync::RecursiveMutex mutex;
 
     // Each thread has its own timers, notifiers and a wake pipe.
     TimeoutSet timeouts;

Libraries/LibGfx/SkiaBackendContext.h

@@ -52,7 +52,7 @@ class SkiaBackendContext : public AtomicRefCounted<SkiaBackendContext> {
     void unlock() { m_mutex.unlock(); }
 
 private:
-    Sync::Mutex m_mutex;
+    Sync::RecursiveMutex m_mutex;
 };
 
 }

Libraries/LibMedia/Audio/PulseAudioWrappers.cpp

@@ -11,7 +11,7 @@
 namespace Audio {
 
 static PulseAudioContext* s_pulse_audio_context;
-static Sync::Mutex s_pulse_audio_context_mutex;
+static Sync::RecursiveMutex s_pulse_audio_context_mutex;
 
 ErrorOr<NonnullRefPtr<PulseAudioContext>> PulseAudioContext::the()
 {

Libraries/LibMedia/Providers/AudioDataProvider.h

@@ -84,8 +84,8 @@ class MEDIA_API AudioDataProvider : public AtomicRefCounted<AudioDataProvider> {
 
         Core::EventLoop& m_main_thread_event_loop;
 
-        mutable Sync::Mutex m_mutex;
-        mutable Sync::ConditionVariable m_wait_condition { m_mutex };
+        mutable Sync::RecursiveMutex m_mutex;
+        mutable Sync::RecursiveConditionVariable m_wait_condition { m_mutex };
         RequestedState m_requested_state { RequestedState::None };
 
         NonnullRefPtr<MutexedDemuxer> m_demuxer;

Libraries/LibMedia/Providers/VideoDataProvider.h

@@ -87,8 +87,8 @@ class MEDIA_API VideoDataProvider final : public AtomicRefCounted<VideoDataProvi
 
         Core::EventLoop& m_main_thread_event_loop;
 
-        mutable Sync::Mutex m_mutex;
-        mutable Sync::ConditionVariable m_wait_condition { m_mutex };
+        mutable Sync::RecursiveMutex m_mutex;
+        mutable Sync::RecursiveConditionVariable m_wait_condition { m_mutex };
         RequestedState m_requested_state { RequestedState::None };
 
         NonnullRefPtr<MutexedDemuxer> m_demuxer;