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
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
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).
Sign in to add a comment