LibWeb/CSS: Add missing commas to rect() serialization

Gets us 1 WPT subtest pass.
Author: https://github.com/AtkinsSJ Commit: https://github.com/LadybirdBrowser/ladybird/commit/8cdb8ca193e Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/2638
2024-11-28 15:50:38 +00:00 · 2024-11-28 15:50:38 +00:00 · 8cdb8ca193 · 2024-11-30 10:03:24 +00:00
commit 8cdb8ca193
parent 5bcd9abc42
2 changed files with 4 additions and 4 deletions
--- a/Libraries/LibWeb/CSS/StyleValues/RectStyleValue.cpp
+++ b/Libraries/LibWeb/CSS/StyleValues/RectStyleValue.cpp
@ -18,7 +18,7 @@ ValueComparingNonnullRefPtr<RectStyleValue> RectStyleValue::create(EdgeRect rect

 String RectStyleValue::to_string() const
 {
-    return MUST(String::formatted("rect({} {} {} {})", m_rect.top_edge, m_rect.right_edge, m_rect.bottom_edge, m_rect.left_edge));
+    return MUST(String::formatted("rect({}, {}, {}, {})", m_rect.top_edge, m_rect.right_edge, m_rect.bottom_edge, m_rect.left_edge));
 }

 }
--- a/Tests/LibWeb/Text/expected/wpt-import/css/cssom/serialize-values.txt
+++ b/Tests/LibWeb/Text/expected/wpt-import/css/cssom/serialize-values.txt
@ -6,8 +6,8 @@ Rerun

 Found 687 tests

-462 Pass
-225 Fail
+463 Pass
+224 Fail
 Details
 Result	Test Name	MessagePass	background-attachment: scroll	
 Pass	background-attachment: fixed	
@ -315,7 +315,7 @@ Pass	clear: left
 Pass	clear: right	
 Pass	clear: both	
 Pass	clear: inherit	
-Fail	clip: rect(1em, auto, 0.5px, 2000em)	
+Pass	clip: rect(1em, auto, 0.5px, 2000em)	
 Pass	clip: auto	
 Pass	clip: inherit	
 Pass	color: black