mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-04-22 20:45:14 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

LibWeb/CSS: Automate parsing view-transition-name

Browse source
This commit is contained in:
Sam Atkins 2025-02-25 11:47:36 +00:00
parent c729c3fcee
commit ab4b46f990
Notes: github-actions[bot] 2025-02-26 11:24:05 +00:00
3 changed files with 3 additions and 37 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
Libraries/LibWeb/CSS/ComputedProperties.cpp
View file

@ -1602,12 +1602,8 @@ MixBlendMode ComputedProperties::mix_blend_mode() const
Optional<FlyString> ComputedProperties::view_transition_name() const
{
auto const& value = property(PropertyID::ViewTransitionName);
if (value.is_custom_ident()) {
auto ident = value.as_custom_ident().custom_ident();
if (ident == "none"_fly_string)
return {};
return ident;
}
if (value.is_custom_ident())
return value.as_custom_ident().custom_ident();
return {};
}

1
Libraries/LibWeb/CSS/Parser/Parser.h
View file

@ -404,7 +404,6 @@ private:
RefPtr<CSSStyleValue> parse_transition_value(TokenStream<ComponentValue>&);
RefPtr<CSSStyleValue> parse_translate_value(TokenStream<ComponentValue>&);
RefPtr<CSSStyleValue> parse_scale_value(TokenStream<ComponentValue>&);
RefPtr<CSSStyleValue> parse_view_transition_name_value(TokenStream<ComponentValue>&);
RefPtr<CSSStyleValue> parse_grid_track_size_list(TokenStream<ComponentValue>&, bool allow_separate_line_name_blocks = false);
RefPtr<CSSStyleValue> parse_grid_auto_track_sizes(TokenStream<ComponentValue>&);
RefPtr<GridAutoFlowStyleValue> parse_grid_auto_flow_value(TokenStream<ComponentValue>&);

31
Libraries/LibWeb/CSS/Parser/PropertyParsing.cpp
View file

@ -158,7 +158,7 @@ Optional<Parser::PropertyAndValue> Parser::parse_css_value_for_properties(Readon
// Custom idents
if (auto property = any_property_accepts_type(property_ids, ValueType::CustomIdent); property.has_value()) {
auto context_guard = push_temporary_value_parsing_context(*property);
if (auto custom_ident = parse_custom_ident_value(tokens, {}))
if (auto custom_ident = parse_custom_ident_value(tokens, property_custom_ident_blacklist(*property)))
return PropertyAndValue { *property, custom_ident };
}
}
@ -714,10 +714,6 @@ Parser::ParseErrorOr<NonnullRefPtr<CSSStyleValue>> Parser::parse_css_value(Prope
if (auto parsed_value = parse_scale_value(tokens); parsed_value && !tokens.has_next_token())
return parsed_value.release_nonnull();
return ParseError::SyntaxError;
case PropertyID::ViewTransitionName:
if (auto parsed_value = parse_view_transition_name_value(tokens); parsed_value && !tokens.has_next_token())
return parsed_value.release_nonnull();
return ParseError::SyntaxError;
default:
break;
}
@ -4443,29 +4439,4 @@ RefPtr<CSSStyleValue> Parser::parse_filter_value_list_value(TokenStream<Componen
return FilterValueListStyleValue::create(move(filter_value_list));
}
RefPtr<CSSStyleValue> Parser::parse_view_transition_name_value(TokenStream<ComponentValue>& tokens)
{
// none | <custom-ident>
tokens.discard_whitespace();
{
auto transaction = tokens.begin_transaction();
// The values 'none' and 'auto' are excluded from <custom-ident> here.
// Note: Only auto is excluded here since none isn't parsed differently.
auto ident = parse_custom_ident_value(tokens, { { "auto"sv } });
if (!ident)
return {};
tokens.discard_whitespace();
transaction.commit();
if (Infra::is_ascii_case_insensitive_match(ident->custom_ident().to_string(), "none"sv)) {
return CustomIdentStyleValue::create("none"_fly_string);
} else {
return ident;
}
}
return {};
}
}