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

Issue 618404 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

window.scrollTo fling canceling differs between main and impl thread

Project Member Reported by bokan@chromium.org, Jun 8 2016

Issue description

Version: 53.0.2761.0
OS: Android

What steps will reproduce the problem?
(1) Visit bokan.ca/clamp.html.
(2) Scroll down
(3) Fling up to the top of the page, when the page is scrolled to y=0 it'll window.scrollTo the document bottom.

What is the expected output? What do you see instead?

When flinging on the impl thread, calling window.scrollTo will cancel the fling so the page will remain at the bottom.

When run with --disable-threaded-scrolling, the fling will continue uninterrupted; the page continues to scroll up after window.scrollTo.

I'm not sure which of the two behaviors is correct/intended but we should make sure both paths match.

 
I think this should cancel the fling.
Labels: Hotlist-Polish
The original repro is no longer available. bokan@ can you upload the file elsewhere if you still have it.

I agree with #1. Another reason for pushing to have only a single place for fling to be generated.

Comment 3 by bokan@chromium.org, Jun 29 2017

Cc: sahel@chromium.org
+sahel@ who will likely be working on moving fling this quarter. The plan is to move fling to happen in the browser, but this situation would require Blink to somehow signal to the browser that we should cancel the fling. 

Here's an updated link: http://bokand.github.io/clamp.html
Project Member

Comment 4 by sheriffbot@chromium.org, Jul 2

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: -sahel@chromium.org
Owner: sahel@chromium.org
Status: Fixed (was: Untriaged)
This is fixed by browser-side fling.

Sign in to add a comment