New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 4 users

Issue metadata

Status: Available
Owner: ----
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug

Blocked on:
issue 175502

Sign in to add a comment

Windows scroll vertically by page feature doesn't take scrollbar visibility into account

Project Member Reported by, Apr 29 2013 Back to list

Issue description

On windows, mouse wheels can scroll by a fixed delta or scroll by page.  Scroll by page requires special handling since the amount to scroll depends on the content.  There is basic support for this, but it fails on content that is vertically scrollable but doesn't have a compositor scrollbar.

For now, these scrolls just go to the main thread.  It'd be better to handle them on the compositor thread.

Comment 1 by, Apr 29 2013

The specific bug is here:

    if (!layer_impl->vertical_scrollbar_layer())

Without that line, we incorrectly scroll on pages that don't have a vertical scrollbar (say they have overflow-y hidden).  With that line, we incorrectly fail to scroll on pages that do have a vertical scrollbar but that scrollbar is not registered with the compositor - i.e. it's a CSS styled one or something else the compositor does not understand.
Project Member

Comment 3 by, Aug 26 2016

Labels: Hotlist-Recharge-Cold
Status: Untriaged
This issue has been available for more than 365 days, and should be re-evaluated. Please re-triage this issue.
The Hotlist-Recharge-Cold label is applied for tracking purposes, and should not be removed after re-triaging the issue.

For more details visit - Your friendly Sheriffbot

Comment 4 by, Jan 10 2017

Components: Blink>Scroll
I am not sure if this bug is still relevant.

We don't check scroll bar existence anymore rather we seem to depend on user_scrollable_horizontal and user_scrollable_vertical [1] which are set by blink based on the overflow property and do not depend on existence of scrollbar.


Comment 6 by, Jan 12 2017

Labels: -Pri-2 Hotlist-Input-Dev Pri-3
Status: Available
It is since we're still punting wheels that scroll by page to the main thread:

Though, as you mention, the reasons the bug was intractable no longer applies. We should try removing this and handling the wheel event on the compositor (assuming the scroll by page logic still exists in CC).
Triage check-in, still an issue in 2018-Q1. The line mentioned above is now at That's in InputHandlerProxy::HandleGestureScrollBegin

Sign in to add a comment