Bug 250390
| Summary: | Fix scrollbar removal logic in FrameView::computeScrollbarExistence | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | Scrolling | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | simon.fraser, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
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
OR - https://searchfox.org/wubkat/source/Source/WebCore/editing/FrameSelection.cpp#693
Radar WebKit Bug Importer
<rdar://problem/104335690>
Ahmad Saleem
(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!