Commit b0d27dba authored by tobiasjs's avatar tobiasjs Committed by Commit bot

Remove unnecessary sync token on CopyPlaneTexture returned Resource.

Because the compositor shares a context with the video resource updater,
the copy target doesn't require a sync token.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Review-Url: https://codereview.chromium.org/2341743004
Cr-Commit-Position: refs/heads/master@{#419138}
parent c68e2320
......@@ -638,16 +638,12 @@ void VideoResourceUpdater::CopyPlaneTexture(
false);
gl->DeleteTextures(1, &src_texture_id);
// Sync point for use of frame copy.
gpu::SyncToken sync_token;
const uint64_t fence_sync = gl->InsertFenceSyncCHROMIUM();
gl->ShallowFlushCHROMIUM();
gl->GenSyncTokenCHROMIUM(fence_sync, sync_token.GetData());
// Done with the source video frame texture at this point.
video_frame->UpdateReleaseSyncToken(&client);
TextureMailbox mailbox(resource->mailbox(), sync_token, GL_TEXTURE_2D,
// VideoResourceUpdater shares a context with the compositor so a
// sync token is not required.
TextureMailbox mailbox(resource->mailbox(), gpu::SyncToken(), GL_TEXTURE_2D,
video_frame->coded_size(), false, false);
mailbox.set_color_space(video_frame->ColorSpace());
external_resources->mailboxes.push_back(mailbox);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment