LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-04 05:19:58 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb+Base: Use AK::SourceGenerator for error pages

Browse Source 
Instead of storing a format string in a file, let's be reasonable
and use SourceGenerator's template functionality. :^)
This commit is contained in:
Andreas Kling 2021-04-22 00:00:06 +02:00
parent edf0b14e23
commit 3857148028
Notes: sideshowbarker 2024-07-18 19:15:27 +09:00 
Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/38571480281
2 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Base/res/html/error.html
View File

@ -14,8 +14,8 @@
<body>
<header>
<img src="file:///res/icons/32x32/msgbox-warning.png" alt="Warning" width="24" height="24">
<h1>Failed to load {}</h1>
<h1>Failed to load @failed_url@</h1>
</header>
<p>Error: {}</p>
<p>Error: @error@</p>
</body>
</html>

14
Userland/Libraries/LibWeb/Loader/FrameLoader.cpp
View File

@ -26,6 +26,7 @@
#include <AK/Debug.h>
#include <AK/LexicalPath.h>
#include <AK/SourceGenerator.h>
#include <LibGemini/Document.h>
#include <LibGfx/ImageDecoder.h>
#include <LibMarkdown/Document.h>
@ -226,13 +227,12 @@ void FrameLoader::load_error_page(const URL& failed_url, const String& error)
error_page_url,
[this, failed_url, error](auto data, auto&, auto) {
VERIFY(!data.is_null());
#pragma GCC diagnostic ignored "-Wformat-nonliteral"
auto html = String::formatted(
data,
escape_html_entities(failed_url.to_string()),
escape_html_entities(error));
#pragma GCC diagnostic pop
auto document = HTML::parse_html_document(html, failed_url, "utf-8");
StringBuilder builder;
SourceGenerator generator { builder };
generator.set("failed_url", failed_url.to_string());
generator.set("error", error);
generator.append(data);
auto document = HTML::parse_html_document(generator.as_string_view(), failed_url, "utf-8");
VERIFY(document);
frame().set_document(document);
},