Bug 250383

Summary:

Adjust text input scrollWidth and scrollHeight to include padding and any whitespace added by decorations

Product:

WebKit

Reporter:

Ahmad Saleem <ahmad.saleem792>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bfulgham, gpetrioli+webkit-bugzilla, karlcow, simon.fraser, webkit-bug-importer, zalan

Priority:

Keywords:

BrowserCompat, InRadar

Version:

Safari Technology Preview

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
GitHub Local Instance Patch	none

Description Ahmad Saleem 2023-01-10 07:03:21 PST

Hi Team,

While going through Blink's commit, I came across following failing test case:

Test Case - https://jsfiddle.net/2barv9Lu/

^ Safari 16.2 only passes two while Chrome Canary 111 only fail one with fraction issue while Firefox Nightly 110 passes all.

Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=186123

WebKit Source - https://github.com/WebKit/WebKit/blob/28de25b653f64ca3fc8e87fbf1d1a963329a5342/Source/WebCore/rendering/RenderTextControlSingleLine.cpp#L367

Just wanted to raise so we can track and fixes these issues.

Thanks!

Comment 1 Radar WebKit Bug Importer 2023-01-17 07:04:15 PST

<rdar://problem/104332108>

Comment 2 Ahmad Saleem 2023-03-01 08:29:33 PST

Created attachment 465240 [details]
GitHub Local Instance Patch

Using the attached in WebKit ToT will fix this test case. Just tested locally.

Comment 3 Ahmad Saleem 2023-03-01 09:59:33 PST

*** Bug 201453 has been marked as a duplicate of this bug. ***

Comment 4 EWS 2023-03-03 00:56:39 PST

Committed 261121@main (393d4a1bb437): <https://commits.webkit.org/261121@main>

Reviewed commits have been landed. Closing PR #10875 and removing active labels.