[Webkit-unassigned] [Bug 79526] Incorrect handling of sizes in "em" when first-line changes font size

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=79526





--- Comment #6 from Eric Seidel <eric at webkit.org>  2012-03-07 15:15:05 PST ---
(From update of attachment 130441)
View in context: https://bugs.webkit.org/attachment.cgi?id=130441&action=review

> Source/WebCore/ChangeLog:11
> +        When pseudo class first-line changes the font size, the "em" unit is handled incorrectly as the element's original
> +        size (the size of the rest of the paragraph) rather than the font-size of the first-line of the element. This was
> +        corrected by checking if the InlineFlowBox was the first line using the existing InlineFlowBox::isFirstLineStyle()
> +        function.

You should include commentary here on how this does or does not match other browsers behavior, and what the CSS spec says on the topic.

> Source/WebCore/rendering/RenderBoxModelObject.cpp:2586
> +    getBorderEdgeInfo(edges, this->style());

Why this->style() here?  I don't se a "style" local to conflict.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.