WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
250390
Fix scrollbar removal logic in FrameView::computeScrollbarExistence
https://bugs.webkit.org/show_bug.cgi?id=250390
Summary
Fix scrollbar removal logic in FrameView::computeScrollbarExistence
Ahmad Saleem
Reported
2023-01-10 08:56:28 PST
Hi Team, While going through Blink's commit, I came across another failing test case: Test Case (from Chrome Monorail) -
https://jsfiddle.net/m6aLLhur/
^ Try to select "scrollbar" and make it drag and the thumb / scrollbar will jump to bottom (in Safari 16.2) while in Chrome Canary 111 - you can control it. Blink Commit -
https://src.chromium.org/viewvc/blink?view=revision&revision=185555
WebKit Source -
https://searchfox.org/wubkat/source/Source/WebKitLegacy/mac/WebView/WebDynamicScrollBarsView.mm#357
Just wanted to raise so we can also fix this bug. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
Ahmad Saleem
Comment 1
2023-01-10 08:57:01 PST
OR -
https://searchfox.org/wubkat/source/Source/WebCore/editing/FrameSelection.cpp#693
Radar WebKit Bug Importer
Comment 2
2023-01-17 08:57:16 PST
<
rdar://problem/104335690
>
Ahmad Saleem
Comment 3
2023-02-04 17:32:54 PST
(In reply to Ahmad Saleem from
comment #0
)
> Hi Team, > > While going through Blink's commit, I came across another failing test case: > > Test Case (from Chrome Monorail) -
https://jsfiddle.net/m6aLLhur/
> > ^ Try to select "scrollbar" and make it drag and the thumb / scrollbar will > jump to bottom (in Safari 16.2) while in Chrome Canary 111 - you can control > it. > > Blink Commit - >
https://src.chromium.org/viewvc/blink?view=revision&revision=185555
> > WebKit Source - >
https://searchfox.org/wubkat/source/Source/WebKitLegacy/mac/WebView/
> WebDynamicScrollBarsView.mm#357 > > Just wanted to raise so we can also fix this bug. > > Thanks!
It seems to be here:
https://searchfox.org/wubkat/source/Source/WebCore/platform/ScrollView.cpp#670
I removed this on local test branch but the scrollbar jumping on the bottom issue still persist. It could be there is some pre-requisite etc. Thanks!
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