From d947253c51c68d7a59e9128432e053d699487033 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Sat, 29 May 2021 01:27:06 +0100 Subject: [PATCH] LibWeb: Also call page_did_start_loading() for FrameLoader::Type::Reload Surprisingly this is not used by the browser's page reload functionality but only JS's location.reload() - that's probably why this hasn't been noticed yet. Make sure we notify the page client about the load start in that case as well. :^) --- Userland/Libraries/LibWeb/Loader/FrameLoader.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Libraries/LibWeb/Loader/FrameLoader.cpp b/Userland/Libraries/LibWeb/Loader/FrameLoader.cpp index b33878c742..f1ff25c13e 100644 --- a/Userland/Libraries/LibWeb/Loader/FrameLoader.cpp +++ b/Userland/Libraries/LibWeb/Loader/FrameLoader.cpp @@ -143,7 +143,7 @@ bool FrameLoader::load(const LoadRequest& request, Type type) auto& url = request.url(); - if (type == Type::Navigation) { + if (type == Type::Navigation || type == Type::Reload) { if (auto* page = frame().page()) page->client().page_did_start_loading(url); }