| commit | ce2671bf523d439ed7db7a606e871ca0416ef7a6 | |
| author | mithro <mithro@mithis.com> | |
| Fri, 8 May 2015 03:13:04 +0000 (7 20:13 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Fri, 8 May 2015 03:13:53 +0000 (8 03:13 +0000) | ||
| tree | 0d1dc192145ef54b682aad699123e8ed27bdb2cd | treesnapshot (tar.gz zip) |
| parent | 0a8bcc4f85e1b3ca300a133e36f5966dee468acc | commitdiff |
cc: Adding BeginFrameTracker object and removing Now() from LTHI.
**Sheriffs**: If this patch is making a test flaky / fail,
then the test was already broken and this just makes the
problem visible.
This patch reduces the jitter in LTHI's animation time to
be identical to the vsync jitter rather than being affected
by the system load and OS scheduling delays.
BeginFrameTracker uses DCHECKs and TRACE_EVENT to strictly
track how the BeginFrameArgs are used throughout the LTHI.
Using this information incorrect usage of Now() has been
eliminated. Methods which violate correct usage of
BeginFrameArgs are now clearly marked to prevent
regressions.
BUG=346230
Review URL: https://codereview.chromium.org/787763006
Cr-Commit-Position: refs/heads/master@{#328911}
**Sheriffs**: If this patch is making a test flaky / fail,
then the test was already broken and this just makes the
problem visible.
This patch reduces the jitter in LTHI's animation time to
be identical to the vsync jitter rather than being affected
by the system load and OS scheduling delays.
BeginFrameTracker uses DCHECKs and TRACE_EVENT to strictly
track how the BeginFrameArgs are used throughout the LTHI.
Using this information incorrect usage of Now() has been
eliminated. Methods which violate correct usage of
BeginFrameArgs are now clearly marked to prevent
regressions.
BUG=346230
Review URL: https://codereview.chromium.org/787763006
Cr-Commit-Position: refs/heads/master@{#328911}
16 files changed: