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

Issue 835314 link

Starred by 9 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Auto-scroll bug while synchronous mouse scrolling

Reported by robinroeper10@gmail.com, Apr 20 2018

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.117 Safari/537.36

Steps to reproduce the problem:
1. Scroll once
2. During scrolling, middle click anywhere to activate auto-scrolling

What is the expected behavior?
Auto-scroll on website.

What went wrong?
Auto-scroll is "activated" but doesn't work properly.

Did this work before? N/A 

Chrome version: 66.0.3359.117  Channel: stable
OS Version: 10.0
Flash Version: 

This Bug/issue was originally filed by a user on the Google Chrome Product Forum (https://productforums.google.com/forum/?utm_medium=email&utm_source=footer#!msg/chrome/XZylVBBqDM8/LS-EMWgfAQAJ) 

It is also reproducible on - User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.117 Safari/537.36
 
case-XZylVBBqDM8.gif
4.5 MB View Download
Labels: Needs-Triage-M66
Cc: susan.boorgula@chromium.org
Components: Blink>Scroll
Labels: Triaged-ET Needs-Feedback
robinroeper10@ Thanks for the issue.

Tested this issue on Windows 10 on the reported version 66.0.3359.117 and the latest Canary 68.0.3403.0 and unable to reproduce the issue by following the given steps above.

Scrolled for a while on a page and middle clicked on the mouse to activate the auto-scrolling. Cannot observe any issues while auto-scrolling through a website.
Attached is the screen cast for reference.

Request you to retry the issue on a new chrome profile without any flags/extensions and update the thread with the observations.

Thanks..
835314.webm
8.1 MB View Download
Labels: Hotlist-ConOps
Hey all,

It looks like we may have a couple more reports as well of similar behavior:
- https://productforums.google.com/forum/#!msg/chrome/FWdpwVX4V7I/vTKqL2xvBAAJ
- https://listnr.corp.google.com/report/85272772133
- https://listnr.corp.google.com/report/85335290588

Thanks!

Comment 4 by bokan@chromium.org, Apr 26 2018

Cc: bokan@chromium.org
Owner: sahel@chromium.org
Status: Assigned (was: Unconfirmed)
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/71f099520097b050b6c9363391fde6c758e0aa9b

commit 71f099520097b050b6c9363391fde6c758e0aa9b
Author: sahel <sahel@chromium.org>
Date: Fri Apr 27 20:30:50 2018

Autoscroll has priority over wheel scrolling.

On autoscroll start the mouse_wheel_phase_handler_ ends the current
scrolling sequence(if such exists). While Autoscrolling wheel events
in the mouse wheel event queue are ignored and once autoscrolling ends
the mouse_wheel_event_queue resumes processing wheel events normally.

This change is identical to giving priority to touchscreen scrolling
over wheel scrolling and it is done to make sure that attempting to
wheel scroll during a middle click autoscroll doesn't cause unwanted GSB
/GSE generation.

Bug:  835314 
Test: *.TimerBasedLatchingBreaksWithAutoscrollStart
Change-Id: Id568ee2676a6dafc2705c144ea4d92d46311b788
Reviewed-on: https://chromium-review.googlesource.com/1011457
Commit-Queue: Sahel Sharifymoghaddam <sahel@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554496}
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/input/mouse_wheel_event_queue.cc
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_impl.cc
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_android.h
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_aura.cc
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_aura.h
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_base.cc
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_base.h
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_mac.h
[modify] https://crrev.com/71f099520097b050b6c9363391fde6c758e0aa9b/content/browser/renderer_host/render_widget_host_view_mac.mm

Comment 6 by sahel@chromium.org, May 2 2018

robinroeper10@ could you please check the latest Chrome Canary to see if the bug still happens there or not?

The fix is first landed in 68.0.3412.0

Comment 7 Deleted

Comment 8 by sahel@chromium.org, May 4 2018

Status: Fixed (was: Assigned)
Great, thanks!
sahel@ Confirm that the bug has been fixed.

Tested on Chrome Canary Version 68.0.3416.0

Comment 10 Deleted

Comment 11 by sahel@chromium.org, May 10 2018

Deleted comment #10 since it was wrong duplication.
Cc: krajshree@chromium.org sahel@chromium.org
 Issue 811156  has been merged into this issue.

Sign in to add a comment