DescriptionRevert 287325 because it breaks smoothness benchmarks on windows: "ValueError: LatencyInfo has no begin component"
> Implement scroll handler latency tracking
>
> Since scroll handlers do not block composited scrolling, the existing
> input latency metric will only report the response time for the
> compositor when scrolling a page with a handler. This is not accurate
> for pages that implement scroll-synchronized effects, because the
> scroll handler needs to be invoked so that the page contents are
> updated with the new scroll offset.
>
> This patch fixes the problem by forwarding latency information from the
> impl to the main thread when we are scrolling a layer that has an active
> scroll event handler.
>
> In the case the scroll handler is no-op, the latency information will be
> terminated as a failed commit.
>
> Sample trace:
> https://drive.google.com/file/d/0ByyxMXB38gLDRE40bkU5TVd0U00/edit?usp=sharing
>
> BUG=347366
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282117
>
> Review URL: https://codereview.chromium.org/365463003
TBR=skyostil@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287458
Patch Set 1 #Messages
Total messages: 2 (0 generated)
|