diff --git a/Userland/Libraries/LibWeb/Loader/Resource.cpp b/Userland/Libraries/LibWeb/Loader/Resource.cpp index dbef11c1cb..123909380c 100644 --- a/Userland/Libraries/LibWeb/Loader/Resource.cpp +++ b/Userland/Libraries/LibWeb/Loader/Resource.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -141,13 +142,23 @@ void ResourceClient::set_resource(Resource* resource) m_resource->register_client({}, *this); - // Make sure that reused resources also have their load callback fired. - if (resource->is_loaded()) - resource_did_load(); + // For resources that are already loaded, we fire their load/fail callbacks via the event loop. + // This ensures that these callbacks always happen in a consistent way, instead of being invoked + // synchronously in some cases, and asynchronously in others. + if (resource->is_loaded() || resource->is_failed()) { + Core::deferred_invoke([this, strong_resource = NonnullRefPtr { *m_resource }] { + if (m_resource != strong_resource.ptr()) + return; - // Make sure that reused resources also have their fail callback fired. - if (resource->is_failed()) - resource_did_fail(); + // Make sure that reused resources also have their load callback fired. + if (m_resource->is_loaded()) + resource_did_load(); + + // Make sure that reused resources also have their fail callback fired. + if (m_resource->is_failed()) + resource_did_fail(); + }); + } } }