New issue
Advanced search Search tips

Issue 859924 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

ScrollInertial metrics regression.

Project Member Reported by sahel@chromium.org, Jul 3

Issue description

There seems to be regressions on ScrollInertial metrics in this range:
https://chromium.googlesource.com/chromium/src/+log/69.0.3453.0..69.0.3457.0?pretty=fuller&n=10000

https://uma.googleplex.com/p/chrome/timeline_v2/?sid=5d0560fb8983fc37877227afdfd9d6b7

The 50th percentile graphs show regressions in these metrics:
HandledToRendererSwap2_Impl/Main
RendererSwapToBrowserNotified2
TimeToHandled2_Impl/Main
TimeToScrollUpdateSwapBegin2

 
Owner: samans@chromium.org
Status: Assigned (was: Available)
https://chromium-review.googlesource.com/1089144 from the regression range can explain changes in TimeToHandled2_Impl/Main and partially the end to end metric: TimeToScrollUpdateSwapBegin2

samans@ do you have any idea how this cl (or any other cl from the regression list) might cause the regressions in RendererSwapToBrowserNotified2 and HandledToRendererSwap2_Impl/Main?
I don't know enough about these metrics or how fling is handled renderer-side to comment. Probably the same issue with BeginFrames that you saw on Aura platforms? Have you come up with a solution for that issue?
Cc: sadrul@chromium.org
>I don't know enough about these metrics or how fling is handled renderer-side to comment.
The fling is not handled on the renderer side anymore. If you mean the generated GSU events from the fling controller, they are handled identically to GSU events from normal scrolling.

>Probably the same issue with BeginFrames that you saw on Aura platforms? Have you come up with a solution for that issue?
I am not sure what you are referring at.. Are you talking about the regression that happened on Aura when we switched to OnAnimiationStep instead of setNeedsBeginFrame?

+sadrul@ do you know how using OnAnimate and SetNeedsAnimate to progress fling can cause regressions on RendererSwapToBrowserNotified2 and HandledToRendererSwap2_Impl/Main metrics?



> If you mean the generated GSU events...
Yes I mean generally I'm not very familiar with how input is handled renderer-side.

> Are you talking about the regression that
Yes, probably something similar has happened here. If the events arrive after BeginFrame, wouldn't these regressions happen?

Sign in to add a comment