New issue
Advanced search Search tips

Issue 725967 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Mac touchpad scrolling interacts strangely with the shift key

Project Member Reported by tdres...@chromium.org, May 24 2017

Issue description

To repro: 
- Fling on a long page
- Tap shift

While shift is held, we don't scroll. When it's released, we do.
I'd expect shift to either cancel the fling, or not stop it at all.

Another repro:
- Hold shift
- Touchpad scroll slowly

I'd expect holding shift to either not impact things at all, or to completely prevent scrolling. Instead, it appears to prevent some deltas, but not all.
 

Comment 1 by meh...@chromium.org, May 24 2017

Components: Blink>Scroll

Comment 2 by meh...@chromium.org, May 24 2017

Labels: -Type-Bug Needs-Bisect Type-Bug-Regression
This seems to be broken at some point. For example, it works as expected in the following build: https://commondatastorage.googleapis.com/chromium-browser-snapshots/index.html?prefix=Mac/362232/

May be a bisect can help?

Cc: dtapu...@chromium.org
It's broken in 58.0.3029.96 and 60.0.3100.0.

What version did you find it not broken in?
It might have been broken by wheel gestures?

Comment 4 by meh...@chromium.org, May 24 2017

I picked randomly an older build from https://commondatastorage.googleapis.com/chromium-browser-snapshots/index.html?prefix=Mac/

For example build 362232:

Chromium	49.0.2578.0 (Entwickler-Build) (64-Bit)
Überarbeitung	8dfd114ecca6c1b2c2f28885e6d7173ddad7c1fe-refs/heads/master@{#362232}
Betriebssystem	Mac OS X 
Blink	537.36 (@8dfd114ecca6c1b2c2f28885e6d7173ddad7c1fe)
JavaScript	V8 4.9.103


Comment 5 by meh...@chromium.org, May 24 2017

Labels: -Needs-Bisect
Hi again,

I did a manual bisect, and I found the following regression range:

https://chromium.googlesource.com/chromium/src/+log/052c36a69d1d01b13b965ead4613c96d1806b835..3598c8bc9c56409f7c239c377fe6e42786d6aaa0

This CL could be the reason: https://codereview.chromium.org/2359463002
Cc: -dtapu...@chromium.org tdres...@chromium.org
Owner: dtapu...@chromium.org
Status: Assigned (was: Available)
Yup, that's almost certainly it.
Over to Dave.

Sign in to add a comment