Bug 279699 – Nested inlines' vertical alignment is wrong when line-fit-edge is set

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[alan]

<rdar://135962993>

[alan]

Created attachment 472552 [details]
Patch

[Tim Nguyen (:ntim)]

Comment on attachment 472552 [details]
Patch

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

> LayoutTests/fast/inline/nested-inline-boxes-with-line-fit-edge.html:8
> +  line-fit-edge: cap alphabetic;

would be good to have WPT for line-fit-edge

[alan]

Created attachment 472557 [details]
Patch

[alan]

(the integral change (bug 279717) needs to land first to make this test pass)

[alan]

(In reply to Tim Nguyen (:ntim) from comment #2)
> Comment on attachment 472552 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=472552&action=review
> 
> > LayoutTests/fast/inline/nested-inline-boxes-with-line-fit-edge.html:8
> > +  line-fit-edge: cap alphabetic;
> 
> would be good to have WPT for line-fit-edge
Will talk to Elika about line-fit-edge coverage for WPT.

[alan]

Created attachment 472564 [details]
[fast-cq]Patch

[EWS]

Committed 283682@main (d54de3189d86): <https://commits.webkit.org/283682@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 472564 [details].

[EWS]

Committed 283286.92@safari-7620-branch (26fae845736e): <https://commits.webkit.org/283286.92@safari-7620-branch>

Reviewed commits have been landed. Closing PR #1830 and removing active labels.