LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-01-08 12:19:37 +03:00
Code Issues Packages Projects Releases Wiki Activity

SoundPlayer: Don't enqueue samples depending on the GUI loop

Browse Source 
Previously, SoundPlayer would read and enqueue samples in the GUI loop
(through a Timer). Apart from general problems with doing audio on the
GUI thread, this is particularly bad as the audio would lag or drop out
when the GUI lags (e.g. window resizes and moves, changing the
visualizer). As Piano does, now SoundPlayer enqueues more audio once the
audio server signals that a buffer has finished playing. The GUI-
dependent decoding is still kept as a "backup" and to start the entire
cycle, but it's not solely depended on. A queue of buffer IDs is used to
keep track of playing buffers and how many there are. The buffer
overhead, i.e. how many buffers "too many" currently exist, is currently
set to its absolute minimum of 2.
This commit is contained in:
kleines Filmröllchen 2021-12-17 18:47:31 +01:00 committed by Andreas Kling
parent b48badc3b6
commit c748c0726a
Notes: sideshowbarker 2024-07-17 21:46:30 +09:00 
Author: https://github.com/kleinesfilmroellchen
Commit: https://github.com/SerenityOS/serenity/commit/c748c0726a8
Pull-request: https://github.com/SerenityOS/serenity/pull/11290
2 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
Userland/Applications/SoundPlayer/PlaybackManager.cpp
View File

@ -12,8 +12,16 @@ PlaybackManager::PlaybackManager(NonnullRefPtr<Audio::ClientConnection> connecti
m_timer = Core::Timer::construct(PlaybackManager::update_rate_ms, [&]() {
if (!m_loader)
return;
next_buffer();
// Make sure that we have some buffers queued up at all times: an audio dropout is the last thing we want.
if (m_enqueued_buffers.size() < always_enqueued_buffer_count)
next_buffer();
});
m_connection->on_finish_playing_buffer = [this](auto finished_buffer) {
auto last_buffer_in_queue = m_enqueued_buffers.dequeue();
// A fail here would mean that the server skipped one of our buffers, which is BAD.
VERIFY(last_buffer_in_queue == finished_buffer);
next_buffer();
};
m_timer->stop();
m_device_sample_rate = connection->get_sample_rate();
}
@ -117,7 +125,7 @@ void PlaybackManager::next_buffer()
return;
}
if (audio_server_remaining_samples < m_device_samples_per_buffer) {
if (audio_server_remaining_samples < m_device_samples_per_buffer * always_enqueued_buffer_count) {
auto maybe_buffer = m_loader->get_more_samples(m_source_buffer_size_bytes);
if (!maybe_buffer.is_error()) {
m_current_buffer = maybe_buffer.release_value();
@ -126,6 +134,7 @@ void PlaybackManager::next_buffer()
// FIXME: Handle OOM better.
m_current_buffer = MUST(Audio::resample_buffer(m_resampler.value(), *m_current_buffer));
m_connection->enqueue(*m_current_buffer);
m_enqueued_buffers.enqueue(m_current_buffer->id());
}
}
}

6
Userland/Applications/SoundPlayer/PlaybackManager.h
View File

@ -6,6 +6,7 @@
#pragma once
#include <AK/Queue.h>
#include <AK/Vector.h>
#include <LibAudio/Buffer.h>
#include <LibAudio/ClientConnection.h>
@ -39,6 +40,9 @@ public:
Function<void()> on_finished_playing;
private:
// Number of buffers we want to always keep enqueued.
static constexpr size_t always_enqueued_buffer_count = 2;
void next_buffer();
void set_paused(bool);
@ -52,12 +56,12 @@ private:
RefPtr<Audio::Loader> m_loader { nullptr };
NonnullRefPtr<Audio::ClientConnection> m_connection;
RefPtr<Audio::Buffer> m_current_buffer;
Queue<i32, always_enqueued_buffer_count + 1> m_enqueued_buffers;
Optional<Audio::ResampleHelper<double>> m_resampler;
RefPtr<Core::Timer> m_timer;
// Controls the GUI update rate. A smaller value makes the visualizations nicer.
static constexpr u32 update_rate_ms = 50;
// Number of milliseconds of audio data contained in each audio buffer
static constexpr u32 buffer_size_ms = 100;
};