Title: [273554] trunk/LayoutTests
Revision
273554
Author
[email protected]
Date
2021-02-26 06:19:40 -0800 (Fri, 26 Feb 2021)

Log Message

Remove unnecessary whitespace (css1/text_properties/text_decoration.html)
https://bugs.webkit.org/show_bug.cgi?id=222446

Reviewed by Antti Koivisto.

Whitespace characters often end up as trailing content and legacy line layout does not trim them properly.
They cause unnecessary noise in renderer tree dumps when switching to modern line layout.

* css1/box_properties/border.html:
* css1/text_properties/text_decoration.html:
* platform/mac/css1/box_properties/border-expected.txt:
* platform/mac/css1/text_properties/text_decoration-expected.txt:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (273553 => 273554)


--- trunk/LayoutTests/ChangeLog	2021-02-26 14:07:17 UTC (rev 273553)
+++ trunk/LayoutTests/ChangeLog	2021-02-26 14:19:40 UTC (rev 273554)
@@ -1,3 +1,18 @@
+2021-02-26  Zalan Bujtas  <[email protected]>
+
+        Remove unnecessary whitespace (css1/text_properties/text_decoration.html)
+        https://bugs.webkit.org/show_bug.cgi?id=222446
+
+        Reviewed by Antti Koivisto.
+
+        Whitespace characters often end up as trailing content and legacy line layout does not trim them properly.
+        They cause unnecessary noise in renderer tree dumps when switching to modern line layout.
+
+        * css1/box_properties/border.html:
+        * css1/text_properties/text_decoration.html:
+        * platform/mac/css1/box_properties/border-expected.txt:
+        * platform/mac/css1/text_properties/text_decoration-expected.txt:
+
 2021-02-25  Lauro Moura  <[email protected]>
 
         [GLIB] Gardening a couple crashes

Modified: trunk/LayoutTests/css1/text_properties/text_decoration.html (273553 => 273554)


--- trunk/LayoutTests/css1/text_properties/text_decoration.html	2021-02-26 14:07:17 UTC (rev 273553)
+++ trunk/LayoutTests/css1/text_properties/text_decoration.html	2021-02-26 14:19:40 UTC (rev 273554)
@@ -58,7 +58,7 @@
 There should be nothing visible between this sentence and the one above (there is an empty paragraph element with class of seven).
 </P>
 <P class="one">
-Text decorations only apply to the text of an element, so the image at the end of this sentence should <EM>not</EM> be overlined: <IMG SRC="" class="two" alt="[Image]">.  The underline of the parent element should hold true beneath the image, however, since text-decoration 'spans' child elements.
+Text decorations only apply to the text of an element, so the image at the end of this sentence should <EM>not</EM> be overlined:<IMG SRC="" class="two" alt="[Image]">.  The underline of the parent element should hold true beneath the image, however, since text-decoration 'spans' child elements.
 </P>
 <P style="color: green;" class="one">
 The underlining <SPAN style="color: blue;">in this sentence</SPAN> should be green, no matter what the <SPAN style="color: black;">text color may be</SPAN>.

Modified: trunk/LayoutTests/platform/ios/css1/text_properties/text_decoration-expected.txt (273553 => 273554)


--- trunk/LayoutTests/platform/ios/css1/text_properties/text_decoration-expected.txt	2021-02-26 14:07:17 UTC (rev 273553)
+++ trunk/LayoutTests/platform/ios/css1/text_properties/text_decoration-expected.txt	2021-02-26 14:19:40 UTC (rev 273554)
@@ -65,11 +65,11 @@
         RenderInline {EM} at (0,0) size 21x19
           RenderText {#text} at (646,0) size 21x19
             text run at (646,0) width 21: "not"
-        RenderText {#text} at (666,0) size 93x19
-          text run at (666,0) width 93: " be overlined: "
-        RenderImage {IMG} at (758,0) size 16x15
-        RenderText {#text} at (773,0) size 778x59
-          text run at (773,0) width 5: "."
+        RenderText {#text} at (666,0) size 89x19
+          text run at (666,0) width 89: " be overlined:"
+        RenderImage {IMG} at (754,0) size 16x15
+        RenderText {#text} at (769,0) size 774x59
+          text run at (769,0) width 5: "."
           text run at (0,20) width 743: "The underline of the parent element should hold true beneath the image, however, since text-decoration 'spans' child"
           text run at (0,40) width 61: "elements."
       RenderBlock {P} at (0,583) size 784x20 [color=#008000]

Modified: trunk/LayoutTests/platform/mac/css1/text_properties/text_decoration-expected.txt (273553 => 273554)


--- trunk/LayoutTests/platform/mac/css1/text_properties/text_decoration-expected.txt	2021-02-26 14:07:17 UTC (rev 273553)
+++ trunk/LayoutTests/platform/mac/css1/text_properties/text_decoration-expected.txt	2021-02-26 14:19:40 UTC (rev 273554)
@@ -65,8 +65,8 @@
         RenderInline {EM} at (0,0) size 21x18
           RenderText {#text} at (646,0) size 21x18
             text run at (646,0) width 21: "not"
-        RenderText {#text} at (666,0) size 93x18
-          text run at (666,0) width 93: " be overlined: "
+        RenderText {#text} at (666,0) size 89x18
+          text run at (666,0) width 89: " be overlined:"
         RenderImage {IMG} at (0,18) size 15x15
         RenderText {#text} at (15,19) size 766x36
           text run at (15,19) width 8: ". "
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to