Bill McCloskey [inactive unless it's an emergency] (:billm) Assignee
	Description • 11 years ago

I've been trying to get hardware acceleration working with OMTC on my Linux machine. Without this patch, I was getting errors about "X ERROR: BadValue" from a call to glXBindTexImageEXT. My nVidia driver doesn't like it when you call glXBindTexImageEXT twice on the same pixmap/buffer combination without a call to glXReleaseTexImageEXT in between.

This patch just does the release calls. It allows me to run with OMTC and accelerated layers enabled.

	Nick Cameron [:nrc]
	Comment 1 • 11 years ago

Comment on attachment 765063 [details] [diff] [review]
patch

Review of attachment 765063 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM with the nits fixed. Thanks for the patch!

::: gfx/layers/opengl/CompositorOGL.cpp
@@ +951,5 @@
>      return gl::RGBALayerProgramType;
>    }
>  }
>  
> +struct AutoBindTexture

MOZ_STACK_CLASS

@@ +956,5 @@
> +{
> +  AutoBindTexture() : mTexture(nullptr) {}
> +  AutoBindTexture(TextureSourceOGL* aTexture, GLenum aTextureUnit)
> +   : mTexture(nullptr) { Bind(aTexture, aTextureUnit); }
> +  ~AutoBindTexture() {

nit: new line for {

@@ +958,5 @@
> +  AutoBindTexture(TextureSourceOGL* aTexture, GLenum aTextureUnit)
> +   : mTexture(nullptr) { Bind(aTexture, aTextureUnit); }
> +  ~AutoBindTexture() {
> +    if (mTexture)
> +      mTexture->ReleaseTexture();

nit: {} please

	Bill McCloskey [inactive unless it's an emergency] (:billm) Assignee
	Comment 2 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/d33262333925

	Ed Morley [:emorley]
	Comment 3 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/d33262333925