WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
265098
Elements should avoid floats even if they are as high as LayoutUnit::max()
https://bugs.webkit.org/show_bug.cgi?id=265098
Summary
Elements should avoid floats even if they are as high as LayoutUnit::max()
Ahmad Saleem
Reported
2023-11-18 17:22:56 PST
Hi Team, While going through Blink's commit, I came across another failing test but I don't know whether it is applicable as much since it seems to be edge case: Blink Commit:
https://chromium.googlesource.com/chromium/blink/+/af76e7423374640fa2aa5e5c550709e9a7fd39c4
Test Case:
https://jsfiddle.net/01r2ez5f/show
^ Chrome Canary 121 - Passes & Firefox show below error: Elements should avoid floats even if they are as high as LayoutUnit::max() a FAIL: Expected 33554431 for offsetTop, but got 27. <table data-offset-y="33554431" id="table"> <tbody><tr><td>a</td> </tr></tbody></table> While WebKit ToT running via 'run-webkit-tests' have following output: Elements should avoid floats even if they are as high as LayoutUnit::max() a FAIL: Expected 33554432 for offsetTop, but got 33554431. <table data-offset-y="33554432" id="table"> <tbody><tr><td>a</td> </tr></tbody></table> __ Since it is failing with 1 px, I just wanted to raise to get input whether it is worth fixing or we can just ignore. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2023-11-25 17:23:12 PST
<
rdar://problem/118795348
>
Ahmad Saleem
Comment 2
2024-01-04 20:16:09 PST
This 1px might have been fixed here:
https://github.com/WebKit/WebKit/commit/2afefe7449cd82d4b70c9ed85add5827f44e7278
Not confirmed, just speculating.
Ahmad Saleem
Comment 3
2025-12-15 19:29:52 PST
Pull request:
https://github.com/WebKit/WebKit/pull/55453
EWS
Comment 4
2025-12-17 06:30:52 PST
Committed
304586@main
(5b60e9ccada1): <
https://commits.webkit.org/304586@main
> Reviewed commits have been landed. Closing PR #55453 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug