Bug 95848 – [css3-text] Add parsing support for -webkit-text-decoration-skip

• Component: CSS
• Status: RESOLVED DUPLICATE

[Bruno Abinader (history only)]

This bug intends to add parsing support for "text-decoration-skip" CSS3 property. Rendering support is going to be implemented on another bug. This is a sub-task for bug 92801.

[Bruno Abinader (history only)]

Created attachment 162252 [details]
Patch

Proposed patch.

[Bruno Abinader (history only)]

Created attachment 162254 [details]
Patch (EWS run only)

[Bruno Abinader (history only)]

Created attachment 170142 [details]
Patch

Updates on namespace (now CSS3_TEXT), layout test directories and StyleBuilder handler interface.

[Bruno Abinader (history only)]

Created attachment 170145 [details]
Patch (EWS only)

[Build Bot]

Comment on attachment 170145 [details]
Patch (EWS only)

Attachment 170145 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14483956

[Build Bot]

Comment on attachment 170145 [details]
Patch (EWS only)

Attachment 170145 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14491877

[kov's GTK+ EWS bot]

Comment on attachment 170145 [details]
Patch (EWS only)

Attachment 170145 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/14524022

[WebKit Review Bot]

Comment on attachment 170145 [details]
Patch (EWS only)

Attachment 170145 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/14500616

New failing tests:
fast/css3-text/css3-text-decoration/text-decoration-style.html
fast/css3-text/css3-text-decoration/repaint/repaint-text-decoration-style.html
fast/css3-text/css3-text-decoration/getComputedStyle/getComputedStyle-text-decoration-skip.html

[Bruno Abinader (history only)]

Created attachment 170925 [details]
Patch

Fixed layout test internal paths after directory changes.

[Bruno Abinader (history only)]

Created attachment 170973 [details]
Patch (EWS only)

More info on failing layout tests in bug 100546 "[css3-text] Provide pixel results for text decoration style layout tests".

[Build Bot]

Comment on attachment 170973 [details]
Patch (EWS only)

Attachment 170973 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14593766

[WebKit Review Bot]

Comment on attachment 170973 [details]
Patch (EWS only)

Attachment 170973 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/14593790

New failing tests:
fast/css3-text/css3-text-decoration/text-decoration-style.html
fast/css3-text/css3-text-decoration/repaint/repaint-text-decoration-style.html

[kov's GTK+ EWS bot]

Comment on attachment 170973 [details]
Patch (EWS only)

Attachment 170973 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/14612130

[Bruno Abinader (history only)]

Created attachment 179069 [details]
Patch

Rebased after text-align-last changes.

[fantasai]

Please don't land any parsing support for properties for which there is no actual support. This is counter to the way CSS parsing is supposed to work.
  http://www.w3.org/TR/CSS/#partial

[Bruno Abinader (history only)]

(In reply to comment #15)
> Please don't land any parsing support for properties for which there is no actual support. This is counter to the way CSS parsing is supposed to work.
>   http://www.w3.org/TR/CSS/#partial

Rendering support should be handled in bug 95856 (as refered in "blocks" section), though there's no actually implementation yet. Unfortunately I do not have resources to implement it, so indeed, the patch proposed on this bug shall not land before there's a corresponding implementation on bug 95856.

[Julien Chaffraix]

(In reply to comment #15)
> Please don't land any parsing support for properties for which there is no actual support. This is counter to the way CSS parsing is supposed to work.
>   http://www.w3.org/TR/CSS/#partial

You are missing the point here. All new code is landed behind a compile flag that is *disabled* by default (ie not exposed to the web). So yes, we are breaking feature detection but on a feature that is not shipped. We also make sure we don't toggle the flag until all the bits are done.

AFAICT this is matching what you said and I like this practice because it enables better quality reviews by splitting massive patches into manageable pieces.

[Anders Carlsson]

Comment on attachment 179069 [details]
Patch

Clearing review flag on patches from before 2014. If this patch is still relevant, please reset the r? flag.

[Tim Nguyen (:ntim)]

*** This bug has been marked as a duplicate of bug 230244 ***