mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-08-09 01:29:17 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibWeb: Avoid vector creation getting {short,long}hands for CSS property

Browse source 
The `shorthands_for_longhand`, `longhands_for_shorthand`, and
`expanded_longhands_for_shorthand` methods can be pretty hot in
profiles where we serialize a lot of CSS properties.

By returning a const reference to a static vector instead of allocating
and returning a new vector every time we can avoid a decent amount of
work.

Overall runtime for the particularly serialization heavy
wpt.live/css/cssom/cssom-getPropertyValue-common-checks.html
decreased by ~20% comparing before and after this change.
This commit is contained in:
Callum Law 2025-07-14 15:58:29 +12:00 • committed by Jelle Raaijmakers
commit 7cc718f1c1
Notes: github-actions[bot] 2025-07-14 07:10:06 +00:00
1 changed files with 28 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

48
Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSPropertyID.cpp
View file

@ -319,10 +319,10 @@ bool property_accepts_resolution(PropertyID, Resolution const&);
bool property_accepts_time(PropertyID, Time const&); bool property_accepts_time(PropertyID, Time const&);
bool property_is_shorthand(PropertyID); bool property_is_shorthand(PropertyID);
Vector<PropertyID> longhands_for_shorthand(PropertyID); Vector<PropertyID> const& longhands_for_shorthand(PropertyID);
Vector<PropertyID> expanded_longhands_for_shorthand(PropertyID); Vector<PropertyID> const& expanded_longhands_for_shorthand(PropertyID);
bool property_maps_to_shorthand(PropertyID); bool property_maps_to_shorthand(PropertyID);
Vector<PropertyID> shorthands_for_longhand(PropertyID); Vector<PropertyID> const& shorthands_for_longhand(PropertyID);
size_t property_maximum_value_count(PropertyID); size_t property_maximum_value_count(PropertyID);
@ -1126,7 +1126,7 @@ bool property_is_shorthand(PropertyID property_id)
)~~~"); )~~~");
generator.append(R"~~~( generator.append(R"~~~(
Vector<PropertyID> longhands_for_shorthand(PropertyID property_id) Vector<PropertyID> const& longhands_for_shorthand(PropertyID property_id)
{ {
switch (property_id) { switch (property_id) {
)~~~"); )~~~");
@ -1161,21 +1161,23 @@ Vector<PropertyID> longhands_for_shorthand(PropertyID property_id)
} }
property_generator.set("longhands", builder.to_byte_string()); property_generator.set("longhands", builder.to_byte_string());
property_generator.append(R"~~~( property_generator.append(R"~~~(
case PropertyID::@name:titlecase@: case PropertyID::@name:titlecase@: {
return { @longhands@ }; static Vector<PropertyID> longhands = { @longhands@ };
)~~~"); return longhands;
})~~~");
} }
}); });
generator.append(R"~~~( generator.append(R"~~~(
default: default:
return { }; static Vector<PropertyID> empty_longhands;
return empty_longhands;
} }
} }
)~~~"); )~~~");
generator.append(R"~~~( generator.append(R"~~~(
Vector<PropertyID> expanded_longhands_for_shorthand(PropertyID property_id) Vector<PropertyID> const& expanded_longhands_for_shorthand(PropertyID property_id)
{ {
switch (property_id) { switch (property_id) {
)~~~"); )~~~");
@ -1213,15 +1215,18 @@ Vector<PropertyID> expanded_longhands_for_shorthand(PropertyID property_id)
} }
property_generator.set("longhands", builder.to_byte_string()); property_generator.set("longhands", builder.to_byte_string());
property_generator.append(R"~~~( property_generator.append(R"~~~(
case PropertyID::@name:titlecase@: case PropertyID::@name:titlecase@: {
return { @longhands@ }; static Vector<PropertyID> longhands = { @longhands@ };
)~~~"); return longhands;
})~~~");
} }
}); });
generator.append(R"~~~( generator.append(R"~~~(
default: default: {
return { }; static Vector<PropertyID> empty_longhands;
return empty_longhands;
}
} }
} }
)~~~"); )~~~");
@ -1266,7 +1271,7 @@ bool property_maps_to_shorthand(PropertyID property_id)
)~~~"); )~~~");
generator.append(R"~~~( generator.append(R"~~~(
Vector<PropertyID> shorthands_for_longhand(PropertyID property_id) Vector<PropertyID> const& shorthands_for_longhand(PropertyID property_id)
{ {
switch (property_id) { switch (property_id) {
)~~~"); )~~~");
@ -1317,14 +1322,17 @@ Vector<PropertyID> shorthands_for_longhand(PropertyID property_id)
} }
property_generator.set("shorthands", builder.to_byte_string()); property_generator.set("shorthands", builder.to_byte_string());
property_generator.append(R"~~~( property_generator.append(R"~~~(
case PropertyID::@name:titlecase@: case PropertyID::@name:titlecase@: {
return { @shorthands@ }; static Vector<PropertyID> shorthands = { @shorthands@ };
)~~~"); return shorthands;
})~~~");
} }
generator.append(R"~~~( generator.append(R"~~~(
default: default: {
return { }; static Vector<PropertyID> empty_shorthands;
return empty_shorthands;
}
} }
} }
)~~~"); )~~~");