tree c98b9404555fc79e2367379cd6178fbb3ff95029
parent 2943ba28a03c8c38d30cf3631dce75485320e819
author Chris Craik <ccraik@google.com> 1631747400 -0700
committer Chris Craik <ccraik@google.com> 1631751640 -0700

Trace-based FrameTimingMetric, Step 2

Bug: 195987329
Bug: 183129298
Fixes: 200082324
Test: TrivialListScrollBenchmark # Validated on 29 & 31
Test: ./gradlew benchmark:benchmark-common:cC benchmark:benchmark-macro:cC

Completes migration of FrameTimingMetric to be trace-based, and
surface FrameNegativeSlack when available.

This change also makes several related changes to frame metrics coming
from FrameTimingQuery:

* Deleted FrameTime, as it's always preferable to use something
  deadline based

* Renamed BasicFrameTime -> FrameCpuTime for clarity, and to match metric plan

* Made FrameSlack negative, so the *worst* frames show up in 99P,
  instead of best. This avoids having to differentate between low=bad
  and high=bad metrics.

Adds FrameTimingGfxInfoMetric to existing scroll benchmarks for
temporary continuity in CI tracking.

Change-Id: Ied56278b1f76f37df44181dc736590753aa1cb30