Bug 250209
| Summary: | Fix layoutOverflowRect in case of explicit 'overflow: scroll', but no overflow content | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | bfulgham, simon.fraser, webkit-bug-importer, zalan |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
While going through Blink's commit, came across another which can be merged:
Blink Commit - https://src.chromium.org/viewvc/blink?revision=174808&view=revision
WebKit Source - https://github.com/WebKit/WebKit/blob/fa3e08712e3219aed72097238ae77493974a5092/Source/WebCore/rendering/LegacyLineLayout.cpp#L678, while we don't need other change, since it is already done here: https://github.com/WebKit/WebKit/blob/fa3e08712e3219aed72097238ae77493974a5092/Source/WebCore/rendering/RenderBox.cpp#L5061
if (style().shouldPlaceBlockDirectionScrollbarOnLeft())
logicalLeft += verticalScrollbarWidth();
___
I just wanted to raise and get input before doing PR since this is in Legacy Line Layout rather than IFC, which might have already progressed this.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/104224627>