eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-05-05 06:32:30 +02:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb/DOM: Always give Document a FontFaceSet

Browse Source 
In practice, the event loop queries Document::fonts(), so we don't gain
anything by delaying the allocation, apart from making it unclear when
it happens.
This commit is contained in:
Sam Atkins
2026-02-20 11:43:14 +00:00
parent f82201e908
commit bca6f3e4b5
Notes: github-actions[bot] 2026-02-24 15:46:00 +00:00 
Author: https://github.com/AtkinsSJ
Commit: https://github.com/LadybirdBrowser/ladybird/commit/bca6f3e4b5b
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/8110
2 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
Libraries/LibWeb/DOM/Document.cpp
View File

@@ -501,6 +501,7 @@ Document::Document(JS::Realm& realm, URL::URL const& url, TemporaryDocumentForFr
, m_style_computer(realm.heap().allocate<CSS::StyleComputer>(*this))
, m_font_computer(realm.heap().allocate<CSS::FontComputer>(*this))
, m_url(url)
, m_fonts(CSS::FontFaceSet::create(realm))
, m_temporary_document_for_fragment_parsing(temporary_document_for_fragment_parsing)
, m_editing_host_manager(EditingHostManager::create(realm, *this))
, m_dynamic_view_transition_style_sheet(parse_css_stylesheet(CSS::Parser::ParsingParams(realm), ""sv, {}))
@@ -2319,9 +2320,7 @@ GC::Ref<HTML::HTMLAllCollection> Document::all()
// https://drafts.csswg.org/css-font-loading/#font-source
GC::Ref<CSS::FontFaceSet> Document::fonts()
{
if (!m_fonts)
m_fonts = CSS::FontFaceSet::create(realm());
return *m_fonts;
return m_fonts;
}
// https://html.spec.whatwg.org/multipage/obsolete.html#dom-document-clear