Bug 137108 - [freetype] Fix support for the initial-letter CSS property to first-letter
Summary: [freetype] Fix support for the initial-letter CSS property to first-letter
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Lorenzo Tilve
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-25 00:43 PDT by Lorenzo Tilve
Modified: 2014-09-26 01:40 PDT (History)
5 users (show)

See Also:


Attachments
Patch (14.70 KB, patch)
2014-09-25 01:01 PDT, Lorenzo Tilve
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lorenzo Tilve 2014-09-25 00:43:44 PDT
The support for these properties added at ​http://webkit.org/b/136484 is not working on WebKitGTK+. Also, the LayoutTests introduced at that patch have missing expectations for GTK.
Comment 1 Lorenzo Tilve 2014-09-25 01:01:05 PDT
Created attachment 238646 [details]
Patch
Comment 2 Carlos Garcia Campos 2014-09-25 02:00:00 PDT
Is freetype only used by GTK+ port?
Comment 3 Lorenzo Tilve 2014-09-25 02:13:59 PDT
(In reply to comment #2)
> Is freetype only used by GTK+ port?
Comment 4 Lorenzo Tilve 2014-09-25 02:16:04 PDT
(In reply to comment #2)
> Is freetype only used by GTK+ port?

It is being also used on EFL, so renaming the bug from [GTK] to [freetype]. Thanks for the remark.
Comment 5 Myles C. Maxfield 2014-09-25 08:52:26 PDT
Comment on attachment 238646 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=238646&action=review

> LayoutTests/ChangeLog:15
> +        * platform/gtk/fast/css-generated-content/initial-letter-basic-expected.txt: Added.
> +        * platform/gtk/fast/css-generated-content/initial-letter-border-padding-expected.txt: Added.
> +        * platform/gtk/fast/css-generated-content/initial-letter-clearance-expected.txt: Added.
> +        * platform/gtk/fast/css-generated-content/initial-letter-descender-expected.txt: Added.
> +        * platform/gtk/fast/css-generated-content/initial-letter-raised-expected.txt: Added.
> +        * platform/gtk/fast/css-generated-content/initial-letter-sunken-expected.txt: Added.

This is confusing. Are these tests currently skipped, which is why they didn't have -expected.txt's?

Is it possible to use reftests instead so that they can be platform non-specific?
Comment 6 Lorenzo Tilve 2014-09-25 09:39:57 PDT
(In reply to comment #5)

> > LayoutTests/ChangeLog:15
> > +        * platform/gtk/fast/css-generated-content/initial-letter-basic-expected.txt: Added.
> > +        * platform/gtk/fast/css-generated-content/initial-letter-border-padding-expected.txt: Added.
> > +        * platform/gtk/fast/css-generated-content/initial-letter-clearance-expected.txt: Added.
> > +        * platform/gtk/fast/css-generated-content/initial-letter-descender-expected.txt: Added.
> > +        * platform/gtk/fast/css-generated-content/initial-letter-raised-expected.txt: Added.
> > +        * platform/gtk/fast/css-generated-content/initial-letter-sunken-expected.txt: Added.
> 
> This is confusing. Are these tests currently skipped, which is why they didn't have -expected.txt's?

Those -expected.txt's were missing (they were not skipped on any TestExpectations either):
https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/2994/steps/layout-test/logs/stdio

Regressions: Unexpected missing results (6)
  fast/css-generated-content/initial-letter-basic.html [ Missing ]
  fast/css-generated-content/initial-letter-border-padding.html [ Missing ]
  fast/css-generated-content/initial-letter-clearance.html [ Missing ]
  fast/css-generated-content/initial-letter-descender.html [ Missing ]
  fast/css-generated-content/initial-letter-raised.html [ Missing ]
  fast/css-generated-content/initial-letter-sunken.html [ Missing ]

When the tests were introduced at r173217 <http://trac.webkit.org/changeset/173217>, only the mac expectations were added, so that's why I'm adding those missing ones now that it would be also supported in GTK+ too.
Comment 7 Myles C. Maxfield 2014-09-25 10:33:01 PDT
unofficial r=me
Comment 8 WebKit Commit Bot 2014-09-26 01:40:09 PDT
Comment on attachment 238646 [details]
Patch

Clearing flags on attachment: 238646

Committed r174002: <http://trac.webkit.org/changeset/174002>
Comment 9 WebKit Commit Bot 2014-09-26 01:40:13 PDT
All reviewed patches have been landed.  Closing bug.