Issue metadata
Sign in to add a comment
|
Regression: Unable to scroll the page via mouse wheel on hovering the mouse over scrollbar. |
||||||||||||||||||||
Issue descriptionChrome Version : 51.0.2674.0 (Official Build) OS Version : Ubuntu 14.04 What steps will reproduce the problem? (1)Launch chrome and open any page with scrollbar. (2)Hover the mouse over scrollbar and try to scroll the page through mouse wheel What is the expected result? Should be able to scroll the page. What happens instead? The page does not scroll. This is a regression issue broken in M-51.
,
Mar 11 2016
Good Build: 51.0.2673.0 Bad Build : 51.0.2674.0 CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/cb9d633e2f1f1f13881899fa92f356bf8cd79e02..fd7ede807eb0e64820c805e8d5d0caf7ba9c5516 Suspecting https://codereview.chromium.org/1752043002 from changelog @bokan: Please look into the issue.
,
Mar 11 2016
Yes, this was caused by that CL. I'll submit a fix shortly.
,
Mar 14 2016
,
Mar 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2931e932cacf7c7e7ca1629971c169b5b52c18c0 commit 2931e932cacf7c7e7ca1629971c169b5b52c18c0 Author: bokan <bokan@chromium.org> Date: Wed Mar 16 20:51:33 2016 Fix wheel scrolling over scrollbar not scrolling page. I introduced this bug in my refactoring in r380403. The fix is to simply use the document node as the element to scroll if the hit test doesn't return a node but does hit a scrollbar. BUG= 594074 Review URL: https://codereview.chromium.org/1804063002 Cr-Commit-Position: refs/heads/master@{#381533} [add] https://crrev.com/2931e932cacf7c7e7ca1629971c169b5b52c18c0/third_party/WebKit/LayoutTests/fast/events/wheel-in-scrollbar-expected.txt [add] https://crrev.com/2931e932cacf7c7e7ca1629971c169b5b52c18c0/third_party/WebKit/LayoutTests/fast/events/wheel-in-scrollbar.html [modify] https://crrev.com/2931e932cacf7c7e7ca1629971c169b5b52c18c0/third_party/WebKit/Source/core/input/EventHandler.cpp
,
Mar 16 2016
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by kavvaru@chromium.org
, Mar 11 2016Status: Untriaged (was: Unconfirmed)