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

Unified Diff: gpu/command_buffer/client/gles2_implementation.cc

Issue 1427543002: Modified old wait sync point functions to also accept new sync tokens. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added DCHECK for valid sync token before IPC conversions Created 5 years, 2 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/client/gles2_implementation.cc
diff --git a/gpu/command_buffer/client/gles2_implementation.cc b/gpu/command_buffer/client/gles2_implementation.cc
index f0b8f8f1335ab0807eb189dffd4747c4a93486cc..a35587db3d06c3ed11fc9c7f2dad9fa3d3d53c6f 100644
--- a/gpu/command_buffer/client/gles2_implementation.cc
+++ b/gpu/command_buffer/client/gles2_implementation.cc
@@ -5366,6 +5366,11 @@ GLuint GLES2Implementation::InsertSyncPointCHROMIUM() {
return gpu_control_->InsertSyncPoint();
}
+void GLES2Implementation::WaitSyncPointCHROMIUM(GLuint sync_point) {
+ // This should no longer be called.
+ NOTREACHED();
+}
+
GLuint GLES2Implementation::InsertFutureSyncPointCHROMIUM() {
GPU_CLIENT_SINGLE_THREAD_CHECK();
GPU_CLIENT_LOG("[" << GetLogPrefix() << "] glInsertFutureSyncPointCHROMIUM");
@@ -5453,9 +5458,20 @@ void GLES2Implementation::WaitSyncTokenCHROMIUM(const GLbyte* sync_token) {
return;
}
- helper_->WaitSyncTokenCHROMIUM(sync_token_data.namespace_id(),
- sync_token_data.command_buffer_id(),
- sync_token_data.release_count());
+ // TODO(dyen): Temporarily support old sync points, remove once all old sync
+ // points have been removed.
+ const gpu::CommandBufferNamespace namespace_id =
+ sync_token_data.namespace_id();
+ if (namespace_id == gpu::CommandBufferNamespace::OLD_SYNC_POINTS) {
+ const uint32_t sync_point =
+ static_cast<uint32_t>(sync_token_data.release_count());
+ helper_->WaitSyncPointCHROMIUM(sync_point);
+ return;
+ }
+
+ helper_->WaitSyncTokenCHROMIUM(
+ static_cast<GLuint>(sync_token_data.namespace_id()),
+ sync_token_data.command_buffer_id(), sync_token_data.release_count());
}
namespace {

Powered by Google App Engine
This is Rietveld 408576698