Bug 250390 - Fix scrollbar removal logic in FrameView::computeScrollbarExistence
Summary: Fix scrollbar removal logic in FrameView::computeScrollbarExistence
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Scrolling (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-01-10 08:56 PST by Ahmad Saleem
Modified: 2023-02-04 17:32 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 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!
Comment 2 Radar WebKit Bug Importer 2023-01-17 08:57:16 PST
<rdar://problem/104335690>
Comment 3 Ahmad Saleem 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!