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

Unified Diff: cc/resources/video_resource_updater.cc

Issue 2639973002: Add target argument to Copy{Sub}TextureCHROMIUM entry point (Closed)
Patch Set: use dest_target instead of target Created 3 years, 11 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: cc/resources/video_resource_updater.cc
diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc
index a8068f9f67e6b175a9cafd1f6eafeac7495e24bb..cb6b6c189dcf60d0bfa976d679dc34969b551f18 100644
--- a/cc/resources/video_resource_updater.cc
+++ b/cc/resources/video_resource_updater.cc
@@ -713,10 +713,10 @@ void VideoResourceUpdater::CopyPlaneTexture(
gl->WaitSyncTokenCHROMIUM(mailbox_holder.sync_token.GetConstData());
uint32_t src_texture_id = gl->CreateAndConsumeTextureCHROMIUM(
mailbox_holder.texture_target, mailbox_holder.mailbox.name);
- gl->CopySubTextureCHROMIUM(src_texture_id, 0, lock.texture_id(), 0, 0, 0, 0,
- 0, output_plane_resource_size.width(),
- output_plane_resource_size.height(), false, false,
- false);
+ gl->CopySubTextureCHROMIUM(
+ src_texture_id, 0, GL_TEXTURE_2D, lock.texture_id(), 0, 0, 0, 0, 0,
+ output_plane_resource_size.width(), output_plane_resource_size.height(),
+ false, false, false);
gl->DeleteTextures(1, &src_texture_id);
// Done with the source video frame texture at this point.

Powered by Google App Engine
This is Rietveld 408576698