tree 1c6255f02362774d3d22ebffd1e9c8b6a80a5b0c
parent 69b8cf01d984a70bc3cd738fbc6e6fbe40bb38cb
author Kevin Lubick <kjlubick@google.com> 1574195188 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> 1574252426 +0000

[gold] get rid of struct initializations for GoldenTrace

Saw another inexplicable nil dereference in a place I wouldn't have thought
it possible to be one. The only similarity to the other nil dereference was
that it was shortly after a call to gtr.TestName(), which made me think
that the pre-caching on that map lookup wasn't happening like I thought.

Thus I fixed the 3 call sites that initialized the struct directly
instead of calling the helper that makes sure it is filled out.

Change-Id: I79c32edabc0557e8d18d70993e3758b1555eab43
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/255307
Reviewed-by: Ben Wagner aka dogben <benjaminwagner@google.com>
Reviewed-by: Leandro Lovisolo <lovisolo@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>