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

Unified Diff: mojo/converters/surfaces/surfaces_type_converters.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: format Created 5 years, 1 month 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: mojo/converters/surfaces/surfaces_type_converters.cc
diff --git a/mojo/converters/surfaces/surfaces_type_converters.cc b/mojo/converters/surfaces/surfaces_type_converters.cc
index 5f2f25db8d12f4ec3efcd2aba407f847f52876bd..8908fc807f7189b4cbc3aa2af08a2b12fdb018a6 100644
--- a/mojo/converters/surfaces/surfaces_type_converters.cc
+++ b/mojo/converters/surfaces/surfaces_type_converters.cc
@@ -25,6 +25,7 @@
using mus::mojom::Color;
using mus::mojom::ColorPtr;
+using mus::mojom::CommandBufferNamespace;
using mus::mojom::CompositorFrame;
using mus::mojom::CompositorFramePtr;
using mus::mojom::CompositorFrameMetadata;
@@ -54,6 +55,8 @@ using mus::mojom::SurfaceId;
using mus::mojom::SurfaceIdPtr;
using mus::mojom::SurfaceQuadState;
using mus::mojom::SurfaceQuadStatePtr;
+using mus::mojom::SyncToken;
+using mus::mojom::SyncTokenPtr;
using mus::mojom::TextureQuadState;
using mus::mojom::TextureQuadStatePtr;
using mus::mojom::TileQuadState;
@@ -527,12 +530,38 @@ gpu::Mailbox TypeConverter<gpu::Mailbox, MailboxPtr>::Convert(
}
// static
+SyncTokenPtr TypeConverter<SyncTokenPtr, gpu::SyncToken>::Convert(
+ const gpu::SyncToken& input) {
+ DCHECK(!input.HasData() || input.verified_flush());
+ SyncTokenPtr sync_token(SyncToken::New());
+ sync_token->verified_flush = input.verified_flush();
+ sync_token->namespace_id =
+ static_cast<CommandBufferNamespace>(input.namespace_id());
+ sync_token->command_buffer_id = input.command_buffer_id();
+ sync_token->release_count = input.release_count();
+ return sync_token.Pass();
+}
+
+// static
+gpu::SyncToken TypeConverter<gpu::SyncToken, SyncTokenPtr>::Convert(
+ const SyncTokenPtr& input) {
+ const gpu::CommandBufferNamespace namespace_id =
+ static_cast<gpu::CommandBufferNamespace>(input->namespace_id);
+ gpu::SyncToken sync_token(namespace_id, input->command_buffer_id,
+ input->release_count);
+ if (input->verified_flush)
+ sync_token.SetVerifyFlush();
+
+ return sync_token;
+}
+
+// static
MailboxHolderPtr TypeConverter<MailboxHolderPtr, gpu::MailboxHolder>::Convert(
const gpu::MailboxHolder& input) {
MailboxHolderPtr holder(MailboxHolder::New());
holder->mailbox = Mailbox::From<gpu::Mailbox>(input.mailbox);
+ holder->sync_token = SyncToken::From<gpu::SyncToken>(input.sync_token);
holder->texture_target = input.texture_target;
- holder->sync_point = input.sync_point;
return holder.Pass();
}
@@ -541,8 +570,8 @@ gpu::MailboxHolder TypeConverter<gpu::MailboxHolder, MailboxHolderPtr>::Convert(
const MailboxHolderPtr& input) {
gpu::MailboxHolder holder;
holder.mailbox = input->mailbox.To<gpu::Mailbox>();
+ holder.sync_token = input->sync_token.To<gpu::SyncToken>();
holder.texture_target = input->texture_target;
- holder.sync_point = input->sync_point;
return holder;
}
@@ -603,7 +632,7 @@ TypeConverter<ReturnedResourcePtr, cc::ReturnedResource>::Convert(
const cc::ReturnedResource& input) {
ReturnedResourcePtr returned = ReturnedResource::New();
returned->id = input.id;
- returned->sync_point = input.sync_point;
+ returned->sync_token = SyncToken::From<gpu::SyncToken>(input.sync_token);
returned->count = input.count;
returned->lost = input.lost;
return returned.Pass();
@@ -615,7 +644,7 @@ TypeConverter<cc::ReturnedResource, ReturnedResourcePtr>::Convert(
const ReturnedResourcePtr& input) {
cc::ReturnedResource returned;
returned.id = input->id;
- returned.sync_point = input->sync_point;
+ returned.sync_token = input->sync_token.To<gpu::SyncToken>();
returned.count = input->count;
returned.lost = input->lost;
return returned;
« no previous file with comments | « mojo/converters/surfaces/surfaces_type_converters.h ('k') | mojo/converters/surfaces/tests/surface_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698