diff --git a/gfx/webrender_bindings/RenderDMABUFTextureHost.cpp b/gfx/webrender_bindings/RenderDMABUFTextureHost.cpp index ad6c97e0dd1a..4c2d7701eb6f 100644 --- a/gfx/webrender_bindings/RenderDMABUFTextureHost.cpp +++ b/gfx/webrender_bindings/RenderDMABUFTextureHost.cpp @@ -24,6 +24,17 @@ RenderDMABUFTextureHost::~RenderDMABUFTextureHost() { wr::WrExternalImage RenderDMABUFTextureHost::Lock(uint8_t aChannelIndex, gl::GLContext* aGL) { + const gfx::IntSize size(mSurface->GetWidth(aChannelIndex), + mSurface->GetHeight(aChannelIndex)); + + // Wayland native compositor doesn't use textures so pass zero + // there. It saves GPU resources. + if (!aGL) { + return NativeTextureToWrExternalImage(0, 0.0, 0.0, + static_cast(size.width), + static_cast(size.height)); + } + if (mGL.get() != aGL) { if (mGL) { // This should not happen. EGLImage is created only in @@ -50,8 +61,6 @@ wr::WrExternalImage RenderDMABUFTextureHost::Lock(uint8_t aChannelIndex, mSurface->MaybeSemaphoreWait(texture); } - const gfx::IntSize size(mSurface->GetWidth(aChannelIndex), - mSurface->GetHeight(aChannelIndex)); return NativeTextureToWrExternalImage( mSurface->GetTexture(aChannelIndex), 0.0, 0.0, static_cast(size.width), static_cast(size.height)); @@ -81,10 +90,6 @@ void RenderDMABUFTextureHost::ClearCachedResources() { mGL = nullptr; } -gfx::SurfaceFormat RenderDMABUFTextureHost::GetFormat() const { - return mSurface->GetFormat(); -} - bool RenderDMABUFTextureHost::MapPlane(RenderCompositor* aCompositor, uint8_t aChannelIndex, PlaneInfo& aPlaneInfo) { diff --git a/gfx/webrender_bindings/RenderDMABUFTextureHost.h b/gfx/webrender_bindings/RenderDMABUFTextureHost.h index 33d0fad75322..ad97bb9feb0a 100644 --- a/gfx/webrender_bindings/RenderDMABUFTextureHost.h +++ b/gfx/webrender_bindings/RenderDMABUFTextureHost.h @@ -37,7 +37,9 @@ class RenderDMABUFTextureHost final : public RenderTextureHostSWGL { RefPtr GetSurface() { return mSurface; } // RenderTextureHostSWGL - gfx::SurfaceFormat GetFormat() const override; + gfx::SurfaceFormat GetFormat() const override { + return mSurface->GetFormat(); + } gfx::ColorDepth GetColorDepth() const override { return gfx::ColorDepth::COLOR_8; } diff --git a/gfx/webrender_bindings/RendererOGL.cpp b/gfx/webrender_bindings/RendererOGL.cpp index 44100a2d5560..ccd8bb80fabf 100644 --- a/gfx/webrender_bindings/RendererOGL.cpp +++ b/gfx/webrender_bindings/RendererOGL.cpp @@ -69,6 +69,16 @@ wr::WrExternalImage wr_renderer_lock_external_image(void* aObj, << AsUint64(aId); return InvalidToWrExternalImage(); } + +#if defined(MOZ_WAYLAND) + // Wayland native compositor doesn't use textures so pass null GL context. + if (texture->AsRenderDMABUFTextureHost() && + renderer->GetCompositor()->CompositorType() == + layers::WebRenderCompositor::WAYLAND) { + return texture->Lock(aChannelIndex, nullptr); + } +#endif + if (auto* gl = renderer->gl()) { return texture->Lock(aChannelIndex, gl); } else if (auto* swgl = renderer->swgl()) {