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

Unified Diff: cc/resources/video_resource_updater.cc

Issue 2341743004: Remove unnecessary sync token on CopyPlaneTexture returned Resource. (Closed)
Patch Set: Created 4 years, 3 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/video_resource_updater.cc
diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc
index be382ce4084c433ee3f8fa4052ab6af9a243c203..c376541bd6597c1c3d8fcc69e877d81e0a96beaa 100644
--- a/cc/resources/video_resource_updater.cc
+++ b/cc/resources/video_resource_updater.cc
@@ -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);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698