New issue
Advanced search Search tips

Issue 626277 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Scroll position overrides are not enforced for pinch gestures

Project Member Reported by eseckler@chromium.org, Jul 7 2016

Issue description

We enforce scroll position overrides only within Blink in the main thread (via ScrollAndScaleEmulator). Therefore, we disable threaded scrolling to prevent scrolling on the compositor thread that otherwise would ignore the override.

However, disabling threaded scrolling does not prevent handling of pinch-zoom gestures on the compositor thread. As we enforce the scale override through min/max page scale settings, which are pushed to the LayerTreeHost and thus reflected in the compositor thread, this only affects scroll position changes in the context of pinch-anchoring, which ignore the scroll position override. When this happens, the change in the scroll position is later reverted on the main thread through clamping the updated position, but it is possible to visually observe intermittent jumps in the scroll position.
 

Comment 1 by bokan@chromium.org, Jul 7 2016

IMO, the correct way to fix this is to send pinch gestures to the main thread if the layer should scroll on the main thread. This code predates having pinch handling on the main thread so it should be possible now, though the main thread code was written with Mac trackpad zoom in mind so there might be some subtleties.
Status: WontFix (was: Assigned)
With the revised screenshotting approach, this is now obsolete.

Sign in to add a comment