Bug 122150 - REGRESSION (r156586): fast/css3-text/css3-text-indent/getComputedStyle/getComputedStyle-text-indent.html is failing
Summary: REGRESSION (r156586): fast/css3-text/css3-text-indent/getComputedStyle/getCom...
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk, InRadar, LayoutTestFailure, Regression
Depends on:
Blocks:
 
Reported: 2013-10-01 02:46 PDT by Zan Dobersek
Modified: 2022-07-15 14:31 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Zan Dobersek 2013-10-01 02:46:15 PDT
The fast/css3-text/css3-text-indent/getComputedStyle/getComputedStyle-text-indent.html layout test started failing after r156586.
http://trac.webkit.org/changeset/156586

At the moment it only fails on GTK since that port seems to be the only one both enabling the feature and running the related tests.
http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fcss3-text%2Fcss3-text-indent%2FgetComputedStyle%2FgetComputedStyle-text-indent.html

Diff:
--- /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fast/css3-text/css3-text-indent/getComputedStyle/getComputedStyle-text-indent-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fast/css3-text/css3-text-indent/getComputedStyle/getComputedStyle-text-indent-actual.txt
@@ -26,8 +26,8 @@
 PASS computedStyle.getPropertyValue('text-indent') is '50%'
 
 Value 'calc(10px + 20px)':
-PASS e.style.textIndent is 'calc(10px + 20px)'
-PASS e.style.getPropertyValue('text-indent') is 'calc(10px + 20px)'
+FAIL e.style.textIndent should be calc(10px + 20px). Was calc(30px).
+FAIL e.style.getPropertyValue('text-indent') should be calc(10px + 20px). Was calc(30px).
 PASS computedStyle.textIndent is '30px'
 PASS computedStyle.getPropertyValue('text-indent') is '30px'
 
@@ -68,14 +68,14 @@
 PASS computedStyle.getPropertyValue('text-indent') is '30% -webkit-each-line'
 
 Value 'calc(10px + 20px) -webkit-each-line':
-PASS e.style.textIndent is 'calc(10px + 20px) -webkit-each-line'
-PASS e.style.getPropertyValue('text-indent') is 'calc(10px + 20px) -webkit-each-line'
+FAIL e.style.textIndent should be calc(10px + 20px) -webkit-each-line. Was calc(30px) -webkit-each-line.
+FAIL e.style.getPropertyValue('text-indent') should be calc(10px + 20px) -webkit-each-line. Was calc(30px) -webkit-each-line.
 PASS computedStyle.textIndent is '30px -webkit-each-line'
 PASS computedStyle.getPropertyValue('text-indent') is '30px -webkit-each-line'
 
 Value '-webkit-each-line calc(10px + 20px)':
-PASS e.style.textIndent is '-webkit-each-line calc(10px + 20px)'
-PASS e.style.getPropertyValue('text-indent') is '-webkit-each-line calc(10px + 20px)'
+FAIL e.style.textIndent should be -webkit-each-line calc(10px + 20px). Was -webkit-each-line calc(30px).
+FAIL e.style.getPropertyValue('text-indent') should be -webkit-each-line calc(10px + 20px). Was -webkit-each-line calc(30px).
 PASS computedStyle.textIndent is '30px -webkit-each-line'
 PASS computedStyle.getPropertyValue('text-indent') is '30px -webkit-each-line'
 
@@ -116,14 +116,14 @@
 PASS computedStyle.getPropertyValue('text-indent') is '30% -webkit-hanging'
 
 Value 'calc(10px + 20px) -webkit-hanging':
-PASS e.style.textIndent is 'calc(10px + 20px) -webkit-hanging'
-PASS e.style.getPropertyValue('text-indent') is 'calc(10px + 20px) -webkit-hanging'
+FAIL e.style.textIndent should be calc(10px + 20px) -webkit-hanging. Was calc(30px) -webkit-hanging.
+FAIL e.style.getPropertyValue('text-indent') should be calc(10px + 20px) -webkit-hanging. Was calc(30px) -webkit-hanging.
 PASS computedStyle.textIndent is '30px -webkit-hanging'
 PASS computedStyle.getPropertyValue('text-indent') is '30px -webkit-hanging'
 
 Value '-webkit-hanging calc(10px + 20px)':
-PASS e.style.textIndent is '-webkit-hanging calc(10px + 20px)'
-PASS e.style.getPropertyValue('text-indent') is '-webkit-hanging calc(10px + 20px)'
+FAIL e.style.textIndent should be -webkit-hanging calc(10px + 20px). Was -webkit-hanging calc(30px).
+FAIL e.style.getPropertyValue('text-indent') should be -webkit-hanging calc(10px + 20px). Was -webkit-hanging calc(30px).
 PASS computedStyle.textIndent is '30px -webkit-hanging'
 PASS computedStyle.getPropertyValue('text-indent') is '30px -webkit-hanging'
Comment 1 Brent Fulgham 2022-07-15 14:30:32 PDT
This code has been significantly refactored since this patch was proposed. There doesn't seem to be any action we can take here.
Comment 2 Radar WebKit Bug Importer 2022-07-15 14:31:16 PDT
<rdar://problem/97094653>