Bug 111606 - border-fit should happen at layout time rather than paint time
Summary: border-fit should happen at layout time rather than paint time
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dave Hyatt
Keywords: InRadar
Depends on:
Reported: 2013-03-06 11:30 PST by Dave Hyatt
Modified: 2021-12-24 13:25 PST (History)
12 users (show)

See Also:

Patch (13.67 KB, patch)
2013-03-07 09:52 PST, Dave Hyatt
koivisto: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Hyatt 2013-03-06 11:30:57 PST
border-fit-adjust should happen at layout time rather than paint time. When the border-fit-adjusted box is right-aligned but contains left-aligned text, the paint time hack chops off the right side of the box correctly but then doesn't shift the smaller box to the right. We basically need to genuinely shrink the box during layout so that cases like this work.
Comment 1 Dave Hyatt 2013-03-07 09:52:27 PST
Created attachment 192024 [details]
Comment 2 Antti Koivisto 2013-03-07 10:35:06 PST
Comment on attachment 192024 [details]

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


> Source/WebCore/rendering/RenderBlock.cpp:6885
> +    layoutBlock(false);

Recursion :(
Comment 3 Dave Hyatt 2013-03-07 10:44:21 PST
Fixed in r145100.
Comment 4 Roger Fong 2013-03-07 14:18:46 PST
Hi, this broke

on mac and windows.

Comment 5 Dean Jackson 2013-03-07 16:29:18 PST
Test Expectations added to get our EWS going again.
Reopening bug until the tests can be reenabled.
Comment 6 Dean Jackson 2013-03-07 16:29:39 PST
Comment 7 Jon Lee 2013-03-13 21:36:56 PDT
r145139 fixed one of the tests. See bug 111776.
Comment 8 Jon Lee 2013-03-13 21:40:02 PDT
Comment 9 Brent Fulgham 2015-10-12 12:03:56 PDT
I no longer see border-fit-lines.html failing on Windows.
Comment 10 Tim Nguyen (:ntim) 2021-12-24 13:25:09 PST
Bug 229564 removed -webkit-border-fit.