Bug 122150 – REGRESSION (r156586): fast/css3-text/css3-text-indent/getComputedStyle/getComputedStyle-text-indent.html is failing

• Component: CSS
• Status: RESOLVED CONFIGURATION CHANGED

[Zan Dobersek]

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'

[Brent Fulgham]

This code has been significantly refactored since this patch was proposed. There doesn't seem to be any action we can take here.

[Radar WebKit Bug Importer]

<rdar://problem/97094653>