GitLab

With DCHECKs enabled base::HashInts uses a random seed, which is only
valid in its process. This means that a given viz::LocalSurfaceId would
have different hashes in different process. Which breaks performing
Traces with Flow.

Additionally while debugging issue 1060058 I noticed it was difficult to
know when Viz was blocked on pending surfaces, and when rotations were
occurring. This change adds a new trace to viz::Surface to list the
Surface being waited on. As well as adding new traces to LayerTreeHost
to denote the current viewport rect.

Bug: 1154826
Change-Id: I60d9f2f195ae6ca45e9eabd101df971462b34224
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633098
Commit-Queue: Jonathan Ross <jonross@chromium.org>
Reviewed-by: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844783}