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

Unified Diff: cc/resources/resource_provider_unittest.cc

Issue 2555743004: Delay activation/draw on GPU tile work completion (Closed)
Patch Set: rebase compile fix Created 3 years, 11 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
« no previous file with comments | « cc/resources/resource_provider.cc ('k') | cc/test/fake_raster_buffer_provider.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/resource_provider_unittest.cc
diff --git a/cc/resources/resource_provider_unittest.cc b/cc/resources/resource_provider_unittest.cc
index 33512829dd4524b6d2a0b87fcaef2df7c4e3fcd9..c4777b70db207e627b827815a99068c0dc5d0ad0 100644
--- a/cc/resources/resource_provider_unittest.cc
+++ b/cc/resources/resource_provider_unittest.cc
@@ -3784,5 +3784,35 @@ TEST(ResourceProviderTest, TextureAllocationChunkSize) {
}
}
+TEST_P(ResourceProviderTest, GetSyncTokenForResources) {
+ if (GetParam() != ResourceProvider::RESOURCE_TYPE_GL_TEXTURE)
+ return;
+
+ gfx::Size size(1, 1);
+ ResourceFormat format = RGBA_8888;
+
+ // ~Random set of |release_count|s to set on sync tokens.
+ uint64_t release_counts[5] = {7, 3, 10, 2, 5};
+
+ ResourceProvider::ResourceIdArray array;
+ for (uint32_t i = 0; i < arraysize(release_counts); ++i) {
+ ResourceId id = resource_provider_->CreateResource(
+ size, ResourceProvider::TEXTURE_HINT_IMMUTABLE, format,
+ gfx::ColorSpace());
+ array.push_back(id);
+
+ ResourceProvider::ScopedWriteLockGL lock(resource_provider_.get(), id,
+ false);
+ gpu::SyncToken token;
+ token.Set(gpu::CommandBufferNamespace::INVALID, 0, gpu::CommandBufferId(),
+ release_counts[i]);
+ lock.set_sync_token(token);
+ }
+
+ gpu::SyncToken last_token =
+ resource_provider_->GetSyncTokenForResources(array);
+ EXPECT_EQ(last_token.release_count(), 10u);
+}
+
} // namespace
} // namespace cc
« no previous file with comments | « cc/resources/resource_provider.cc ('k') | cc/test/fake_raster_buffer_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698