New issue
Advanced search Search tips

Issue 878418 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 795386
Owner:
Closed: Dec 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Chrome
Pri: 2
Type: Bug



Sign in to add a comment

[SitePerProcess] Touch scrolling on iframe is too fast when zoomed in

Project Member Reported by bokan@chromium.org, Aug 28

Issue description

Chrome Version       : 70.0.3528.4
OS Version: 
URLs (if applicable) : http://csreis.github.io/tests/cross-site-iframe-simple.html

What steps will reproduce the problem?
1. Run chrome with --site-per-process
2. Visit http://csreis.github.io/tests/cross-site-iframe-simple.html
3. Pinch-zoom into the page
4. Scroll the iframe with a touchscreen

What is the expected result?
The iframe should stick to the finger

What happens instead of that?
The iframe scrolls faster than the finger

We're probably not scaling the gesture events as we should be.
 
Cc: creis@chromium.org
Labels: -Pri-3 Pri-2
Owner: chaopeng@chromium.org
Status: Assigned (was: Untriaged)
Chao, ptal when you have a chance.
By comparing with non-oopif iframe, it seems this issue is causing by oopif iframe don't know the scale factor but we need it to scale the scroll delta. I checked in LayerTreeHostImpl::ComputeScrollDelta. Maybe we should scale the delta before we route the event to oopif?

https://cs.chromium.org/chromium/src/cc/trees/layer_tree_host_impl.cc?rcl=ff6da4e16a772f2026a129de8cba0951c2cc85f6&l=3826
Cc: wjmaclean@chromium.org riajiang@chromium.org kenrb@chromium.org sahel@chromium.org alex...@chromium.org mcnee@chromium.org
Labels: OS-Android
I can also repro this on Android Canary 72.0.3608.5 on a Nexus 5 with chrome://flags/#enable-site-per-process turned on.  Note that you have to pinch-zoom pretty closely for the scroll to start getting out of sync with the gesture.  I think we'll want to fix this for shipping site isolation on Android.  Adding wjmaclean@, who's worked on zoom/propagating page scale factor/gestures, and a few other people who might have opinions here.  (Note that this happens ever after James's r605456 for page scale factor)
Labels: OS-Chrome
Also repros on a Pixelbook running M70 stable, where --site-per-process is on by default.
This appears to be a duplicate of issue 795386.
Mergedinto: 795386
Status: Duplicate (was: Assigned)

Sign in to add a comment