tree 89f2fb55aaf1a3d19633b6409a859c36c44182d5
parent bf375ef672033cbdeaf56ae59b26d515ba34e63c
author Jim Van Verth <jvanverth@google.com> 1685111181 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> 1685120569 +0000

[graphite] Add extraResourceRefs before adding commands to cmdBuffer.

In some cases we fail while trying add commands from the Recording.
However, we've already added some commands that use resources, and
some of those might be in the fExtraResourcesRefs list and so haven't
had their commandBufferRefs incremented. Hence when we tear down the
Recorder after failing to add the Recording, it will shutdown the
ResourceCache and delete those resources since they don't have a
commandBufferRef. Trying to subsequently end the command buffer in
this state (necessary prior to deleting it) makes the Vulkan validator
very unhappy. The solution is to track the extraResourceRefs before
trying to add commands.

Bug: b/277583609
Change-Id: Iaad66a3d3ab7f764a405964564ffb3e7336805c9
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/703680
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>