[Webkit-unassigned] [Bug 80010] New: CSS 2.1: height of inline boxes incorrectly excludes line gap

[bugzilla-daemon at webkit.org]

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

           Summary: CSS 2.1: height of inline boxes incorrectly excludes
                    line gap
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
               URL: test.csswg.org/suites/css2.1/nightly-unstable/html4/bl
                    ock-in-inline-001.htm
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: CSS
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: robert at webkit.org
                CC: hyatt at apple.com, jchaffraix at webkit.org,
                    simon.fraser at apple.com
            Blocks: 47141


This bug has been around forever and shows up in the linked test from the CSS 2.1 test suite - the red background from the table cell shouldn't show through. InlineBox::logicalHeight() uses FontMetrics::height() to establish the height of the line box, but that is only the height of the text and excludes the spacing below it - resulting in a line box 1 px shorter than it should be. So it looks like it should use FontMetrics::lineSpacing() instead.

Fixing this will require an enormous number of test results to be rebaselined so may not be worth the overhead.

What do you think?
#

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