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

Unified Diff: cc/trees/layer_tree_host_unittest_context.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
« no previous file with comments | « cc/trees/layer_tree_host_pixeltest_readback.cc ('k') | cc/trees/layer_tree_host_unittest_copyrequest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_unittest_context.cc
diff --git a/cc/trees/layer_tree_host_unittest_context.cc b/cc/trees/layer_tree_host_unittest_context.cc
index 624009d8988db0f4fea2c55e617cf33a95e9382e..514727598d25f9985ee00b2c3744eda795c3eb96 100644
--- a/cc/trees/layer_tree_host_unittest_context.cc
+++ b/cc/trees/layer_tree_host_unittest_context.cc
@@ -910,7 +910,8 @@ class LayerTreeHostContextTestDontUseLostResources
child_output_surface_.get(), shared_bitmap_manager_.get());
}
- static void EmptyReleaseCallback(unsigned sync_point, bool lost) {}
+ static void EmptyReleaseCallback(const gpu::SyncToken& sync_token,
+ bool lost) {}
void SetupTree() override {
gpu::gles2::GLES2Interface* gl =
@@ -949,7 +950,7 @@ class LayerTreeHostContextTestDontUseLostResources
gpu::Mailbox mailbox;
gl->GenMailboxCHROMIUM(mailbox.name);
- GLuint sync_point = gl->InsertSyncPointCHROMIUM();
+ gpu::SyncToken sync_token(gl->InsertSyncPointCHROMIUM());
scoped_refptr<Layer> root = Layer::Create(layer_settings());
root->SetBounds(gfx::Size(10, 10));
@@ -973,10 +974,10 @@ class LayerTreeHostContextTestDontUseLostResources
texture->SetBounds(gfx::Size(10, 10));
texture->SetIsDrawable(true);
texture->SetTextureMailbox(
- TextureMailbox(mailbox, GL_TEXTURE_2D, sync_point),
+ TextureMailbox(mailbox, sync_token, GL_TEXTURE_2D),
SingleReleaseCallback::Create(
base::Bind(&LayerTreeHostContextTestDontUseLostResources::
- EmptyReleaseCallback)));
+ EmptyReleaseCallback)));
root->AddChild(texture);
scoped_refptr<PictureLayer> mask =
@@ -1012,12 +1013,12 @@ class LayerTreeHostContextTestDontUseLostResources
gfx::Size(4, 4), 0x80, 0x80, 0x80, base::TimeDelta());
hw_video_frame_ = VideoFrame::WrapNativeTexture(
media::PIXEL_FORMAT_ARGB,
- gpu::MailboxHolder(mailbox, GL_TEXTURE_2D, sync_point),
+ gpu::MailboxHolder(mailbox, sync_token, GL_TEXTURE_2D),
media::VideoFrame::ReleaseMailboxCB(), gfx::Size(4, 4),
gfx::Rect(0, 0, 4, 4), gfx::Size(4, 4), base::TimeDelta());
scaled_hw_video_frame_ = VideoFrame::WrapNativeTexture(
media::PIXEL_FORMAT_ARGB,
- gpu::MailboxHolder(mailbox, GL_TEXTURE_2D, sync_point),
+ gpu::MailboxHolder(mailbox, sync_token, GL_TEXTURE_2D),
media::VideoFrame::ReleaseMailboxCB(), gfx::Size(4, 4),
gfx::Rect(0, 0, 3, 2), gfx::Size(4, 4), base::TimeDelta());
« no previous file with comments | « cc/trees/layer_tree_host_pixeltest_readback.cc ('k') | cc/trees/layer_tree_host_unittest_copyrequest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698