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

Issue 652000 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

The app chooser on google.com defers to main thread scrolling due to NonFastScrollableRegion.

Project Member Reported by khushals...@chromium.org, Sep 30 2016

Issue description

In the blimp world, we can not really afford sending scroll/fling gestures to the engine (main thread). This will probably be a long running bug to understand the cases where that happens and what the fallback should be.

 
Components: Mobile>Blimp

Comment 2 by w...@chromium.org, Nov 11 2016

Labels: Needs-Feedback
Owner: khushals...@chromium.org
Status: Assigned (was: Untriaged)
Khushal, can you elaborate on what you mean here? It sounds like this bug is really about remoting touch input, not really a threading thing?
Labels: -Needs-Feedback
Summary: Dealing with cases necessitating scrolling by blink. (was: Dealing with main thread input sent to the engine.)
The terminology in chrome for scrolling performed by blink is main thread scrolling. Sorry if it created confusion here.

The bug is about the cases where the scroll/fling gesture can not be handled by the compositor. In chrome, it would be sent to blink on the main thread, but that would be an extremely jarring experience for blimp, since this would mean sending these gestures to blink on the engine. The bug is about what should be done for blimp in these cases. Currently we send them to the engine.

The list of such cases can be found here: https://cs.chromium.org/chromium/src/cc/input/main_thread_scrolling_reason.h
Cc: rbyers@chromium.org aelias@chromium.org
Components: Blink>Scroll
Summary: The app chooser on google.com defers to main thread scrolling due to NonLayerViewportConstrainedObjects. (was: Dealing with cases necessitating scrolling by blink.)
I had originally filed this bug after I ran into this case with google.com. Go to google.com, click on the app chooser on the top right and try to scrolling on it goes to the main thread. Some digging and it looks like the app chooser on google.com defers to main thread scrolling because of NonLayerViewportConstrainedObjects.

+rbyers, aelias who are the experts with blink scrolling. Could you explain what this reason precisely means and what necessitates falling back to it? I think this is something we would really want to change for blimp, if possible. I also looked at UMA, and close to half the cases for main thread scrolling on android are for this reason.
Labels: Needs-Feedback
Summary: The app chooser on google.com defers to main thread scrolling due to NonFastScrollableRegion. (was: The app chooser on google.com defers to main thread scrolling due to NonLayerViewportConstrainedObjects.)
Sorry, my bad. The reason is different...
Status: WontFix (was: Assigned)
Obsolete, WontFix.
Labels: Archive-Blimp

Sign in to add a comment