commit 08c1dbb334a42cddf3ad8db21a59f3c19f2ae99d
author Gregg Tavares <gman@chromium.org> Sun Dec 07 14:14:23 2025 -0800
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Sun Dec 07 14:14:23 2025 -0800
tree 1d666ef9ed182dc5d84d5fca2644928d401bc6f1
parent 577d50da861223a23317fbc1cd9811595ab65205

Track Command Encoder QuerySets

Even with validation off, the querysets need to be
tracked for proper synchronization.

There's no good way to test this AFAICT as not
syncrhonizing just means the value could be incorrect.

Bug: 464001949
Change-Id: Iba3d9b75ddd39a4e89c823f3be18d7cd6a6a6964
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/278855
Reviewed-by: Shrek Shao <shrekshao@google.com>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Gregg Tavares <gman@chromium.org>

src/dawn/native/CommandEncoder.cpp

diff --git a/src/dawn/native/CommandEncoder.cpp b/src/dawn/native/CommandEncoder.cpp
index 60c777c..74fd70f 100644
--- a/src/dawn/native/CommandEncoder.cpp
+++ b/src/dawn/native/CommandEncoder.cpp
@@ -1257,9 +1257,7 @@
 void CommandEncoder::TrackQueryAvailability(QuerySetBase* querySet, uint32_t queryIndex) {
     DAWN_ASSERT(querySet != nullptr);
 
-    if (GetDevice()->IsValidationEnabled()) {
-        TrackUsedQuerySet(querySet);
-    }
+    TrackUsedQuerySet(querySet);
 
     // Set the query at queryIndex to available for resolving in query set.
     querySet->SetQueryAvailability(queryIndex, true);