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

Unified Diff: media/gpu/avda_codec_image.cc

Issue 2014313002: StreamTextureImages can now override a Texture's service id (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Only clear the stream texture service id if the image is changed Created 4 years, 7 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: media/gpu/avda_codec_image.cc
diff --git a/media/gpu/avda_codec_image.cc b/media/gpu/avda_codec_image.cc
index 70b6e47fc5beb1273822f74fe1d155a6da36cd89..e89d4cb35e91441822ecbf02a89a081b01edba59 100644
--- a/media/gpu/avda_codec_image.cc
+++ b/media/gpu/avda_codec_image.cc
@@ -84,8 +84,9 @@ bool AVDACodecImage::CopyTexImage(unsigned target) {
// CopyTexImage() is called each time the surface texture is used for drawing.
// It would be nice if we could do this via asking for the currently bound
// Texture, but the active unit never seems to change.
- texture_->SetLevelStreamTextureImage(GL_TEXTURE_EXTERNAL_OES, 0, this,
- gpu::gles2::Texture::UNBOUND);
+ texture_->SetLevelStreamTextureImage(
liberato (no reviews please) 2016/06/02 14:37:02 i think that this would be clearer as SetLevelImag
watk 2016/06/03 01:09:46 Done.
+ GL_TEXTURE_EXTERNAL_OES, 0, this, gpu::gles2::Texture::UNBOUND,
+ shared_state_->surface_texture_service_id());
return true;
}

Powered by Google App Engine
This is Rietveld 408576698