Bug 119321
| Summary: | Correct overflow propagation in BTT and RTL writing-modes | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED CONFIGURATION CHANGED | ||
| Severity: | Normal | CC: | ahmad.saleem792, ap, bdakin, bfulgham, dino, enrica, hyatt, robert, rreno, simon.fraser, thorton, zalan |
| Priority: | P2 | Keywords: | BlinkMergeCandidate |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ryosuke Niwa
Consider merging https://chromium.googlesource.com/chromium/blink/+/5ffebc0f0a12542f4dd9824042904642528ad665
Overflow rectangles are not quite physical, not quite logical. This
means that we cannot use clientBoxRect() directly to represent a
rectangle that expresses exactly no overflow. This rectangle is the
padding box (relative to the border box) in vertical-lr and
horizontal-tb, but the block-direction borders need to be flipped in
vertical-rl and horizontal-bt.
fast/multicol/vertical-rl/rules-with-border-before.html now needs a
rebaseline, because it now renders differently, but correctly.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ahmad Saleem
I changed the test cases from Chromium patch:
Test Case - overflow-btt-border-after.html - https://jsfiddle.net/k9e2qhz6/ (Remove -webkit- prefix)
Test Case - overflow-rtl-border-after.html - https://jsfiddle.net/hcpjde3z/
Both show "PASS" for Safari 15.6.1, Chrome Canary 106 and Firefox Nightly 105. Is something else needed? Thanks!