Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(253)

Unified Diff: gpu/command_buffer/service/query_manager_unittest.cc

Issue 1687353002: Force time elapsed queries on certain drivers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add bug to commit message Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: gpu/command_buffer/service/query_manager_unittest.cc
diff --git a/gpu/command_buffer/service/query_manager_unittest.cc b/gpu/command_buffer/service/query_manager_unittest.cc
index 8f61ff22bd606af859e2ab3bfcc4e5985b91917d..cbf2d8918945cf540e8323bb714b0e57c7749017 100644
--- a/gpu/command_buffer/service/query_manager_unittest.cc
+++ b/gpu/command_buffer/service/query_manager_unittest.cc
@@ -814,7 +814,7 @@ TEST_F(QueryManagerTest, TimeStampQuery) {
const uint64_t expected_result =
100u * base::Time::kNanosecondsPerMicrosecond;
fake_timing_queries.SetCurrentGLTime(expected_result);
- fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false);
+ fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false, true);
EXPECT_TRUE(manager_->QueryCounter(query, kSubmitCount));
EXPECT_TRUE(manager_->ProcessPendingQueries(false));
@@ -852,7 +852,7 @@ TEST_F(QueryManagerManualSetupTest, TimeStampDisjoint) {
// Disjoint happening before the query should not trigger a disjoint event.
fake_timing_queries.SetDisjoint();
- fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false);
+ fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false, true);
EXPECT_TRUE(manager_->QueryCounter(query, kSubmitCount));
EXPECT_TRUE(manager_->ProcessPendingQueries(false));
@@ -860,7 +860,7 @@ TEST_F(QueryManagerManualSetupTest, TimeStampDisjoint) {
EXPECT_EQ(current_disjoint_value, disjoint_sync->GetDisjointCount());
// Disjoint happening during query should trigger disjoint event.
- fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false);
+ fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false, false);
EXPECT_TRUE(manager_->QueryCounter(query, kSubmitCount));
fake_timing_queries.SetDisjoint();
EXPECT_TRUE(manager_->ProcessPendingQueries(false));
@@ -900,7 +900,7 @@ TEST_F(QueryManagerManualSetupTest, DisjointContinualTest) {
kTarget, kClient1Id, kSharedMemoryId, kSharedMemoryOffset);
ASSERT_TRUE(query != NULL);
- fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false);
+ fake_timing_queries.ExpectGPUTimeStampQuery(*gl_, false, true);
EXPECT_TRUE(manager_->QueryCounter(query, kSubmitCount));
EXPECT_TRUE(manager_->ProcessPendingQueries(false));

Powered by Google App Engine
This is Rietveld 408576698