commit | 882579082ed74cc8642b7e23e5d27c2b4242124e | [log] [tgz] |
---|---|---|
author | Saman Sami <samans@chromium.org> | Fri Mar 22 17:47:39 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Mar 22 17:47:39 2019 |
tree | 1d9982976ca1f5a39c0d9164a3d8c70c8171673c | |
parent | f14299dbed3ca54a9079ee09538fa9af04e1b778 [diff] |
Don't carry over LatencyInfo across surfaces The LatencyInfo in a CompositorFrame is supposed to be processed when that CompositorFrame or a newer one is shown. SetPreviousFrameSurface carries over the LatencyInfo of the previous surface into the new one. This means if the previous surface is shown before the new surface, it will not have any LatencyInfo on it that we can process, and the delay in processing the LatencyInfo can cause artificial metric regressions. In order to fix this, don't automatically carry over LatencyInfo in SetPreviousFrameSurface. Instead, add a method to SurfaceAllocationGroup that aggregates the LatencyInfo on all surfaces up to a particular surface. Bug: 939911 Change-Id: I5deb6c3fd0bd84b47380ba9caf03d4b199ac2813 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1532669 Reviewed-by: Jonathan Ross <jonross@chromium.org> Commit-Queue: Saman Sami <samans@chromium.org> Cr-Commit-Position: refs/heads/master@{#643434}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .