summaryrefslogtreecommitdiff
path: root/Userland
diff options
context:
space:
mode:
authorAli Mohammad Pur <ali.mpfard@gmail.com>2023-05-26 23:25:25 +0330
committerAndreas Kling <kling@serenityos.org>2023-05-29 05:35:41 +0200
commitb7e3a68bfc0cfeda0bbabc35d631764e4a91b77c (patch)
treea5b18de8a6d829552118dc0f08b611fef59f0c2b /Userland
parent279924242de22bd97a27a52a63469728e1f40653 (diff)
downloadserenity-b7e3a68bfc0cfeda0bbabc35d631764e4a91b77c.zip
LibWeb: Keep track of CSS property source declarations
This allows us to figure out where a specific CSS property comes from, which is going to be used in a future commit to uniquely identify running animations.
Diffstat (limited to 'Userland')
-rw-r--r--Userland/Libraries/LibWeb/CSS/StyleComputer.cpp374
-rw-r--r--Userland/Libraries/LibWeb/CSS/StyleProperties.cpp32
-rw-r--r--Userland/Libraries/LibWeb/CSS/StyleProperties.h13
-rw-r--r--Userland/Libraries/LibWeb/DOM/Element.cpp8
-rw-r--r--Userland/Libraries/LibWeb/HTML/HTMLBodyElement.cpp6
5 files changed, 223 insertions, 210 deletions
diff --git a/Userland/Libraries/LibWeb/CSS/StyleComputer.cpp b/Userland/Libraries/LibWeb/CSS/StyleComputer.cpp
index 305d69360a..86a104efdb 100644
--- a/Userland/Libraries/LibWeb/CSS/StyleComputer.cpp
+++ b/Userland/Libraries/LibWeb/CSS/StyleComputer.cpp
@@ -277,75 +277,75 @@ static bool contains(Edge a, Edge b)
return a == b || b == Edge::All;
}
-static void set_property_expanding_shorthands(StyleProperties& style, CSS::PropertyID property_id, StyleValue const& value, DOM::Document& document)
+static void set_property_expanding_shorthands(StyleProperties& style, CSS::PropertyID property_id, StyleValue const& value, DOM::Document& document, CSS::CSSStyleDeclaration const* declaration)
{
if (value.is_composite()) {
auto& composite_value = value.as_composite();
auto& properties = composite_value.sub_properties();
auto& values = composite_value.values();
for (size_t i = 0; i < properties.size(); ++i)
- set_property_expanding_shorthands(style, properties[i], values[i], document);
+ set_property_expanding_shorthands(style, properties[i], values[i], document, declaration);
}
- auto assign_edge_values = [&style](PropertyID top_property, PropertyID right_property, PropertyID bottom_property, PropertyID left_property, auto const& values) {
+ auto assign_edge_values = [&style, &declaration](PropertyID top_property, PropertyID right_property, PropertyID bottom_property, PropertyID left_property, auto const& values) {
if (values.size() == 4) {
- style.set_property(top_property, values[0]);
- style.set_property(right_property, values[1]);
- style.set_property(bottom_property, values[2]);
- style.set_property(left_property, values[3]);
+ style.set_property(top_property, values[0], declaration);
+ style.set_property(right_property, values[1], declaration);
+ style.set_property(bottom_property, values[2], declaration);
+ style.set_property(left_property, values[3], declaration);
} else if (values.size() == 3) {
- style.set_property(top_property, values[0]);
- style.set_property(right_property, values[1]);
- style.set_property(bottom_property, values[2]);
- style.set_property(left_property, values[1]);
+ style.set_property(top_property, values[0], declaration);
+ style.set_property(right_property, values[1], declaration);
+ style.set_property(bottom_property, values[2], declaration);
+ style.set_property(left_property, values[1], declaration);
} else if (values.size() == 2) {
- style.set_property(top_property, values[0]);
- style.set_property(right_property, values[1]);
- style.set_property(bottom_property, values[0]);
- style.set_property(left_property, values[1]);
+ style.set_property(top_property, values[0], declaration);
+ style.set_property(right_property, values[1], declaration);
+ style.set_property(bottom_property, values[0], declaration);
+ style.set_property(left_property, values[1], declaration);
} else if (values.size() == 1) {
- style.set_property(top_property, values[0]);
- style.set_property(right_property, values[0]);
- style.set_property(bottom_property, values[0]);
- style.set_property(left_property, values[0]);
+ style.set_property(top_property, values[0], declaration);
+ style.set_property(right_property, values[0], declaration);
+ style.set_property(bottom_property, values[0], declaration);
+ style.set_property(left_property, values[0], declaration);
}
};
if (property_id == CSS::PropertyID::TextDecoration) {
if (value.is_text_decoration()) {
auto const& text_decoration = value.as_text_decoration();
- style.set_property(CSS::PropertyID::TextDecorationLine, text_decoration.line());
- style.set_property(CSS::PropertyID::TextDecorationThickness, text_decoration.thickness());
- style.set_property(CSS::PropertyID::TextDecorationStyle, text_decoration.style());
- style.set_property(CSS::PropertyID::TextDecorationColor, text_decoration.color());
+ style.set_property(CSS::PropertyID::TextDecorationLine, text_decoration.line(), declaration);
+ style.set_property(CSS::PropertyID::TextDecorationThickness, text_decoration.thickness(), declaration);
+ style.set_property(CSS::PropertyID::TextDecorationStyle, text_decoration.style(), declaration);
+ style.set_property(CSS::PropertyID::TextDecorationColor, text_decoration.color(), declaration);
return;
}
- style.set_property(CSS::PropertyID::TextDecorationLine, value);
- style.set_property(CSS::PropertyID::TextDecorationThickness, value);
- style.set_property(CSS::PropertyID::TextDecorationStyle, value);
- style.set_property(CSS::PropertyID::TextDecorationColor, value);
+ style.set_property(CSS::PropertyID::TextDecorationLine, value, declaration);
+ style.set_property(CSS::PropertyID::TextDecorationThickness, value, declaration);
+ style.set_property(CSS::PropertyID::TextDecorationStyle, value, declaration);
+ style.set_property(CSS::PropertyID::TextDecorationColor, value, declaration);
return;
}
if (property_id == CSS::PropertyID::Overflow) {
if (value.is_overflow()) {
auto const& overflow = value.as_overflow();
- style.set_property(CSS::PropertyID::OverflowX, overflow.overflow_x());
- style.set_property(CSS::PropertyID::OverflowY, overflow.overflow_y());
+ style.set_property(CSS::PropertyID::OverflowX, overflow.overflow_x(), declaration);
+ style.set_property(CSS::PropertyID::OverflowY, overflow.overflow_y(), declaration);
return;
}
- style.set_property(CSS::PropertyID::OverflowX, value);
- style.set_property(CSS::PropertyID::OverflowY, value);
+ style.set_property(CSS::PropertyID::OverflowX, value, declaration);
+ style.set_property(CSS::PropertyID::OverflowY, value, declaration);
return;
}
if (property_id == CSS::PropertyID::Border) {
- set_property_expanding_shorthands(style, CSS::PropertyID::BorderTop, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BorderRight, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BorderBottom, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BorderLeft, value, document);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BorderTop, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BorderRight, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BorderBottom, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BorderLeft, value, document, declaration);
// FIXME: Also reset border-image, in line with the spec: https://www.w3.org/TR/css-backgrounds-3/#border-shorthands
return;
}
@@ -353,17 +353,17 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
if (property_id == CSS::PropertyID::BorderRadius) {
if (value.is_border_radius_shorthand()) {
auto const& shorthand = value.as_border_radius_shorthand();
- style.set_property(CSS::PropertyID::BorderTopLeftRadius, shorthand.top_left());
- style.set_property(CSS::PropertyID::BorderTopRightRadius, shorthand.top_right());
- style.set_property(CSS::PropertyID::BorderBottomRightRadius, shorthand.bottom_right());
- style.set_property(CSS::PropertyID::BorderBottomLeftRadius, shorthand.bottom_left());
+ style.set_property(CSS::PropertyID::BorderTopLeftRadius, shorthand.top_left(), declaration);
+ style.set_property(CSS::PropertyID::BorderTopRightRadius, shorthand.top_right(), declaration);
+ style.set_property(CSS::PropertyID::BorderBottomRightRadius, shorthand.bottom_right(), declaration);
+ style.set_property(CSS::PropertyID::BorderBottomLeftRadius, shorthand.bottom_left(), declaration);
return;
}
- style.set_property(CSS::PropertyID::BorderTopLeftRadius, value);
- style.set_property(CSS::PropertyID::BorderTopRightRadius, value);
- style.set_property(CSS::PropertyID::BorderBottomRightRadius, value);
- style.set_property(CSS::PropertyID::BorderBottomLeftRadius, value);
+ style.set_property(CSS::PropertyID::BorderTopLeftRadius, value, declaration);
+ style.set_property(CSS::PropertyID::BorderTopRightRadius, value, declaration);
+ style.set_property(CSS::PropertyID::BorderBottomRightRadius, value, declaration);
+ style.set_property(CSS::PropertyID::BorderBottomLeftRadius, value, declaration);
return;
}
@@ -393,24 +393,24 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
if (value.is_border()) {
auto const& border = value.as_border();
if (contains(Edge::Top, edge)) {
- style.set_property(PropertyID::BorderTopWidth, border.border_width());
- style.set_property(PropertyID::BorderTopStyle, border.border_style());
- style.set_property(PropertyID::BorderTopColor, border.border_color());
+ style.set_property(PropertyID::BorderTopWidth, border.border_width(), declaration);
+ style.set_property(PropertyID::BorderTopStyle, border.border_style(), declaration);
+ style.set_property(PropertyID::BorderTopColor, border.border_color(), declaration);
}
if (contains(Edge::Right, edge)) {
- style.set_property(PropertyID::BorderRightWidth, border.border_width());
- style.set_property(PropertyID::BorderRightStyle, border.border_style());
- style.set_property(PropertyID::BorderRightColor, border.border_color());
+ style.set_property(PropertyID::BorderRightWidth, border.border_width(), declaration);
+ style.set_property(PropertyID::BorderRightStyle, border.border_style(), declaration);
+ style.set_property(PropertyID::BorderRightColor, border.border_color(), declaration);
}
if (contains(Edge::Bottom, edge)) {
- style.set_property(PropertyID::BorderBottomWidth, border.border_width());
- style.set_property(PropertyID::BorderBottomStyle, border.border_style());
- style.set_property(PropertyID::BorderBottomColor, border.border_color());
+ style.set_property(PropertyID::BorderBottomWidth, border.border_width(), declaration);
+ style.set_property(PropertyID::BorderBottomStyle, border.border_style(), declaration);
+ style.set_property(PropertyID::BorderBottomColor, border.border_color(), declaration);
}
if (contains(Edge::Left, edge)) {
- style.set_property(PropertyID::BorderLeftWidth, border.border_width());
- style.set_property(PropertyID::BorderLeftStyle, border.border_style());
- style.set_property(PropertyID::BorderLeftColor, border.border_color());
+ style.set_property(PropertyID::BorderLeftWidth, border.border_width(), declaration);
+ style.set_property(PropertyID::BorderLeftStyle, border.border_style(), declaration);
+ style.set_property(PropertyID::BorderLeftColor, border.border_color(), declaration);
}
return;
}
@@ -424,10 +424,10 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
return;
}
- style.set_property(CSS::PropertyID::BorderTopStyle, value);
- style.set_property(CSS::PropertyID::BorderRightStyle, value);
- style.set_property(CSS::PropertyID::BorderBottomStyle, value);
- style.set_property(CSS::PropertyID::BorderLeftStyle, value);
+ style.set_property(CSS::PropertyID::BorderTopStyle, value, declaration);
+ style.set_property(CSS::PropertyID::BorderRightStyle, value, declaration);
+ style.set_property(CSS::PropertyID::BorderBottomStyle, value, declaration);
+ style.set_property(CSS::PropertyID::BorderLeftStyle, value, declaration);
return;
}
@@ -438,10 +438,10 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
return;
}
- style.set_property(CSS::PropertyID::BorderTopWidth, value);
- style.set_property(CSS::PropertyID::BorderRightWidth, value);
- style.set_property(CSS::PropertyID::BorderBottomWidth, value);
- style.set_property(CSS::PropertyID::BorderLeftWidth, value);
+ style.set_property(CSS::PropertyID::BorderTopWidth, value, declaration);
+ style.set_property(CSS::PropertyID::BorderRightWidth, value, declaration);
+ style.set_property(CSS::PropertyID::BorderBottomWidth, value, declaration);
+ style.set_property(CSS::PropertyID::BorderLeftWidth, value, declaration);
return;
}
@@ -452,43 +452,43 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
return;
}
- style.set_property(CSS::PropertyID::BorderTopColor, value);
- style.set_property(CSS::PropertyID::BorderRightColor, value);
- style.set_property(CSS::PropertyID::BorderBottomColor, value);
- style.set_property(CSS::PropertyID::BorderLeftColor, value);
+ style.set_property(CSS::PropertyID::BorderTopColor, value, declaration);
+ style.set_property(CSS::PropertyID::BorderRightColor, value, declaration);
+ style.set_property(CSS::PropertyID::BorderBottomColor, value, declaration);
+ style.set_property(CSS::PropertyID::BorderLeftColor, value, declaration);
return;
}
if (property_id == CSS::PropertyID::Background) {
if (value.is_background()) {
auto const& background = value.as_background();
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundColor, background.color(), document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundImage, background.image(), document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundPosition, background.position(), document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundSize, background.size(), document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundRepeat, background.repeat(), document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundAttachment, background.attachment(), document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundOrigin, background.origin(), document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundClip, background.clip(), document);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundColor, background.color(), document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundImage, background.image(), document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundPosition, background.position(), document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundSize, background.size(), document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundRepeat, background.repeat(), document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundAttachment, background.attachment(), document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundOrigin, background.origin(), document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundClip, background.clip(), document, declaration);
return;
}
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundColor, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundImage, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundPosition, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundSize, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundRepeat, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundAttachment, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundOrigin, value, document);
- set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundClip, value, document);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundColor, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundImage, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundPosition, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundSize, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundRepeat, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundAttachment, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundOrigin, value, document, declaration);
+ set_property_expanding_shorthands(style, CSS::PropertyID::BackgroundClip, value, document, declaration);
return;
}
if (property_id == CSS::PropertyID::BackgroundPosition) {
if (value.is_position()) {
auto const& position = value.as_position();
- style.set_property(CSS::PropertyID::BackgroundPositionX, position.edge_x());
- style.set_property(CSS::PropertyID::BackgroundPositionY, position.edge_y());
+ style.set_property(CSS::PropertyID::BackgroundPositionX, position.edge_x(), declaration);
+ style.set_property(CSS::PropertyID::BackgroundPositionY, position.edge_y(), declaration);
} else if (value.is_value_list()) {
// Expand background-position layer list into separate lists for x and y positions:
auto const& values_list = value.as_value_list();
@@ -506,11 +506,11 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
y_positions.unchecked_append(layer);
}
}
- style.set_property(CSS::PropertyID::BackgroundPositionX, StyleValueList::create(move(x_positions), values_list.separator()).release_value_but_fixme_should_propagate_errors());
- style.set_property(CSS::PropertyID::BackgroundPositionY, StyleValueList::create(move(y_positions), values_list.separator()).release_value_but_fixme_should_propagate_errors());
+ style.set_property(CSS::PropertyID::BackgroundPositionX, StyleValueList::create(move(x_positions), values_list.separator()).release_value_but_fixme_should_propagate_errors(), declaration);
+ style.set_property(CSS::PropertyID::BackgroundPositionY, StyleValueList::create(move(y_positions), values_list.separator()).release_value_but_fixme_should_propagate_errors(), declaration);
} else {
- style.set_property(CSS::PropertyID::BackgroundPositionX, value);
- style.set_property(CSS::PropertyID::BackgroundPositionY, value);
+ style.set_property(CSS::PropertyID::BackgroundPositionX, value, declaration);
+ style.set_property(CSS::PropertyID::BackgroundPositionY, value, declaration);
}
return;
@@ -523,10 +523,10 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
return;
}
- style.set_property(CSS::PropertyID::MarginTop, value);
- style.set_property(CSS::PropertyID::MarginRight, value);
- style.set_property(CSS::PropertyID::MarginBottom, value);
- style.set_property(CSS::PropertyID::MarginLeft, value);
+ style.set_property(CSS::PropertyID::MarginTop, value, declaration);
+ style.set_property(CSS::PropertyID::MarginRight, value, declaration);
+ style.set_property(CSS::PropertyID::MarginBottom, value, declaration);
+ style.set_property(CSS::PropertyID::MarginLeft, value, declaration);
return;
}
@@ -537,47 +537,47 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
return;
}
- style.set_property(CSS::PropertyID::PaddingTop, value);
- style.set_property(CSS::PropertyID::PaddingRight, value);
- style.set_property(CSS::PropertyID::PaddingBottom, value);
- style.set_property(CSS::PropertyID::PaddingLeft, value);
+ style.set_property(CSS::PropertyID::PaddingTop, value, declaration);
+ style.set_property(CSS::PropertyID::PaddingRight, value, declaration);
+ style.set_property(CSS::PropertyID::PaddingBottom, value, declaration);
+ style.set_property(CSS::PropertyID::PaddingLeft, value, declaration);
return;
}
if (property_id == CSS::PropertyID::ListStyle) {
if (value.is_list_style()) {
auto const& list_style = value.as_list_style();
- style.set_property(CSS::PropertyID::ListStylePosition, list_style.position());
- style.set_property(CSS::PropertyID::ListStyleImage, list_style.image());
- style.set_property(CSS::PropertyID::ListStyleType, list_style.style_type());
+ style.set_property(CSS::PropertyID::ListStylePosition, list_style.position(), declaration);
+ style.set_property(CSS::PropertyID::ListStyleImage, list_style.image(), declaration);
+ style.set_property(CSS::PropertyID::ListStyleType, list_style.style_type(), declaration);
return;
}
- style.set_property(CSS::PropertyID::ListStylePosition, value);
- style.set_property(CSS::PropertyID::ListStyleImage, value);
- style.set_property(CSS::PropertyID::ListStyleType, value);
+ style.set_property(CSS::PropertyID::ListStylePosition, value, declaration);
+ style.set_property(CSS::PropertyID::ListStyleImage, value, declaration);
+ style.set_property(CSS::PropertyID::ListStyleType, value, declaration);
return;
}
if (property_id == CSS::PropertyID::Font) {
if (value.is_font()) {
auto const& font_shorthand = value.as_font();
- style.set_property(CSS::PropertyID::FontSize, font_shorthand.font_size());
- style.set_property(CSS::PropertyID::FontFamily, font_shorthand.font_families());
- style.set_property(CSS::PropertyID::FontStretch, font_shorthand.font_stretch());
- style.set_property(CSS::PropertyID::FontStyle, font_shorthand.font_style());
- style.set_property(CSS::PropertyID::FontWeight, font_shorthand.font_weight());
- style.set_property(CSS::PropertyID::LineHeight, font_shorthand.line_height());
+ style.set_property(CSS::PropertyID::FontSize, font_shorthand.font_size(), declaration);
+ style.set_property(CSS::PropertyID::FontFamily, font_shorthand.font_families(), declaration);
+ style.set_property(CSS::PropertyID::FontStretch, font_shorthand.font_stretch(), declaration);
+ style.set_property(CSS::PropertyID::FontStyle, font_shorthand.font_style(), declaration);
+ style.set_property(CSS::PropertyID::FontWeight, font_shorthand.font_weight(), declaration);
+ style.set_property(CSS::PropertyID::LineHeight, font_shorthand.line_height(), declaration);
// FIXME: Implement font-variant
return;
}
- style.set_property(CSS::PropertyID::FontStretch, value);
- style.set_property(CSS::PropertyID::FontSize, value);
- style.set_property(CSS::PropertyID::FontFamily, value);
- style.set_property(CSS::PropertyID::FontStyle, value);
- style.set_property(CSS::PropertyID::FontWeight, value);
- style.set_property(CSS::PropertyID::LineHeight, value);
+ style.set_property(CSS::PropertyID::FontStretch, value, declaration);
+ style.set_property(CSS::PropertyID::FontSize, value, declaration);
+ style.set_property(CSS::PropertyID::FontFamily, value, declaration);
+ style.set_property(CSS::PropertyID::FontStyle, value, declaration);
+ style.set_property(CSS::PropertyID::FontWeight, value, declaration);
+ style.set_property(CSS::PropertyID::LineHeight, value, declaration);
// FIXME: Implement font-variant
return;
}
@@ -585,106 +585,106 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
if (property_id == CSS::PropertyID::Flex) {
if (value.is_flex()) {
auto const& flex = value.as_flex();
- style.set_property(CSS::PropertyID::FlexGrow, flex.grow());
- style.set_property(CSS::PropertyID::FlexShrink, flex.shrink());
- style.set_property(CSS::PropertyID::FlexBasis, flex.basis());
+ style.set_property(CSS::PropertyID::FlexGrow, flex.grow(), declaration);
+ style.set_property(CSS::PropertyID::FlexShrink, flex.shrink(), declaration);
+ style.set_property(CSS::PropertyID::FlexBasis, flex.basis(), declaration);
return;
}
- style.set_property(CSS::PropertyID::FlexGrow, value);
- style.set_property(CSS::PropertyID::FlexShrink, value);
- style.set_property(CSS::PropertyID::FlexBasis, value);
+ style.set_property(CSS::PropertyID::FlexGrow, value, declaration);
+ style.set_property(CSS::PropertyID::FlexShrink, value, declaration);
+ style.set_property(CSS::PropertyID::FlexBasis, value, declaration);
return;
}
if (property_id == CSS::PropertyID::FlexFlow) {
if (value.is_flex_flow()) {
auto const& flex_flow = value.as_flex_flow();
- style.set_property(CSS::PropertyID::FlexDirection, flex_flow.flex_direction());
- style.set_property(CSS::PropertyID::FlexWrap, flex_flow.flex_wrap());
+ style.set_property(CSS::PropertyID::FlexDirection, flex_flow.flex_direction(), declaration);
+ style.set_property(CSS::PropertyID::FlexWrap, flex_flow.flex_wrap(), declaration);
return;
}
- style.set_property(CSS::PropertyID::FlexDirection, value);
- style.set_property(CSS::PropertyID::FlexWrap, value);
+ style.set_property(CSS::PropertyID::FlexDirection, value, declaration);
+ style.set_property(CSS::PropertyID::FlexWrap, value, declaration);
return;
}
if (property_id == CSS::PropertyID::GridArea) {
if (value.is_grid_area_shorthand()) {
auto const& shorthand = value.as_grid_area_shorthand();
- style.set_property(CSS::PropertyID::GridRowStart, shorthand.row_start());
- style.set_property(CSS::PropertyID::GridColumnStart, shorthand.column_start());
- style.set_property(CSS::PropertyID::GridRowEnd, shorthand.row_end());
- style.set_property(CSS::PropertyID::GridColumnEnd, shorthand.column_end());
+ style.set_property(CSS::PropertyID::GridRowStart, shorthand.row_start(), declaration);
+ style.set_property(CSS::PropertyID::GridColumnStart, shorthand.column_start(), declaration);
+ style.set_property(CSS::PropertyID::GridRowEnd, shorthand.row_end(), declaration);
+ style.set_property(CSS::PropertyID::GridColumnEnd, shorthand.column_end(), declaration);
return;
}
- style.set_property(CSS::PropertyID::GridRowStart, value);
- style.set_property(CSS::PropertyID::GridColumnStart, value);
- style.set_property(CSS::PropertyID::GridRowEnd, value);
- style.set_property(CSS::PropertyID::GridColumnEnd, value);
+ style.set_property(CSS::PropertyID::GridRowStart, value, declaration);
+ style.set_property(CSS::PropertyID::GridColumnStart, value, declaration);
+ style.set_property(CSS::PropertyID::GridRowEnd, value, declaration);
+ style.set_property(CSS::PropertyID::GridColumnEnd, value, declaration);
return;
}
if (property_id == CSS::PropertyID::GridColumn) {
if (value.is_grid_track_placement_shorthand()) {
auto const& shorthand = value.as_grid_track_placement_shorthand();
- style.set_property(CSS::PropertyID::GridColumnStart, shorthand.start());
- style.set_property(CSS::PropertyID::GridColumnEnd, shorthand.end());
+ style.set_property(CSS::PropertyID::GridColumnStart, shorthand.start(), declaration);
+ style.set_property(CSS::PropertyID::GridColumnEnd, shorthand.end(), declaration);
return;
}
- style.set_property(CSS::PropertyID::GridColumnStart, value);
- style.set_property(CSS::PropertyID::GridColumnEnd, value);
+ style.set_property(CSS::PropertyID::GridColumnStart, value, declaration);
+ style.set_property(CSS::PropertyID::GridColumnEnd, value, declaration);
return;
}
if (property_id == CSS::PropertyID::GridRow) {
if (value.is_grid_track_placement_shorthand()) {
auto const& shorthand = value.as_grid_track_placement_shorthand();
- style.set_property(CSS::PropertyID::GridRowStart, shorthand.start());
- style.set_property(CSS::PropertyID::GridRowEnd, shorthand.end());
+ style.set_property(CSS::PropertyID::GridRowStart, shorthand.start(), declaration);
+ style.set_property(CSS::PropertyID::GridRowEnd, shorthand.end(), declaration);
return;
}
- style.set_property(CSS::PropertyID::GridRowStart, value);
- style.set_property(CSS::PropertyID::GridRowEnd, value);
+ style.set_property(CSS::PropertyID::GridRowStart, value, declaration);
+ style.set_property(CSS::PropertyID::GridRowEnd, value, declaration);
return;
}
if (property_id == CSS::PropertyID::GridTemplate || property_id == CSS::PropertyID::Grid) {
if (value.is_grid_track_size_list_shorthand()) {
auto const& shorthand = value.as_grid_track_size_list_shorthand();
- style.set_property(CSS::PropertyID::GridTemplateAreas, shorthand.areas());
- style.set_property(CSS::PropertyID::GridTemplateRows, shorthand.rows());
- style.set_property(CSS::PropertyID::GridTemplateColumns, shorthand.columns());
+ style.set_property(CSS::PropertyID::GridTemplateAreas, shorthand.areas(), declaration);
+ style.set_property(CSS::PropertyID::GridTemplateRows, shorthand.rows(), declaration);
+ style.set_property(CSS::PropertyID::GridTemplateColumns, shorthand.columns(), declaration);
return;
}
- style.set_property(CSS::PropertyID::GridTemplateAreas, value);
- style.set_property(CSS::PropertyID::GridTemplateRows, value);
- style.set_property(CSS::PropertyID::GridTemplateColumns, value);
+ style.set_property(CSS::PropertyID::GridTemplateAreas, value, declaration);
+ style.set_property(CSS::PropertyID::GridTemplateRows, value, declaration);
+ style.set_property(CSS::PropertyID::GridTemplateColumns, value, declaration);
return;
}
if (property_id == CSS::PropertyID::Gap || property_id == CSS::PropertyID::GridGap) {
if (value.is_value_list()) {
auto const& values_list = value.as_value_list();
- style.set_property(CSS::PropertyID::RowGap, values_list.values()[0]);
- style.set_property(CSS::PropertyID::ColumnGap, values_list.values()[1]);
+ style.set_property(CSS::PropertyID::RowGap, values_list.values()[0], declaration);
+ style.set_property(CSS::PropertyID::ColumnGap, values_list.values()[1], declaration);
return;
}
- style.set_property(CSS::PropertyID::RowGap, value);
- style.set_property(CSS::PropertyID::ColumnGap, value);
+ style.set_property(CSS::PropertyID::RowGap, value, declaration);
+ style.set_property(CSS::PropertyID::ColumnGap, value, declaration);
return;
}
if (property_id == CSS::PropertyID::RowGap || property_id == CSS::PropertyID::GridRowGap) {
- style.set_property(CSS::PropertyID::RowGap, value);
+ style.set_property(CSS::PropertyID::RowGap, value, declaration);
return;
}
if (property_id == CSS::PropertyID::ColumnGap || property_id == CSS::PropertyID::GridColumnGap) {
- style.set_property(CSS::PropertyID::ColumnGap, value);
+ style.set_property(CSS::PropertyID::ColumnGap, value, declaration);
return;
}
@@ -694,21 +694,21 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
if (is_horizontal) {
if (property_id == CSS::PropertyID::MaxInlineSize) {
- style.set_property(CSS::PropertyID::MaxWidth, value);
+ style.set_property(CSS::PropertyID::MaxWidth, value, declaration);
} else {
- style.set_property(CSS::PropertyID::MinWidth, value);
+ style.set_property(CSS::PropertyID::MinWidth, value, declaration);
}
} else {
if (property_id == CSS::PropertyID::MaxInlineSize) {
- style.set_property(CSS::PropertyID::MaxHeight, value);
+ style.set_property(CSS::PropertyID::MaxHeight, value, declaration);
} else {
- style.set_property(CSS::PropertyID::MinHeight, value);
+ style.set_property(CSS::PropertyID::MinHeight, value, declaration);
}
}
return;
}
- style.set_property(property_id, value);
+ style.set_property(property_id, value, declaration);
}
static RefPtr<StyleValue const> get_custom_property(DOM::Element const& element, Optional<CSS::Selector::PseudoElement> pseudo_element, FlyString const& custom_property_name)
@@ -934,7 +934,7 @@ void StyleComputer::cascade_declarations(StyleProperties& style, DOM::Element& e
property_value = resolved.release_nonnull();
}
if (!property_value->is_unresolved())
- set_property_expanding_shorthands(style, property.property_id, property_value, m_document);
+ set_property_expanding_shorthands(style, property.property_id, property_value, m_document, &match.rule->declaration());
}
}
@@ -949,7 +949,7 @@ void StyleComputer::cascade_declarations(StyleProperties& style, DOM::Element& e
property_value = resolved.release_nonnull();
}
if (!property_value->is_unresolved())
- set_property_expanding_shorthands(style, property.property_id, property_value, m_document);
+ set_property_expanding_shorthands(style, property.property_id, property_value, m_document, inline_style);
}
}
}
@@ -1025,9 +1025,9 @@ ErrorOr<void> StyleComputer::compute_cascaded_values(StyleProperties& style, DOM
for (auto i = to_underlying(CSS::first_property_id); i <= to_underlying(CSS::last_property_id); ++i) {
auto property_id = (CSS::PropertyID)i;
auto& property = style.m_property_values[i];
- if (property && property->is_unresolved()) {
- if (auto resolved = resolve_unresolved_style_value(element, pseudo_element, property_id, property->as_unresolved()))
- property = resolved.release_nonnull();
+ if (property.has_value() && property->style->is_unresolved()) {
+ if (auto resolved = resolve_unresolved_style_value(element, pseudo_element, property_id, property->style->as_unresolved()))
+ property->style = resolved.release_nonnull();
}
}
}
@@ -1077,33 +1077,33 @@ void StyleComputer::compute_defaulted_property_value(StyleProperties& style, DOM
// FIXME: If we don't know the correct initial value for a property, we fall back to InitialStyleValue.
auto& value_slot = style.m_property_values[to_underlying(property_id)];
- if (!value_slot) {
+ if (!value_slot.has_value()) {
if (is_inherited_property(property_id))
- style.m_property_values[to_underlying(property_id)] = get_inherit_value(document().realm(), property_id, element, pseudo_element);
+ style.m_property_values[to_underlying(property_id)] = { { get_inherit_value(document().realm(), property_id, element, pseudo_element), nullptr } };
else
- style.m_property_values[to_underlying(property_id)] = property_initial_value(document().realm(), property_id).release_value_but_fixme_should_propagate_errors();
+ style.m_property_values[to_underlying(property_id)] = { { property_initial_value(document().realm(), property_id).release_value_but_fixme_should_propagate_errors(), nullptr } };
return;
}
- if (value_slot->is_initial()) {
- value_slot = property_initial_value(document().realm(), property_id).release_value_but_fixme_should_propagate_errors();
+ if (value_slot->style->is_initial()) {
+ value_slot->style = property_initial_value(document().realm(), property_id).release_value_but_fixme_should_propagate_errors();
return;
}
- if (value_slot->is_inherit()) {
- value_slot = get_inherit_value(document().realm(), property_id, element, pseudo_element);
+ if (value_slot->style->is_inherit()) {
+ value_slot->style = get_inherit_value(document().realm(), property_id, element, pseudo_element);
return;
}
// https://www.w3.org/TR/css-cascade-4/#inherit-initial
// If the cascaded value of a property is the unset keyword,
- if (value_slot->is_unset()) {
+ if (value_slot->style->is_unset()) {
if (is_inherited_property(property_id)) {
// then if it is an inherited property, this is treated as inherit,
- value_slot = get_inherit_value(document().realm(), property_id, element, pseudo_element);
+ value_slot->style = get_inherit_value(document().realm(), property_id, element, pseudo_element);
} else {
// and if it is not, this is treated as initial.
- value_slot = property_initial_value(document().realm(), property_id).release_value_but_fixme_should_propagate_errors();
+ value_slot->style = property_initial_value(document().realm(), property_id).release_value_but_fixme_should_propagate_errors();
}
}
}
@@ -1402,8 +1402,8 @@ void StyleComputer::compute_font(StyleProperties& style, DOM::Element const* ele
FontCache::the().set(font_selector, *found_font);
- style.set_property(CSS::PropertyID::FontSize, LengthStyleValue::create(CSS::Length::make_px(font_size_in_px)).release_value_but_fixme_should_propagate_errors());
- style.set_property(CSS::PropertyID::FontWeight, NumericStyleValue::create_integer(weight).release_value_but_fixme_should_propagate_errors());
+ style.set_property(CSS::PropertyID::FontSize, LengthStyleValue::create(CSS::Length::make_px(font_size_in_px)).release_value_but_fixme_should_propagate_errors(), nullptr);
+ style.set_property(CSS::PropertyID::FontWeight, NumericStyleValue::create_integer(weight).release_value_but_fixme_should_propagate_errors(), nullptr);
style.set_computed_font(found_font.release_nonnull());
@@ -1448,24 +1448,24 @@ ErrorOr<void> StyleComputer::absolutize_values(StyleProperties& style, DOM::Elem
// We have to resolve them right away, so that the *computed* line-height is ready for inheritance.
// We can't simply absolutize *all* percentage values against the font size,
// because most percentages are relative to containing block metrics.
- auto& line_height_value_slot = style.m_property_values[to_underlying(CSS::PropertyID::LineHeight)];
- if (line_height_value_slot && line_height_value_slot->is_percentage()) {
- line_height_value_slot = TRY(LengthStyleValue::create(
- Length::make_px(font_size * static_cast<double>(line_height_value_slot->as_percentage().percentage().as_fraction()))));
+ auto line_height_value_slot = style.m_property_values[to_underlying(CSS::PropertyID::LineHeight)].map([](auto& x) -> auto& { return x.style; });
+ if (line_height_value_slot.has_value() && (*line_height_value_slot)->is_percentage()) {
+ *line_height_value_slot = TRY(LengthStyleValue::create(
+ Length::make_px(font_size * static_cast<double>((*line_height_value_slot)->as_percentage().percentage().as_fraction()))));
}
auto line_height = style.line_height(viewport_rect(), font_metrics, m_root_element_font_metrics);
font_metrics.line_height = line_height;
// NOTE: line-height might be using lh which should be resolved against the parent line height (like we did here already)
- if (line_height_value_slot && line_height_value_slot->is_length())
- line_height_value_slot = TRY(LengthStyleValue::create(Length::make_px(line_height)));
+ if (line_height_value_slot.has_value() && (*line_height_value_slot)->is_length())
+ (*line_height_value_slot) = TRY(LengthStyleValue::create(Length::make_px(line_height)));
for (size_t i = 0; i < style.m_property_values.size(); ++i) {
auto& value_slot = style.m_property_values[i];
- if (!value_slot)
+ if (!value_slot.has_value())
continue;
- value_slot = TRY(value_slot->absolutized(viewport_rect(), font_metrics, m_root_element_font_metrics));
+ value_slot->style = TRY(value_slot->style->absolutized(viewport_rect(), font_metrics, m_root_element_font_metrics));
}
return {};
}
@@ -1559,7 +1559,7 @@ void StyleComputer::transform_box_type_if_needed(StyleProperties& style, DOM::El
}
if (new_display != display)
- style.set_property(CSS::PropertyID::Display, DisplayStyleValue::create(new_display).release_value_but_fixme_should_propagate_errors());
+ style.set_property(CSS::PropertyID::Display, DisplayStyleValue::create(new_display).release_value_but_fixme_should_propagate_errors(), style.property_source_declaration(CSS::PropertyID::Display));
}
NonnullRefPtr<StyleProperties> StyleComputer::create_document_style() const
@@ -1568,9 +1568,9 @@ NonnullRefPtr<StyleProperties> StyleComputer::create_document_style() const
compute_font(style, nullptr, {});
compute_defaulted_values(style, nullptr, {});
absolutize_values(style, nullptr, {}).release_value_but_fixme_should_propagate_errors();
- style->set_property(CSS::PropertyID::Width, CSS::LengthStyleValue::create(CSS::Length::make_px(viewport_rect().width())).release_value_but_fixme_should_propagate_errors());
- style->set_property(CSS::PropertyID::Height, CSS::LengthStyleValue::create(CSS::Length::make_px(viewport_rect().height())).release_value_but_fixme_should_propagate_errors());
- style->set_property(CSS::PropertyID::Display, CSS::DisplayStyleValue::create(CSS::Display::from_short(CSS::Display::Short::Block)).release_value_but_fixme_should_propagate_errors());
+ style->set_property(CSS::PropertyID::Width, CSS::LengthStyleValue::create(CSS::Length::make_px(viewport_rect().width())).release_value_but_fixme_should_propagate_errors(), nullptr);
+ style->set_property(CSS::PropertyID::Height, CSS::LengthStyleValue::create(CSS::Length::make_px(viewport_rect().height())).release_value_but_fixme_should_propagate_errors(), nullptr);
+ style->set_property(CSS::PropertyID::Display, CSS::DisplayStyleValue::create(CSS::Display::from_short(CSS::Display::Short::Block)).release_value_but_fixme_should_propagate_errors(), nullptr);
return style;
}
diff --git a/Userland/Libraries/LibWeb/CSS/StyleProperties.cpp b/Userland/Libraries/LibWeb/CSS/StyleProperties.cpp
index da20b86d46..2120116088 100644
--- a/Userland/Libraries/LibWeb/CSS/StyleProperties.cpp
+++ b/Userland/Libraries/LibWeb/CSS/StyleProperties.cpp
@@ -45,22 +45,30 @@ NonnullRefPtr<StyleProperties> StyleProperties::clone() const
return adopt_ref(*new StyleProperties(*this));
}
-void StyleProperties::set_property(CSS::PropertyID id, NonnullRefPtr<StyleValue const> value)
+void StyleProperties::set_property(CSS::PropertyID id, NonnullRefPtr<StyleValue const> value, CSS::CSSStyleDeclaration const* source_declaration)
{
- m_property_values[to_underlying(id)] = move(value);
+ m_property_values[to_underlying(id)] = StyleAndSourceDeclaration { move(value), source_declaration };
}
NonnullRefPtr<StyleValue const> StyleProperties::property(CSS::PropertyID property_id) const
{
auto value = m_property_values[to_underlying(property_id)];
// By the time we call this method, all properties have values assigned.
- VERIFY(!value.is_null());
- return value.release_nonnull();
+ VERIFY(value.has_value());
+ return value->style;
}
RefPtr<StyleValue const> StyleProperties::maybe_null_property(CSS::PropertyID property_id) const
{
- return m_property_values[to_underlying(property_id)];
+ auto value = m_property_values[to_underlying(property_id)];
+ if (value.has_value())
+ return value->style;
+ return {};
+}
+
+CSS::CSSStyleDeclaration const* StyleProperties::property_source_declaration(CSS::PropertyID property_id) const
+{
+ return m_property_values[to_underlying(property_id)].map([](auto& value) { return value.declaration; }).value_or(nullptr);
}
CSS::Size StyleProperties::size_value(CSS::PropertyID id) const
@@ -509,17 +517,17 @@ bool StyleProperties::operator==(StyleProperties const& other) const
return false;
for (size_t i = 0; i < m_property_values.size(); ++i) {
- auto const& my_ptr = m_property_values[i];
- auto const& other_ptr = other.m_property_values[i];
- if (!my_ptr) {
- if (other_ptr)
+ auto const& my_style = m_property_values[i];
+ auto const& other_style = other.m_property_values[i];
+ if (!my_style.has_value()) {
+ if (other_style.has_value())
return false;
continue;
}
- if (!other_ptr)
+ if (!other_style.has_value())
return false;
- auto const& my_value = *my_ptr;
- auto const& other_value = *other_ptr;
+ auto const& my_value = *my_style->style;
+ auto const& other_value = *other_style->style;
if (my_value.type() != other_value.type())
return false;
if (my_value != other_value)
diff --git a/Userland/Libraries/LibWeb/CSS/StyleProperties.h b/Userland/Libraries/LibWeb/CSS/StyleProperties.h
index 8d9e31c789..84ef77c749 100644
--- a/Userland/Libraries/LibWeb/CSS/StyleProperties.h
+++ b/Userland/Libraries/LibWeb/CSS/StyleProperties.h
@@ -30,17 +30,18 @@ public:
inline void for_each_property(Callback callback) const
{
for (size_t i = 0; i < m_property_values.size(); ++i) {
- if (m_property_values[i])
- callback((CSS::PropertyID)i, *m_property_values[i]);
+ if (m_property_values[i].has_value())
+ callback((CSS::PropertyID)i, *m_property_values[i]->style);
}
}
auto& properties() { return m_property_values; }
auto const& properties() const { return m_property_values; }
- void set_property(CSS::PropertyID, NonnullRefPtr<StyleValue const> value);
+ void set_property(CSS::PropertyID, NonnullRefPtr<StyleValue const> value, CSS::CSSStyleDeclaration const* source_declaration = nullptr);
NonnullRefPtr<StyleValue const> property(CSS::PropertyID) const;
RefPtr<StyleValue const> maybe_null_property(CSS::PropertyID) const;
+ CSS::CSSStyleDeclaration const* property_source_declaration(CSS::PropertyID) const;
CSS::Size size_value(CSS::PropertyID) const;
LengthPercentage length_percentage_or_fallback(CSS::PropertyID, LengthPercentage const& fallback) const;
@@ -129,7 +130,11 @@ public:
private:
friend class StyleComputer;
- Array<RefPtr<StyleValue const>, to_underlying(CSS::last_property_id) + 1> m_property_values;
+ struct StyleAndSourceDeclaration {
+ NonnullRefPtr<StyleValue const> style;
+ CSS::CSSStyleDeclaration const* declaration = nullptr;
+ };
+ Array<Optional<StyleAndSourceDeclaration>, to_underlying(CSS::last_property_id) + 1> m_property_values;
Optional<CSS::Overflow> overflow(CSS::PropertyID) const;
Vector<CSS::ShadowData> shadow(CSS::PropertyID) const;
diff --git a/Userland/Libraries/LibWeb/DOM/Element.cpp b/Userland/Libraries/LibWeb/DOM/Element.cpp
index cccc9fda1a..18daa1d700 100644
--- a/Userland/Libraries/LibWeb/DOM/Element.cpp
+++ b/Userland/Libraries/LibWeb/DOM/Element.cpp
@@ -412,10 +412,10 @@ static Element::RequiredInvalidationAfterStyleChange compute_required_invalidati
auto property_id = static_cast<CSS::PropertyID>(i);
auto const& old_value = old_style.properties()[i];
auto const& new_value = new_style.properties()[i];
- if (!old_value && !new_value)
+ if (!old_value.has_value() && !new_value.has_value())
continue;
- bool const property_value_changed = (!old_value || !new_value) || *old_value != *new_value;
+ bool const property_value_changed = (!old_value.has_value() || !new_value.has_value()) || *old_value->style != *new_value->style;
if (!property_value_changed)
continue;
@@ -428,7 +428,7 @@ static Element::RequiredInvalidationAfterStyleChange compute_required_invalidati
// OPTIMIZATION: Special handling for CSS `visibility`:
if (property_id == CSS::PropertyID::Visibility) {
// We don't need to relayout if the visibility changes from visible to hidden or vice versa. Only collapse requires relayout.
- if ((old_value && old_value->to_identifier() == CSS::ValueID::Collapse) != (new_value && new_value->to_identifier() == CSS::ValueID::Collapse))
+ if ((old_value.has_value() && old_value->style->to_identifier() == CSS::ValueID::Collapse) != (new_value.has_value() && new_value->style->to_identifier() == CSS::ValueID::Collapse))
invalidation.relayout = true;
// Of course, we still have to repaint on any visibility change.
invalidation.repaint = true;
@@ -481,7 +481,7 @@ NonnullRefPtr<CSS::StyleProperties> Element::resolved_css_values()
auto maybe_value = element_computed_style->property(property_id);
if (!maybe_value.has_value())
continue;
- properties->set_property(property_id, maybe_value.release_value().value);
+ properties->set_property(property_id, maybe_value.release_value().value, nullptr);
}
return properties;
diff --git a/Userland/Libraries/LibWeb/HTML/HTMLBodyElement.cpp b/Userland/Libraries/LibWeb/HTML/HTMLBodyElement.cpp
index 096a5bb1b1..83eb2b6c71 100644
--- a/Userland/Libraries/LibWeb/HTML/HTMLBodyElement.cpp
+++ b/Userland/Libraries/LibWeb/HTML/HTMLBodyElement.cpp
@@ -37,15 +37,15 @@ void HTMLBodyElement::apply_presentational_hints(CSS::StyleProperties& style) co
// https://html.spec.whatwg.org/multipage/rendering.html#the-page:rules-for-parsing-a-legacy-colour-value
auto color = parse_legacy_color_value(value);
if (color.has_value())
- style.set_property(CSS::PropertyID::BackgroundColor, CSS::ColorStyleValue::create(color.value()).release_value_but_fixme_should_propagate_errors());
+ style.set_property(CSS::PropertyID::BackgroundColor, CSS::ColorStyleValue::create(color.value()).release_value_but_fixme_should_propagate_errors(), nullptr);
} else if (name.equals_ignoring_ascii_case("text"sv)) {
// https://html.spec.whatwg.org/multipage/rendering.html#the-page:rules-for-parsing-a-legacy-colour-value-2
auto color = parse_legacy_color_value(value);
if (color.has_value())
- style.set_property(CSS::PropertyID::Color, CSS::ColorStyleValue::create(color.value()).release_value_but_fixme_should_propagate_errors());
+ style.set_property(CSS::PropertyID::Color, CSS::ColorStyleValue::create(color.value()).release_value_but_fixme_should_propagate_errors(), nullptr);
} else if (name.equals_ignoring_ascii_case("background"sv)) {
VERIFY(m_background_style_value);
- style.set_property(CSS::PropertyID::BackgroundImage, *m_background_style_value);
+ style.set_property(CSS::PropertyID::BackgroundImage, *m_background_style_value, nullptr);
}
});
}