tree 693445574c215aed76f8ef3e75824b8183585b88
parent 166c9d5d561d72bfbfacb99671d99675eb9e7fac
author Stephen White <senorblanco@chromium.org> 1715665258 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> 1715665258 +0000

OpenGL: fix spurious EGLSync on queue destruction.

During Device::Destroy(), Queue::WaitForIdleForDestruction() is
called. In the GL backend, this calls gl.Finish(). But the
"GetGL()" call used to call gl.Finish() causes mHasPendingCommands
to be set to true.

Later in Device::Destroy(), TickImpl() is called one last time
to release resources. In the GL case, since mHasPendingCommands
is true, this inserts a new EGLSync which is never destroyed or
waited on.

The fix is to set mHasPendingCommands to false after gl.Finish().
Leaving no pending commands is the very definition of glFinish(),
so it's safe to do.

Bug: 340202279
Change-Id: I3625363018b564fd4409c957576ab9dfe5917d7f
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/188061
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Stephen White <senorblanco@chromium.org>